Yair Zaslavsky has uploaded a new change for review.

Change subject: aaa: more sync fixes
......................................................................

aaa: more sync fixes

Change-Id: I3dddcf6b1327f5540e333f7e3d66ef644c40822a
Topic: AAA
Signed-off-by: Yair Zaslavsky <[email protected]>
---
M 
backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/89/30589/1

diff --git 
a/backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java
 
b/backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java
index 2ca4eff..03c24b0 100644
--- 
a/backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java
+++ 
b/backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/AuthzUtils.java
@@ -95,8 +95,9 @@
     private static void constructGroupsMembershipTree(ExtMap entity, ExtKey 
key, Map<String, ExtMap> groupsCache) {
         List<ExtMap> groups = new ArrayList<>();
         for (ExtMap memberOf : entity.get(key, Collections.<ExtMap> 
emptyList())) {
-            constructGroupsMembershipTree(memberOf, GroupRecord.GROUPS, 
groupsCache);
-            groups.add(groupsCache.get(memberOf.get(GroupRecord.ID)));
+            ExtMap cachedGroup = groupsCache.get(memberOf.get(GroupRecord.ID));
+            constructGroupsMembershipTree(cachedGroup, GroupRecord.GROUPS, 
groupsCache);
+            groups.add(cachedGroup);
         }
         entity.put(key, groups);
     }


-- 
To view, visit http://gerrit.ovirt.org/30589
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3dddcf6b1327f5540e333f7e3d66ef644c40822a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to