On 18/10/22 06:46, Miaoqian Lin wrote:
We should use g_strfreev to free the memory allocated by g_strsplit().
Use g_free() will cause a memory leak.
Signed-off-by: Miaoqian Lin <linmq...@gmail.com>
---
qga/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/qga/main.c b/qga/main.c
index 5a9d8252e075..04902076b25d 100644
--- a/qga/main.c
+++ b/qga/main.c
@@ -934,7 +934,7 @@ static GList *split_list(const gchar *str, const gchar
*delim)
for (i = 0; strv[i]; i++) {
list = g_list_prepend(list, strv[i]);
}
- g_free(strv);
+ g_strfreev(strv);
return list;
}
Fixes: 4bca81ceed ("qga: make split_list() return allocated strings")
Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
