diff options
| author | Marco Trevisan (Treviño) <mail@3v1n0.net> | 2022-10-23 17:10:59 +0200 |
|---|---|---|
| committer | Marco Trevisan (Treviño) <mail@3v1n0.net> | 2022-10-25 14:33:25 +0200 |
| commit | dafd6602dc653ace1f4b215f0c145445b3640301 (patch) | |
| tree | 35abc01f780446d40786f291f4d5ffa2ab1a2ed4 | |
| parent | 37e61d66c3561a20ce9c0302840d0842eb450d5d (diff) | |
| download | glib-dafd6602dc653ace1f4b215f0c145445b3640301.tar.gz | |
gio/gdesktopappinfo: Free the wrapped argv array on launch failure
We create an array that we never free, ensure this is the case.
The previous commit gives CI a chance to check this with valgrind job.
Found as part of another review:
- https://gitlab.gnome.org/GNOME/glib/-/merge_requests/2839#note_1524922
| -rw-r--r-- | gio/gdesktopappinfo.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/gio/gdesktopappinfo.c b/gio/gdesktopappinfo.c index f8cccca02..c29c30947 100644 --- a/gio/gdesktopappinfo.c +++ b/gio/gdesktopappinfo.c @@ -2963,6 +2963,7 @@ g_desktop_app_info_launch_uris_with_spawn (GDesktopAppInfo *info, g_free (sn_id); g_list_free (launched_uris); + g_clear_pointer (&wrapped_argv, g_strfreev); goto out; } |