diff options
| author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-11-08 19:17:50 +0100 |
|---|---|---|
| committer | Taylor Blau <me@ttaylorr.com> | 2022-11-08 15:50:55 -0500 |
| commit | cb8d7677ea3a0585c03a2da02b944131606b16b8 (patch) | |
| tree | 7ca62e018a6d91bdeb989dfeea6c7bdde2be2659 /builtin/commit.c | |
| parent | 37be611d0b6a1e2f33c3a45a332d1214f0eda216 (diff) | |
| download | git-cb8d7677ea3a0585c03a2da02b944131606b16b8.tar.gz | |
revert: fix parse_options_concat() leak
Free memory from parse_options_concat(), which comes from code
originally added (then extended) in [1].
At this point we could get several more tests leak-free by free()-ing
the xstrdup() just above the line being changed, but that one's
trickier than it seems. The sequencer_remove_state() function
supposedly owns it, but sometimes we don't call it. I have a fix for
it, but it's non-trivial, so let's fix the easy one first.
1. c62f6ec341b (revert: add --ff option to allow fast forward when
cherry-picking, 2010-03-06)
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Taylor Blau <me@ttaylorr.com>
Diffstat (limited to 'builtin/commit.c')
0 files changed, 0 insertions, 0 deletions