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 ) );
+ }
}