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/02/13 15:01:21 UTC

svn commit: r1567930 - /webservices/wss4j/trunk/ws-security-common/src/main/java/org/apache/wss4j/common/saml/SAMLUtil.java

Author: coheigea
Date: Thu Feb 13 14:01:20 2014
New Revision: 1567930

URL: http://svn.apache.org/r1567930
Log:
Avoid potential NPE

Modified:
    webservices/wss4j/trunk/ws-security-common/src/main/java/org/apache/wss4j/common/saml/SAMLUtil.java

Modified: webservices/wss4j/trunk/ws-security-common/src/main/java/org/apache/wss4j/common/saml/SAMLUtil.java
URL: http://svn.apache.org/viewvc/webservices/wss4j/trunk/ws-security-common/src/main/java/org/apache/wss4j/common/saml/SAMLUtil.java?rev=1567930&r1=1567929&r2=1567930&view=diff
==============================================================================
--- webservices/wss4j/trunk/ws-security-common/src/main/java/org/apache/wss4j/common/saml/SAMLUtil.java (original)
+++ webservices/wss4j/trunk/ws-security-common/src/main/java/org/apache/wss4j/common/saml/SAMLUtil.java Thu Feb 13 14:01:20 2014
@@ -144,7 +144,7 @@ public final class SAMLUtil {
                 samlSubject = authzStmt.getSubject();
             }
             
-            if (samlSubject != null) {
+            if (samlSubject != null && samlSubject.getSubjectConfirmation() != null) {
                 Element sub = samlSubject.getSubjectConfirmation().getDOM();
                 Element keyInfoElement = 
                     XMLUtils.getDirectChildElement(sub, "KeyInfo", SIG_NS);