[java-xmltooling COMMIT] /branches/REL_1/src/main/java/org/opensaml/xml/parse/BasicParserPool.java

noreply at shibboleth.net noreply at shibboleth.net
Fri Mar 13 21:48:49 EDT 2015


Author: putmanb
Date: Fri Mar 13 21:48:49 2015
New Revision: 850

URL: http://svn.shibboleth.net/view/java-xmltooling?rev=850&view=rev
Log:
Fix ordering issue.

Modified:
    branches/REL_1/src/main/java/org/opensaml/xml/parse/BasicParserPool.java

Modified: branches/REL_1/src/main/java/org/opensaml/xml/parse/BasicParserPool.java
URL: http://svn.shibboleth.net/view/java-xmltooling/branches/REL_1/src/main/java/org/opensaml/xml/parse/BasicParserPool.java?rev=850&r1=849&r2=850&view=diff
==============================================================================
--- branches/REL_1/src/main/java/org/opensaml/xml/parse/BasicParserPool.java (original)
+++ branches/REL_1/src/main/java/org/opensaml/xml/parse/BasicParserPool.java Fri Mar 13 21:48:49 2015
@@ -169,7 +169,7 @@
             return new DocumentBuilderProxy(builder, this, version);
         }
 
-        return null;
+        throw new XMLParserException("Unable to obtain a DocumentBuilder");
     }
 
     /** {@inheritDoc} */
@@ -208,6 +208,9 @@
         DocumentBuilder builder = getBuilder();
         Document document = builder.newDocument();
         returnBuilder(builder);
+        if (document == null) {
+            throw new XMLParserException("DocumentBuilder returned a null Document");
+        }
         return document;
     }
 
@@ -216,6 +219,9 @@
         DocumentBuilder builder = getBuilder();
         try {
             Document document = builder.parse(input);
+            if (document == null) {
+                throw new XMLParserException("DocumentBuilder parsed a null Document");
+            }
             return document;
         } catch (SAXException e) {
             throw new XMLParserException("Invalid XML", e);
@@ -231,6 +237,9 @@
         DocumentBuilder builder = getBuilder();
         try {
             Document document = builder.parse(new InputSource(input));
+            if (document == null) {
+                throw new XMLParserException("DocumentBuilder parsed a null Document");
+            }
             return document;
         } catch (SAXException e) {
             throw new XMLParserException("Invalid XML", e);



More information about the commits mailing list