svn commit: r1430639 - in /archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security: ArchivaAuthorizer.java ArchivaConfigurableUsersManager.java

[olamy]

Author: olamy
Date: Wed Jan  9 00:05:50 2013
New Revision: 1430639

URL: http://svn.apache.org/viewvc?rev=1430639&view=rev
Log:
NPE free and fix find user with only ldap

Modified:
    
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaAuthorizer.java
    
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaConfigurableUsersManager.java

Modified: 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaAuthorizer.java
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaAuthorizer.java?rev=1430639&r1=1430638&r2=1430639&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaAuthorizer.java
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaAuthorizer.java
 Wed Jan  9 00:05:50 2013
@@ -84,7 +84,7 @@ public class ArchivaAuthorizer
                     authorizationException = e;
                 }
 
-                if ( result.isAuthorized() )
+                if ( result != null && result.isAuthorized() )
                 {
                     return result;
                 }

Modified: 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaConfigurableUsersManager.java
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaConfigurableUsersManager.java?rev=1430639&r1=1430638&r2=1430639&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaConfigurableUsersManager.java
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/security/ArchivaConfigurableUsersManager.java
 Wed Jan  9 00:05:50 2013
@@ -449,7 +449,7 @@ public class ArchivaConfigurableUsersMan
         throws UserManagerException
     {
         UserManager userManager = findFirstWritable();
-        if ( userManager.isReadOnly() )
+        if ( userManager == null )
         {
             log.warn( "cannot find writable user manager implementation, skip 
guest user creation" );
             return null;