[jboss-cvs] Picketbox SVN: r67 - trunk/security-jboss-sx/identity/src/main/java/org/jboss/security/identity/plugins.

[jboss-cvs-commits at lists.jboss.org]

Author: sguilhen at redhat.com
Date: 2010-03-29 15:48:07 -0400 (Mon, 29 Mar 2010)
New Revision: 67

Modified:
   trunk/security-jboss-sx/identity/src/main/java/org/jboss/security/identity/plugins/SimpleRoleGroup.java
Log:
SECURITY-477: Added synchronized block to toString where there is an iteration over the roles list.

Modified: trunk/security-jboss-sx/identity/src/main/java/org/jboss/security/identity/plugins/SimpleRoleGroup.java
===================================================================
--- trunk/security-jboss-sx/identity/src/main/java/org/jboss/security/identity/plugins/SimpleRoleGroup.java	2010-03-27 03:15:15 UTC (rev 66)
+++ trunk/security-jboss-sx/identity/src/main/java/org/jboss/security/identity/plugins/SimpleRoleGroup.java	2010-03-29 19:48:07 UTC (rev 67)
@@ -229,9 +229,12 @@
       StringBuilder builder = new StringBuilder();
       builder.append(this.getRoleName());
       builder.append("(");
-      for (Role role : roles)
+      synchronized (this)
       {
-         builder.append(role.toString()).append(",");
+         for (Role role : roles)
+         {
+            builder.append(role.toString()).append(",");
+         }
       }
       builder.append(")");
       return builder.toString();