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/10/05 00:41:06 UTC

svn commit: r1629455 - /webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/validate/SignatureTrustValidator.java

Author: coheigea
Date: Sat Oct  4 22:41:06 2014
New Revision: 1629455

URL: http://svn.apache.org/r1629455
Log:
Fixing potential NPE

Modified:
    webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/validate/SignatureTrustValidator.java

Modified: webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/validate/SignatureTrustValidator.java
URL: http://svn.apache.org/viewvc/webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/validate/SignatureTrustValidator.java?rev=1629455&r1=1629454&r2=1629455&view=diff
==============================================================================
--- webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/validate/SignatureTrustValidator.java (original)
+++ webservices/wss4j/branches/1_6_x-fixes/src/main/java/org/apache/ws/security/validate/SignatureTrustValidator.java Sat Oct  4 22:41:06 2014
@@ -303,7 +303,7 @@ public class SignatureTrustValidator imp
         final java.security.cert.X509Certificate cert,
         final Collection<Pattern> subjectDNPatterns
     ) {
-        if (subjectDNPatterns.isEmpty()) {
+        if (subjectDNPatterns == null || subjectDNPatterns.isEmpty()) {
             LOG.warn("No Subject DN Certificate Constraints were defined. This could be a security issue");
         }
         if (!subjectDNPatterns.isEmpty()) {