Browse Source

Ticket #256 - debug build assertion in ACL_EvalDestroy()

https://fedorahosted.org/389/ticket/256
Resolves: Ticket #256
Bug Description: debug build assertion in ACL_EvalDestroy()
Reviewed by: nkinder (Thanks!)
Branch: master
Fix Description: aclpb->aclpb_acleval does not own its acllist - so use
ACL_EvalDestroyNoDecrement() instead so the list is not freed.
Platforms tested: RHEL6 x86_64, Fedora 16
Flag Day: no
Doc impact: no
Rich Megginson 14 years ago
parent
commit
e1a7823230
1 changed files with 3 additions and 2 deletions
  1. 3 2
      ldap/servers/plugins/acl/acl_ext.c

+ 3 - 2
ldap/servers/plugins/acl/acl_ext.c

@@ -749,8 +749,9 @@ acl__free_aclpb ( Acl_PBlock **aclpb_ptr)
 
 
     aclpb = *aclpb_ptr;
     aclpb = *aclpb_ptr;
 
 
-    if (aclpb->aclpb_acleval)
-        ACL_EvalDestroy(NULL, NULL, aclpb->aclpb_acleval);
+    if (aclpb->aclpb_acleval) {
+        ACL_EvalDestroyNoDecrement(NULL, NULL, aclpb->aclpb_acleval);
+    }
 
 
     if (aclpb->aclpb_proplist)
     if (aclpb->aclpb_proplist)
         PListDestroy(aclpb->aclpb_proplist);
         PListDestroy(aclpb->aclpb_proplist);