Author: ate
Date: Wed Mar  3 00:14:27 2010
New Revision: 918280

URL: http://svn.apache.org/viewvc?rev=918280&view=rev
Log:
Fix for JS2-1039: OutOfMemoryError when trying to add a role to a user, if the 
user has already that role
See: http://issues.apache.org/jira/browse/JS2-1039
Patch provided by Aysegul Aydin Isiktekin: good catch, thanks!

Modified:
    
portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/model/impl/AttributeImpl.java

Modified: 
portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/model/impl/AttributeImpl.java
URL: 
http://svn.apache.org/viewvc/portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/model/impl/AttributeImpl.java?rev=918280&r1=918279&r2=918280&view=diff
==============================================================================
--- 
portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/model/impl/AttributeImpl.java
 (original)
+++ 
portals/jetspeed-2/portal/trunk/components/jetspeed-security/src/main/java/org/apache/jetspeed/security/mapping/model/impl/AttributeImpl.java
 Wed Mar  3 00:14:27 2010
@@ -137,7 +137,7 @@
             if (values != null){
                 Iterator<String> valIter = values.iterator();
                 while (valIter.hasNext()){
-                    sb.append(values.getClass());
+                    sb.append(valIter.next());
                     if (valIter.hasNext()){
                         sb.append(",");
                     }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to