From: Pierangelo Masarati <ando@openldap.org>
Date: Thu, 9 Aug 2007 16:33:50 +0000 (+0000)
Subject: fixed few leaks in previous commit (ITS#5081)
X-Git-Tag: OPENLDAP_REL_ENG_2_4_MP~249
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=ae9776a580afceb3bf374a07529765688a806612;p=openldap

fixed few leaks in previous commit (ITS#5081)
---

diff --git a/servers/slapd/overlays/pcache.c b/servers/slapd/overlays/pcache.c
index df434e7679..215bd00fcf 100644
--- a/servers/slapd/overlays/pcache.c
+++ b/servers/slapd/overlays/pcache.c
@@ -254,6 +254,8 @@ query2url( CachedQuery *q, struct berval *urlbv )
 	ptr = lutil_strcopy( ptr, ",x-expiry=" );
 	ptr = lutil_strcopy( ptr, expiry_buf );
 
+	ber_memfree( bv_filter.bv_val );
+
 	return 0;
 }
 
@@ -429,6 +431,7 @@ url2query(
 error:;
 	if ( query.filter != NULL ) filter_free( query.filter );
 	if ( !BER_BVISNULL( &tempstr ) ) ch_free( tempstr.bv_val );
+	if ( !BER_BVISNULL( &query.base ) ) ch_free( query.base.bv_val );
 	if ( lud != NULL ) ldap_free_urldesc( lud );
 
 	return rc;
@@ -1125,7 +1128,7 @@ add_query(
 	qbase = avl_find( templ->qbase, &qb, pcache_dn_cmp );
 	if ( !qbase ) {
 		qbase = ch_calloc( 1, sizeof(Qbase) + qb.base.bv_len + 1 );
-		qbase->base.bv_len =qb.base.bv_len;
+		qbase->base.bv_len = qb.base.bv_len;
 		qbase->base.bv_val = (char *)(qbase+1);
 		memcpy( qbase->base.bv_val, qb.base.bv_val, qb.base.bv_len );
 		qbase->base.bv_val[qbase->base.bv_len] = '\0';
@@ -2796,6 +2799,8 @@ pcache_db_close(
 			op->orm_modlist = &mod;
 
 			op->o_bd->be_modify( op, &rs );
+
+			ber_bvarray_free( vals );
 		}
 	}