Commit 1b4dce6f authored by Adel Gadllah's avatar Adel Gadllah
Browse files

meta-window-group: Plug memory leak

We have to free the list returned by clutter_actor_get_effects() .

https://bugzilla.gnome.org/show_bug.cgi?id=660464
parent 3bb17f06
...@@ -108,7 +108,7 @@ meta_window_group_paint (ClutterActor *actor) ...@@ -108,7 +108,7 @@ meta_window_group_paint (ClutterActor *actor)
cairo_region_t *unredirected_window_region = NULL; cairo_region_t *unredirected_window_region = NULL;
ClutterActor *stage; ClutterActor *stage;
cairo_rectangle_int_t visible_rect, unredirected_rect; cairo_rectangle_int_t visible_rect, unredirected_rect;
GList *children, *l; GList *children, *l, *effects;
MetaWindowGroup *window_group = META_WINDOW_GROUP (actor); MetaWindowGroup *window_group = META_WINDOW_GROUP (actor);
MetaCompScreen *info = meta_screen_get_compositor_data (window_group->screen); MetaCompScreen *info = meta_screen_get_compositor_data (window_group->screen);
...@@ -161,8 +161,11 @@ meta_window_group_paint (ClutterActor *actor) ...@@ -161,8 +161,11 @@ meta_window_group_paint (ClutterActor *actor)
* as well for the same reason, but omitted for simplicity in the * as well for the same reason, but omitted for simplicity in the
* hopes that no-one will do that. * hopes that no-one will do that.
*/ */
if (clutter_actor_get_effects (l->data) != NULL) if ((effects = clutter_actor_get_effects (l->data)) != NULL)
continue; {
g_list_free (effects);
continue;
}
if (META_IS_WINDOW_ACTOR (l->data)) if (META_IS_WINDOW_ACTOR (l->data))
{ {
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment