You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mina.apache.org by ng...@apache.org on 2008/08/27 16:21:30 UTC

svn commit: r689490 - /mina/ftpserver/trunk/core/src/main/java/org/apache/ftpserver/usermanager/BaseUser.java

Author: ngn
Date: Wed Aug 27 07:21:30 2008
New Revision: 689490

URL: http://svn.apache.org/viewvc?rev=689490&view=rev
Log:
Added protection against NPE in the unlikely case that the user has no authorities

Modified:
    mina/ftpserver/trunk/core/src/main/java/org/apache/ftpserver/usermanager/BaseUser.java

Modified: mina/ftpserver/trunk/core/src/main/java/org/apache/ftpserver/usermanager/BaseUser.java
URL: http://svn.apache.org/viewvc/mina/ftpserver/trunk/core/src/main/java/org/apache/ftpserver/usermanager/BaseUser.java?rev=689490&r1=689489&r2=689490&view=diff
==============================================================================
--- mina/ftpserver/trunk/core/src/main/java/org/apache/ftpserver/usermanager/BaseUser.java (original)
+++ mina/ftpserver/trunk/core/src/main/java/org/apache/ftpserver/usermanager/BaseUser.java Wed Aug 27 07:21:30 2008
@@ -180,6 +180,11 @@
     public AuthorizationRequest authorize(AuthorizationRequest request) {
         Authority[] authorities = getAuthorities();
 
+        // check for no authorities at all
+        if(authorities == null) {
+            return null;
+        }
+        
         boolean someoneCouldAuthorize = false;
         for (int i = 0; i < authorities.length; i++) {
             Authority authority = authorities[i];