@matheusbsbs / matheus-bagliano / commits / c45dc9cf30

diff: plug memory leak from regcomp() on {log,diff} -I

Fix a memory leak in 296d4a94e7 (diff: add -I<regex> that ignores matching changes, 2020-10-20) by freeing the memory it allocates in the newly introduced diff_free(). See the previous commit for details on that. This memory leak was intentionally introduced in 296d4a94e7, see the discussion on a previous iteration of it in https://lore.kernel.org/git/xmqqeelycajx.fsf@gitster.c.googlers.com/ At that time freeing the memory was somewhat tedious, but since it isn't anymore with the newly introduced diff_free() let's use it. Let's retain the pattern for diff_free_file() and add a diff_free_ignore_regex(), even though (unlike "diff_free_file") we don't need to call it elsewhere. I think this'll make for more readable code than gradually accumulating a giant diff_free() function, sharing "int i" across unrelated code etc. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Ævar Arnfjörð Bjarmason committed Feb 11, 2021 at 11:45 UTC c45dc9cf30a6f7f40adb3ea70dd2f2905ecd4afa

1 file changed +12

diff.c

+12

@@ -6342,12 +6342,24 @@ static void diff_free_file(struct diff_options *options)

6342 fclose(options->file);

6343 }

6344

6345 +static void diff_free_ignore_regex(struct diff_options *options)

6346 +{

6347 + int i;

6348 +

6349 + for (i = 0; i < options->ignore_regex_nr; i++) {

6350 + regfree(options->ignore_regex[i]);

6351 + free(options->ignore_regex[i]);

6352 + }

6353 + free(options->ignore_regex);

6354 +}

6355 +

6356 void diff_free(struct diff_options *options)

6357 {

6358 if (options->no_free)

6359 return;

6360

6361 diff_free_file(options);

6362 + diff_free_ignore_regex(options);

6363 }

6364

6365 void diff_flush(struct diff_options *options)