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 17:25:35 +0200 |
| commit | 76c7e708bbd6b19856d1ffa58f720e8da0c9eb0f (patch) | |
| tree | cb654faac69e30663cb56788ff0644b2feefdf40 /parse-options.c | |
| parent | builtin/pull: fix leaking "ff" option (diff) | |
| download | git-76c7e708bbd6b19856d1ffa58f720e8da0c9eb0f.tar.xz git-76c7e708bbd6b19856d1ffa58f720e8da0c9eb0f.zip | |
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 'parse-options.c')
0 files changed, 0 insertions, 0 deletions