diff options
author | Patrick Steinhardt <ps@pks.im> | 2024-09-26 13:46:29 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2024-09-27 08:25:35 -0700 |
commit | 76c7e708bbd6b19856d1ffa58f720e8da0c9eb0f (patch) | |
tree | cb654faac69e30663cb56788ff0644b2feefdf40 /builtin/commit.c | |
parent | 49af1b772222673759756048344b142544d39849 (diff) |
diff: fix leaking orderfile option
The `orderfile` diff option is being assigned via `OPT_FILENAME()`,
which assigns an allocated string to the variable. We never free it
though, causing a memory leak.
Change the type of the string to `char *` and free it to plug the leak.
This also requires us to use `xstrdup()` to assign the global config to
it in case it is set.
This leak is being hit in t7621, but plugging it alone does not make the
test suite pass.
Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/commit.c')
0 files changed, 0 insertions, 0 deletions