[389-commits] Branch 'Directory_Server_8_2_Branch' - ldap/servers

Richard Allen Megginson rmeggins at fedoraproject.org
Fri Mar 12 15:41:16 UTC 2010


 ldap/servers/plugins/acl/acleffectiverights.c |    2 --
 1 file changed, 2 deletions(-)

New commits:
commit 87d2477da35f4a029a225dd37917d4405d94ba54
Author: Rich Megginson <rmeggins at redhat.com>
Date:   Thu Mar 11 20:54:42 2010 -0700

    Bug 572677 - Memory leak in searches including GER control
    
    https://bugzilla.redhat.com/show_bug.cgi?id=572677
    Resolves: bug 572677
    Bug Description: Memory leak in searches including GER control
    Reviewed by: Andrey Ivanov (Thanks!)
    Branch: Directory_Server_8_2_Branch
    Fix Description: The per-operation acl pblocks are cached.  In order to
    release the pblock back to the cache free list, the connection must be
    provided.  The connection comes from the pblock.
    Platforms tested: RHEL5 x86_64
    Flag Day: no
    Doc impact: no
    (cherry picked from commit ed463407ead1f63ba26f64740a1e5cd1d79a03ee)

diff --git a/ldap/servers/plugins/acl/acleffectiverights.c b/ldap/servers/plugins/acl/acleffectiverights.c
index 70c7c85..9afac07 100644
--- a/ldap/servers/plugins/acl/acleffectiverights.c
+++ b/ldap/servers/plugins/acl/acleffectiverights.c
@@ -280,8 +280,6 @@ _ger_release_gerpb (
 {
 	if ( *gerpb )
 	{
-		/* Return conn to pb */
-		slapi_pblock_set ( *gerpb, SLAPI_CONNECTION, NULL );
 		slapi_pblock_destroy ( *gerpb );
 		*gerpb = NULL;
 	}




More information about the 389-commits mailing list