diff options
author | Junio C Hamano <junkio@cox.net> | 2005-09-15 16:13:43 -0700 |
---|---|---|
committer | Junio C Hamano <junkio@cox.net> | 2005-09-15 16:13:43 -0700 |
commit | 5098bafb756de69d03882707a3382899c0cb7dd1 (patch) | |
tree | 694e88e3a9a2187150f7af09bb9b06a861d7b53b /diff-tree.c | |
parent | 98533b90cb8e88484cb381334b19cbf4f7cf92b1 (diff) | |
download | git-5098bafb756de69d03882707a3382899c0cb7dd1.tar.gz git-5098bafb756de69d03882707a3382899c0cb7dd1.tar.xz |
Plug diff leaks.
It is a bit embarrassing that it took this long for a fix since the
problem was first reported on Aug 13th.
Message-ID: <87y876gl1r.wl@mail2.atmark-techno.com>
From: Yasushi SHOJI <yashi@atmark-techno.com>
Newsgroups: gmane.comp.version-control.git
Subject: [patch] possible memory leak in diff.c::diff_free_filepair()
Date: Sat, 13 Aug 2005 19:58:56 +0900
This time I used valgrind to make sure that it does not overeagerly
discard memory that is still being used.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'diff-tree.c')
-rw-r--r-- | diff-tree.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/diff-tree.c b/diff-tree.c index fc87902a3..e8f5d1b12 100644 --- a/diff-tree.c +++ b/diff-tree.c @@ -370,6 +370,7 @@ static int diff_tree_commit(const unsigned char *commit, const char *name) } offset += 48; } + free(buf); return 0; } |