[PATCH 1/2] double free in builtin-update-index.c

From: Pierre Habouzit <madcoder@debian.org>
Date: Sep 27, 2007, 10:51 UTC
Message-ID: <20070927110456.50993930B@madism.org>
In-Reply-To: <20070927101300.GD10289@artemis.corp>

path_name is either ptr that should not be freed, or a pointer to a strbuf buffer that is deallocated when exiting the loop. Don't do that !

Signed-off-by: Pierre Habouzit <madcoder@debian.org>
---
 builtin-update-index.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/builtin-update-index.c b/builtin-update-index.c
index c76879e..e1a938d 100644
--- a/builtin-update-index.c
+++ b/builtin-update-index.c
@@ -377,8 +377,6 @@ static void read_index_info(int line_termination)
 				die("git-update-index: unable to update %s",
 				    path_name);
 		}
-		if (path_name != ptr)
-			free(path_name);
 		continue;
 
 	bad_line:

-- 
1.5.3.2.1100.g015a-dirty