Message ID | patch-v2-1.6-66c838fd800-20211022T085306Z-avarab@gmail.com (mailing list archive) |
---|---|
State | Accepted |
Commit | a2fb7672c03eef86b8dbf33699c12307d504a299 |
Headers | show |
Series | leaks: miscellaneous small leak fixes | expand |
diff --git a/builtin/grep.c b/builtin/grep.c index 8af5249a7bb..fd184c182a3 100644 --- a/builtin/grep.c +++ b/builtin/grep.c @@ -199,8 +199,8 @@ static void *run(void *arg) grep_source_clear_data(&w->source); work_done(w); } - free_grep_patterns(arg); - free(arg); + free_grep_patterns(opt); + free(opt); return (void*) (intptr_t) hit; }
Stylistically fix up code added in bfac23d9534 (grep: Fix two memory leaks, 2010-01-30). We usually don't use the "arg" at all once we've casted it to the struct we want, let's not do that here when we're freeing it. Perhaps it was thought that a cast to "void *" would otherwise be needed? Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> --- builtin/grep.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)