commit eac70c4f64a618744e05d4a5be61a356c0011033
author Johannes Schindelin <Johannes.Schindelin@gmx.de> Tue Feb 20 03:45:49 2007 +0100
committer Junio C Hamano <junkio@cox.net> Mon Feb 19 19:02:12 2007 -0800
tree a7c4e9bf2ed14fda8804a6e22560a0f00bc6aaea
parent 56185f49d03cae28048146e902089ea366c6cd6c

apply: fix memory leak in prefix_one()

Signed-off-by: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Signed-off-by: Junio C Hamano <junkio@cox.net>

builtin-apply.c

diff --git a/builtin-apply.c b/builtin-apply.c
index 053511e..2a40af3 100644
--- a/builtin-apply.c
+++ b/builtin-apply.c
@@ -2502,11 +2502,13 @@
 	return 1;
 }
 
-static char *prefix_one(char *name)
+static void prefix_one(char **name)
 {
-	if (!name)
-		return name;
-	return xstrdup(prefix_filename(prefix, prefix_length, name));
+	char *old_name = *name;
+	if (!old_name)
+		return;
+	*name = xstrdup(prefix_filename(prefix, prefix_length, *name));
+	free(old_name);
 }
 
 static void prefix_patches(struct patch *p)
@@ -2514,8 +2516,9 @@
 	if (!prefix)
 		return;
 	for ( ; p; p = p->next) {
-		p->new_name = prefix_one(p->new_name);
-		p->old_name = prefix_one(p->old_name);
+		if (p->new_name != p->old_name)
+			prefix_one(&p->new_name);
+		prefix_one(&p->old_name);
 	}
 }