revisions API: have release_revisions() release "grep_filter"

avar · gitster · commit f41fb662f57a · 2022-04-13T23:56:09.000-07:00
Extend the the release_revisions() function so that it frees the
"grep_filter" in the "struct rev_info".This allows us to mark a test
as passing under "TEST_PASSES_SANITIZE_LEAK=true".

Signed-off-by: Ævar Arnfjörð Bjarmason &lt;avarab@gmail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/revision.c b/revision.c
@@ -2950,6 +2950,7 @@ void release_revisions(struct rev_info *revs)
 	release_revisions_cmdline(&revs->cmdline);
 	list_objects_filter_release(&revs->filter);
 	release_revisions_mailmap(revs->mailmap);
+	free_grep_patterns(&revs->grep_filter);
 }
 
 static void add_child(struct rev_info *revs, struct commit *parent, struct commit *child)
diff --git a/t/t9151-svn-mergeinfo.sh b/t/t9151-svn-mergeinfo.sh
@@ -5,7 +5,6 @@
 
 test_description='git-svn svn mergeinfo properties'
 
-TEST_FAILS_SANITIZE_LEAK=true
 . ./lib-git-svn.sh
 
 test_expect_success 'load svn dump' "

Original file line number	Diff line number	Diff line change
`@@ -2950,6 +2950,7 @@ void release_revisions(struct rev_info *revs)`
`2950`	`2950`	`release_revisions_cmdline(&revs->cmdline);`
`2951`	`2951`	`list_objects_filter_release(&revs->filter);`
`2952`	`2952`	`release_revisions_mailmap(revs->mailmap);`
	`2953`	`+ free_grep_patterns(&revs->grep_filter);`
`2953`	`2954`	`}`
`2954`	`2955`
`2955`	`2956`	`static void add_child(struct rev_info revs, struct commit parent, struct commit *child)`