[389-commits] ldap/servers

[Noriko Hosoi]

 ldap/servers/slapd/back-ldbm/id2entry.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 107d40e05c49211b88f5ec47bc09443640a4e570
Author: Noriko Hosoi <nhosoi at redhat.com>
Date:   Mon Feb 15 16:39:13 2010 -0800

    565664 -  Incorrect parameter for CACHE_RETURN()
    
    https://bugzilla.redhat.com/show_bug.cgi?id=565664
    
    Description: Special thanks to edewata at redhat.com for fixing the bug.
    > See ldap/servers/slapd/back-ldbm/id2entry.c line 196.
    > The second parameter should be a pointer to pointer,
    > but the current code passes a regular pointer:
    >      CACHE_RETURN(&inst->inst_dncache, bdn);

diff --git a/ldap/servers/slapd/back-ldbm/id2entry.c b/ldap/servers/slapd/back-ldbm/id2entry.c
index 83bf94d..05a0181 100644
--- a/ldap/servers/slapd/back-ldbm/id2entry.c
+++ b/ldap/servers/slapd/back-ldbm/id2entry.c
@@ -193,7 +193,7 @@ id2entry_delete( backend *be, struct backentry *e, back_txn *txn )
         Slapi_DN *sdn = slapi_sdn_dup(slapi_entry_get_sdn_const(e->ep_entry));
         struct backdn *bdn = backdn_init(sdn, e->ep_id, 1);
         CACHE_REMOVE(&inst->inst_dncache, bdn);
-		CACHE_RETURN(&inst->inst_dncache, bdn);
+		CACHE_RETURN(&inst->inst_dncache, &bdn);
     }
 
     rc = db->del( db,db_txn,&key,0 );