From 78aeeca8095f9f1f0829cf09930708d6b4be636c Mon Sep 17 00:00:00 2001
From: Mathieu Baudier <mbaudier@argeo.org>
Date: Thu, 10 Mar 2011 16:09:09 +0000
Subject: [PATCH] Wrap roles so that it is writable

git-svn-id: https://svn.argeo.org/commons/trunk@4279 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc
---
 .../java/org/argeo/security/core/DefaultSecurityService.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
index 68f97d4a0..62ce6c759 100644
--- a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
+++ b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
@@ -16,6 +16,7 @@
 
 package org.argeo.security.core;
 
+import java.util.HashSet;
 import java.util.Iterator;
 import java.util.Set;
 
@@ -134,7 +135,8 @@ public class DefaultSecurityService extends DefaultCurrentUserService implements
 	}
 
 	public Set<ArgeoUser> listUsersInRole(String role) {
-		Set<ArgeoUser> lst = securityDao.listUsersInRole(role);
+		Set<ArgeoUser> lst = new HashSet<ArgeoUser>(
+				securityDao.listUsersInRole(role));
 		Iterator<ArgeoUser> it = lst.iterator();
 		while (it.hasNext()) {
 			if (it.next().getUsername()
-- 
2.30.2