From baa982daf1df57229f5e91741ebc4f23ecb31275 Mon Sep 17 00:00:00 2001
From: Michael Jumper <zhangmaike@users.sourceforge.net>
Date: Mon, 24 Dec 2012 13:19:13 -0800
Subject: [PATCH] DocumentHandlerState should be static. Automatically throw
 exception if no TagHandler returned.

---
 .../guacamole/net/basic/xml/DocumentHandler.java            | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/guacamole/src/main/java/net/sourceforge/guacamole/net/basic/xml/DocumentHandler.java b/guacamole/src/main/java/net/sourceforge/guacamole/net/basic/xml/DocumentHandler.java
index 36754306e..6eb07a0cc 100644
--- a/guacamole/src/main/java/net/sourceforge/guacamole/net/basic/xml/DocumentHandler.java
+++ b/guacamole/src/main/java/net/sourceforge/guacamole/net/basic/xml/DocumentHandler.java
@@ -106,6 +106,10 @@ public class DocumentHandler extends DefaultHandler {
             handler = parent_handler.childElement(localName, attributes);
         }
 
+        // If no handler returned, the element was not expected
+        if (handler == null)
+            throw new SAXException("Unexpected element: '" + localName + "'");
+        
         // Append new element state to stack
         stack.addLast(new DocumentHandlerState(handler));
         
@@ -136,7 +140,7 @@ public class DocumentHandler extends DefaultHandler {
     /**
      * The current state of the DocumentHandler.
      */
-    private class DocumentHandlerState {
+    private static class DocumentHandlerState {
 
         /**
          * The current text content of the current element being parsed.