From 99ddff4d6d34dae8ca56bf84f591bbd0388a206e Mon Sep 17 00:00:00 2001
From: Ludovic Poitou <ludovic.poitou@forgerock.com>
Date: Thu, 09 Oct 2014 16:32:06 +0000
Subject: [PATCH] Fix for OPENDJ-1586: Changes in processSearch to iterate through nested groups members when a search is based on a filter of the isMemberOf attribute. Fixed incorrect test, preventing the method to process all groups properly. The dev 3 branch code is correct.

---
 opendj-sdk/opends/src/server/org/opends/server/extensions/IsMemberOfVirtualAttributeProvider.java |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/opendj-sdk/opends/src/server/org/opends/server/extensions/IsMemberOfVirtualAttributeProvider.java b/opendj-sdk/opends/src/server/org/opends/server/extensions/IsMemberOfVirtualAttributeProvider.java
index 3d06a39..391228e 100644
--- a/opendj-sdk/opends/src/server/org/opends/server/extensions/IsMemberOfVirtualAttributeProvider.java
+++ b/opendj-sdk/opends/src/server/org/opends/server/extensions/IsMemberOfVirtualAttributeProvider.java
@@ -311,7 +311,7 @@
       {
         returnedDNs = new HashSet<String>();
       }
-      if (returnGroupMembers(searchOperation, group.getMembers(), returnedDNs))
+      if (!returnGroupMembers(searchOperation, group.getMembers(), returnedDNs))
       {
         return;
       }
@@ -319,7 +319,7 @@
       for (DN dn : nestedGroupsDNs)
       {
         group = DirectoryServer.getGroupManager().getGroupInstance(dn);
-        if (returnGroupMembers(searchOperation, group.getMembers(), returnedDNs))
+        if (!returnGroupMembers(searchOperation, group.getMembers(), returnedDNs))
         {
           return;
         }

--
Gitblit v1.10.0