From 40ba9bc80bb28e0f99cc1d24d668e86971401484 Mon Sep 17 00:00:00 2001 From: Kurt Zeilenga Date: Tue, 2 Feb 1999 22:46:21 +0000 Subject: [PATCH] Don't try to free NULL idl. Did not cause a problem, though, as idl_free() had insurance. --- servers/slapd/back-ldbm/id2children.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/servers/slapd/back-ldbm/id2children.c b/servers/slapd/back-ldbm/id2children.c index 11af4cde18..978fd9a4b7 100644 --- a/servers/slapd/back-ldbm/id2children.c +++ b/servers/slapd/back-ldbm/id2children.c @@ -104,7 +104,7 @@ has_children( { struct dbcache *db; Datum key; - int rc; + int rc = 1; ID_BLOCK *idl; char buf[20]; @@ -127,9 +127,13 @@ has_children( idl = idl_fetch( be, db, key ); ldbm_cache_close( be, db ); - rc = idl ? 1 : 0; - idl_free( idl ); - Debug( LDAP_DEBUG_TRACE, "<= has_children %d\n", rc, 0, 0 ); + if( idl != NULL ) { + idl_free( idl ); + rc = 0; + } + + Debug( LDAP_DEBUG_TRACE, "<= has_children( %lu ): %s\n", + p->e_id, rc ? "yes" : "no", 0 ); return( rc ); } -- 2.39.5