From 5e1b6983291d08e7ef2e803ea105704038d22dab Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Wed, 30 Aug 2023 09:19:42 -0400 Subject: [PATCH] keyfile-utils: Quiet a clang-analyzer warning It complains that we could leak memory if the return value pointer isn't set. That's actually a nonsensical case, there's no reason to call this and ignore the return value. So change things to require it be set, and also change the `g_return_val_if_fail` to be hard assertions per our new policy. --- src/libotutil/ot-keyfile-utils.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/libotutil/ot-keyfile-utils.c b/src/libotutil/ot-keyfile-utils.c index a1250dc6..d6cd102c 100644 --- a/src/libotutil/ot-keyfile-utils.c +++ b/src/libotutil/ot-keyfile-utils.c @@ -174,10 +174,11 @@ ot_keyfile_get_string_list_with_separator_choice (GKeyFile *keyfile, const char const char *key, const char *separators, char ***out_value, GError **error) { - g_return_val_if_fail (keyfile != NULL, FALSE); - g_return_val_if_fail (section != NULL, FALSE); - g_return_val_if_fail (key != NULL, FALSE); - g_return_val_if_fail (separators != NULL, FALSE); + g_assert (keyfile != NULL); + g_assert (section != NULL); + g_assert (key != NULL); + g_assert (separators != NULL); + g_assert (out_value != NULL); g_autofree char *value_str = NULL; if (!ot_keyfile_get_value_with_default (keyfile, section, key, NULL, &value_str, error)) @@ -215,7 +216,7 @@ ot_keyfile_get_string_list_with_separator_choice (GKeyFile *keyfile, const char } } - ot_transfer_out_value (out_value, &value_list); + *out_value = g_steal_pointer (&value_list); return TRUE; } -- 2.30.2