From 46ec510ac088ac2669e617a4c8c35e6218dabecc Mon Sep 17 00:00:00 2001 From: Clemens Buchacher Date: Tue, 28 May 2013 00:49:57 +0200 Subject: fix segfault with git log -c --follow In diff_tree_combined we make a copy of diffopts. In try_to_follow_renames, called via diff_tree_sha1, we free and re-initialize diffopts->pathspec->items. Since we did not make a deep copy of diffopts in diff_tree_combined, the original diffopts does not get the update. By the time we return from diff_tree_combined, rev->diffopt->pathspec->items points to an invalid memory address. We get a segfault next time we try to access that pathspec. Instead, along with the copy of diffopts, make a copy pathspec->items as well. We would also have to make a copy of pathspec->raw to keep it consistent with pathspec->items, but nobody seems to rely on that. Signed-off-by: Clemens Buchacher Signed-off-by: Junio C Hamano --- combine-diff.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'combine-diff.c') diff --git a/combine-diff.c b/combine-diff.c index 7f6187f9cd..21fd00ff61 100644 --- a/combine-diff.c +++ b/combine-diff.c @@ -1120,6 +1120,7 @@ void diff_tree_combined(const unsigned char *sha1, int i, num_paths, needsep, show_log_first, num_parent = parents->nr; diffopts = *opt; + diff_tree_setup_paths(diffopts.pathspec.raw, &diffopts); diffopts.output_format = DIFF_FORMAT_NO_OUTPUT; DIFF_OPT_SET(&diffopts, RECURSIVE); DIFF_OPT_CLR(&diffopts, ALLOW_EXTERNAL); @@ -1187,6 +1188,8 @@ void diff_tree_combined(const unsigned char *sha1, paths = paths->next; free(tmp); } + + diff_tree_release_paths(&diffopts); } void diff_tree_combined_merge(const struct commit *commit, int dense, -- cgit v1.2.3