]> Kevux Git Server - rit/commit
git, help: fix memory leaks in alias listing
authorJonatan Holmgren <jonatan@jontes.page>
Thu, 26 Feb 2026 20:53:28 +0000 (21:53 +0100)
committerJunio C Hamano <gitster@pobox.com>
Thu, 26 Feb 2026 21:07:24 +0000 (13:07 -0800)
commitcdef6255095251500cc7d08d304072e1e2fa0bbd
tree084449eadf5366aba494134eb7d17949c676dda5
parent65892943750b0c66017389f0233ba5a6a7205165
git, help: fix memory leaks in alias listing

The list_aliases() function sets the util pointer of each list item to
a heap-allocated copy of the alias command value.  Two callers failed
to free these util pointers:

 - list_cmds() in git.c collects a string list with STRING_LIST_INIT_DUP
   and clears it with string_list_clear(&list, 0), which frees the
   duplicated strings (strdup_strings=1) but not the util pointers.
   Pass free_util=1 to free them.

 - list_cmds_by_config() in help.c calls string_list_sort_u(list, 0) to
   deduplicate the list before processing completion.commands overrides.
   When duplicate entries are removed, the util pointer of each discarded
   item is leaked because free_util=0.  Pass free_util=1 to free them.

Reported-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: Jonatan Holmgren <jonatan@jontes.page>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git.c
help.c