refs: fix memory leak when parsing hideRefs config
commit5eeb9aa2086edc95f4f2c9cc844f60535f0a5ca4
authorPatrick Steinhardt <ps@pks.im>
Thu, 17 Nov 2022 05:46:39 +0000 (17 06:46 +0100)
committerTaylor Blau <me@ttaylorr.com>
Thu, 17 Nov 2022 21:22:51 +0000 (17 16:22 -0500)
treea522eb1e1ec377522965848b69db5cd008fef21e
parent5af5e54106e20f65c913550c80aec3186b859e9b
refs: fix memory leak when parsing hideRefs config

When parsing the hideRefs configuration, we first duplicate the config
value so that we can modify it. We then subsequently append it to the
`hide_refs` string list, which is initialized with `strdup_strings`
enabled. As a consequence we again reallocate the string, but never
free the first duplicate and thus have a memory leak.

While we never clean up the static `hide_refs` variable anyway, this is
no excuse to make the leak worse by leaking every value twice. We are
also about to change the way this variable will be handled so that we do
indeed start to clean it up. So let's fix the memory leak by using the
`string_list_append_nodup()` so that we pass ownership of the allocated
string to `hide_refs`.

Signed-off-by: Patrick Steinhardt <ps@pks.im>
Signed-off-by: Taylor Blau <me@ttaylorr.com>
refs.c