aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPatrick Steinhardt <ps@pks.im>2024-11-05 07:16:43 +0100
committerJunio C Hamano <gitster@pobox.com>2024-11-04 22:37:51 -0800
commit5f5dd8e297b59f3f7bf61098e978a91c5581388a (patch)
treee3954691d44373f15974870ce362e65d04e83928
parent34b6ce9b30747131b6e781ff718a45328aa887d0 (diff)
downloadgit-5f5dd8e297b59f3f7bf61098e978a91c5581388a.tar.gz
builtin/ls-remote: plug leaking server options
The list of server options populated via `OPT_STRING_LIST()` is never cleared, causing a memory leak. Plug it. This leak is exposed by t5702, but plugging it alone does not make the whole test suite pass. Signed-off-by: Patrick Steinhardt <ps@pks.im> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--builtin/ls-remote.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/builtin/ls-remote.c b/builtin/ls-remote.c
index 423318f87e..42f34e1236 100644
--- a/builtin/ls-remote.c
+++ b/builtin/ls-remote.c
@@ -166,6 +166,7 @@ int cmd_ls_remote(int argc,
status = 0; /* we found something */
}
+ string_list_clear(&server_options, 0);
ref_sorting_release(sorting);
ref_array_clear(&ref_array);
if (transport_disconnect(transport))