From: Pierangelo Masarati <ando@openldap.org>
Date: Sun, 25 Jul 2004 22:45:53 +0000 (+0000)
Subject: fix leak when the rewriteContext is disabled (ITS#3256)
X-Git-Tag: OPENDLAP_REL_ENG_2_2_MP~24
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=9c0ed9b6bd2afccb2118942bd89680a2049742a1;p=openldap

fix leak when the rewriteContext is disabled (ITS#3256)
---

diff --git a/libraries/librewrite/context.c b/libraries/librewrite/context.c
index 03e1e74538..aad9f0a0df 100644
--- a/libraries/librewrite/context.c
+++ b/libraries/librewrite/context.c
@@ -218,10 +218,10 @@ rewrite_context_apply(
 	assert( op->lo_depth > 0 );
 
 	Debug( LDAP_DEBUG_TRACE, "==> rewrite_context_apply"
-			" [depth=%d] string='%s'\n%s",
-			op->lo_depth, string, "" );
+			" [depth=%d] string='%s'\n",
+			op->lo_depth, string, 0 );
 	
-	s = strdup( string );
+	s = (char *)string;
 	
 	for ( rule = context->lc_rule->lr_next;
 			rule != NULL && op->lo_num_passes < info->li_max_passes;
@@ -296,7 +296,7 @@ rewrite_context_apply(
 
 				if ( do_continue ) {
 					if ( rule->lr_next == NULL ) {
-						res = s;
+						res = ( s == string ? strdup( s ) : s );
 					}
 					goto rc_continue;
 				}
@@ -321,7 +321,9 @@ rewrite_context_apply(
 			if ( res != NULL ) {
 				struct rewrite_action *action;
 				
-				free( s );
+				if (s != string ) {
+					free( s );
+				}
 				s = res;
 
 				for ( action = rule->lr_action;
@@ -379,7 +381,7 @@ rewrite_context_apply(
 			 * result back to the string
 			 */
 			} else if ( rule->lr_next == NULL ) {
-				res = s;
+				res = ( s == string ? strdup( s ) : s );
 			}
 			
 			break;