You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by fe...@apache.org on 2010/05/18 20:27:59 UTC

svn commit: r945806 - /directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/aci/UserClass.java

Author: felixk
Date: Tue May 18 18:27:59 2010
New Revision: 945806

URL: http://svn.apache.org/viewvc?rev=945806&view=rev
Log:
Fix NPE when parameter 'names' is null

Modified:
    directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/aci/UserClass.java

Modified: directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/aci/UserClass.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/aci/UserClass.java?rev=945806&r1=945805&r2=945806&view=diff
==============================================================================
--- directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/aci/UserClass.java (original)
+++ directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/aci/UserClass.java Tue May 18 18:27:59 2010
@@ -141,7 +141,14 @@ public abstract class UserClass implemen
          */
         protected NamedUserClass( Set<DN> names )
         {
-            this.names = Collections.unmodifiableSet( new HashSet<DN>( names ) );
+            if ( names == null )
+            {
+                this.names = Collections.unmodifiableSet( new HashSet<DN>() );
+            }
+            else
+            {
+                this.names = Collections.unmodifiableSet( new HashSet<DN>( names ) );
+            }
         }