You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ws.apache.org by co...@apache.org on 2014/03/12 16:40:23 UTC

svn commit: r1576787 - /webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/saml/ext/OpenSAMLUtil.java

Author: coheigea
Date: Wed Mar 12 15:40:22 2014
New Revision: 1576787

URL: http://svn.apache.org/r1576787
Log:
Prevent possible NPE

Modified:
    webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/saml/ext/OpenSAMLUtil.java

Modified: webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/saml/ext/OpenSAMLUtil.java
URL: http://svn.apache.org/viewvc/webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/saml/ext/OpenSAMLUtil.java?rev=1576787&r1=1576786&r2=1576787&view=diff
==============================================================================
--- webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/saml/ext/OpenSAMLUtil.java (original)
+++ webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/saml/ext/OpenSAMLUtil.java Wed Mar 12 15:40:22 2014
@@ -165,7 +165,7 @@ public final class OpenSAMLUtil {
             }
         } finally {
             if (frag != null) {
-                if (doc.getDocumentElement() != null
+                if (doc.getDocumentElement() != null && element != null
                     && doc.getDocumentElement() != element) {
                     doc.getDocumentElement().removeChild(element);
                 }