svn commit: r439205 - /incubator/servicemix/trunk/servicemix-core/src/main/java/org/apache/servicemix/jbi/security/acl/impl/AuthorizationEntry.java

[gnodet]

Author: gnodet
Date: Thu Aug 31 23:31:32 2006
New Revision: 439205

URL: http://svn.apache.org/viewvc?rev=439205&view=rev
Log:
Fix a possible NPE when toString is called

Modified:
    
incubator/servicemix/trunk/servicemix-core/src/main/java/org/apache/servicemix/jbi/security/acl/impl/AuthorizationEntry.java

Modified: 
incubator/servicemix/trunk/servicemix-core/src/main/java/org/apache/servicemix/jbi/security/acl/impl/AuthorizationEntry.java
URL: 
http://svn.apache.org/viewvc/incubator/servicemix/trunk/servicemix-core/src/main/java/org/apache/servicemix/jbi/security/acl/impl/AuthorizationEntry.java?rev=439205&r1=439204&r2=439205&view=diff
==============================================================================
--- 
incubator/servicemix/trunk/servicemix-core/src/main/java/org/apache/servicemix/jbi/security/acl/impl/AuthorizationEntry.java
 (original)
+++ 
incubator/servicemix/trunk/servicemix-core/src/main/java/org/apache/servicemix/jbi/security/acl/impl/AuthorizationEntry.java
 Thu Aug 31 23:31:32 2006
@@ -140,11 +140,13 @@
     
     public String getRoles() {
         StringBuffer sb = new StringBuffer();
-        for (Iterator iter = this.acls.iterator(); iter.hasNext();) {
-            GroupPrincipal p = (GroupPrincipal) iter.next();
-            sb.append(p);
-            if (iter.hasNext()) {
-                sb.append(",");
+        if (this.acls != null) {
+            for (Iterator iter = this.acls.iterator(); iter.hasNext();) {
+                GroupPrincipal p = (GroupPrincipal) iter.next();
+                sb.append(p);
+                if (iter.hasNext()) {
+                    sb.append(",");
+                }
             }
         }
         return sb.toString();