[PATCH v6 16/19] grep API: plug memory leaks by freeing "header_list"
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Feb 2, 2023, 09:52 UTC
- Message-ID
- <patch-v6-16.19-0c51ea7fd2d-20230202T094704Z-avarab@gmail.com>
- In-Reply-To
- <cover-v6-00.19-00000000000-20230202T094704Z-avarab@gmail.com>
When the "header_list" struct member was added in [1], freeing this field was neglected. Fix that now, so that commands like
./git -P log -1 --color=always --author=A origin/master
will run leak-free.
1. 80235ba79ef ("log --author=me --grep=it" should find intersection,
not union, 2010-01-17)Helped-by: Elijah Newren <newren@gmail.com> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> --- grep.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/grep.c b/grep.c index f8708e1fd20..92ece4b7fa3 100644 --- a/grep.c +++ b/grep.c @@ -795,6 +795,7 @@ static void free_grep_pat(struct grep_pat *pattern) void free_grep_patterns(struct grep_opt *opt) { free_grep_pat(opt->pattern_list); + free_grep_pat(opt->header_list); if (opt->pattern_expression) free_pattern_expr(opt->pattern_expression);
-- 2.39.1.1392.g63e6d408230