diff options
author | Marco Trevisan (TreviƱo) <mail@3v1n0.net> | 2019-07-09 11:17:18 +0200 |
---|---|---|
committer | Robert Mader <robert.mader@posteo.de> | 2020-04-12 21:40:20 +0000 |
commit | 1d75d5aa2f5e5a7b14573489877e143f6a06cf5d (patch) | |
tree | aad8c19a3eb31dfa0b0b5ea0056d7dcebc2d6976 /src/x11/group.c | |
parent | 645d596f9d39e26c239a307fa699f11eeb921e18 (diff) | |
download | mutter-1d75d5aa2f5e5a7b14573489877e143f6a06cf5d.tar.gz |
group: Free group if returning early
If we get an error when fetching the window attributes, the group isn't ever
free'd, so use an autopointer instead, releasing the stolen one.
https://gitlab.gnome.org/GNOME/mutter/merge_requests/1195
Diffstat (limited to 'src/x11/group.c')
-rw-r--r-- | src/x11/group.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/x11/group.c b/src/x11/group.c index a3227467f..60f2d2902 100644 --- a/src/x11/group.c +++ b/src/x11/group.c @@ -41,7 +41,7 @@ static MetaGroup* meta_group_new (MetaX11Display *x11_display, Window group_leader) { - MetaGroup *group; + g_autofree MetaGroup *group = NULL; #define N_INITIAL_PROPS 3 Atom initial_props[N_INITIAL_PROPS]; int i; @@ -91,7 +91,7 @@ meta_group_new (MetaX11Display *x11_display, "Created new group with leader 0x%lx\n", group->group_leader); - return group; + return g_steal_pointer (&group); } static void |