diff options
| author |  Johannes Schindelin <johannes.schindelin@gmx.de> | 2021-02-04 20:07:49 +0000 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2021-02-04 17:16:42 -0800 |
| commit | 8c29b497946fde2a6ef597d960a05d3dd36dcbf0 (patch) | |
| tree | 56ced0369516976cf63d165e1f3fd2a790a15bcd /range-diff.c | |
| parent | The third batch (diff) | |
| download | tgif-8c29b497946fde2a6ef597d960a05d3dd36dcbf0.tar.xz | |
range-diff: avoid leaking memory in two error code paths
In the code paths in question, we already release a lot of memory, but
the `current_filename` variable was missed. Fix that.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'range-diff.c')
| -rw-r--r-- | range-diff.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/range-diff.c b/range-diff.c index b9950f10c8..a4d7a90dde 100644 --- a/range-diff.c +++ b/range-diff.c @@ -97,6 +97,7 @@ static int read_patches(const char *range, struct string_list *list, if (get_oid(p, &util->oid)) { error(_("could not parse commit '%s'"), p); free(util); + free(current_filename); string_list_clear(list, 1); strbuf_release(&buf); strbuf_release(&contents); @@ -112,6 +113,7 @@ static int read_patches(const char *range, struct string_list *list, error(_("could not parse first line of `log` output: " "did not start with 'commit ': '%s'"), line); + free(current_filename); string_list_clear(list, 1); strbuf_release(&buf); strbuf_release(&contents); |