brl/mutter
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[animation] Do not leak timelines

Browse Source 
The timeline created when calling set_timeline(NULL) is referenced
even though we implicitly own it. When the Animation is destroyed,
the timeline is then leaked.

Thanks to: Richard Heatley <richard.heatley@starleaf.com>

Fixes bug:

  http://bugzilla.openedhand.com/show_bug.cgi?id=1548
This commit is contained in:
Emmanuele Bassi 2009-05-07 18:39:07 +01:00
parent 21e3901d62
commit 2f5012e38c
1 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
clutter/clutter-animation.c
View File

@ -1115,12 +1115,12 @@ clutter_animation_set_timeline (ClutterAnimation *animation,
priv = animation->priv;
if (timeline && priv->timeline == timeline)
if (timeline != NULL && priv->timeline == timeline)
return;
g_object_freeze_notify (G_OBJECT (animation));
if (priv->timeline)
if (priv->timeline != NULL)
{
if (priv->timeline_started_id)
g_signal_handler_disconnect (priv->timeline,
@ -1136,13 +1136,17 @@ clutter_animation_set_timeline (ClutterAnimation *animation,
priv->timeline = 0;
}
if (!timeline)
timeline = g_object_new (CLUTTER_TYPE_TIMELINE,
"duration", priv->duration,
"loop", priv->loop,
NULL);
if (timeline == NULL)
{
timeline = g_object_new (CLUTTER_TYPE_TIMELINE,
"duration", priv->duration,
"loop", priv->loop,
NULL);
}
else
{
g_object_ref (timeline);
priv->duration = clutter_timeline_get_duration (timeline);
g_object_notify (G_OBJECT (animation), "duration");
@ -1150,7 +1154,7 @@ clutter_animation_set_timeline (ClutterAnimation *animation,
g_object_notify (G_OBJECT (animation), "loop");
}
priv->timeline = g_object_ref (timeline);
priv->timeline = timeline;
g_object_notify (G_OBJECT (animation), "timeline");
priv->timeline_started_id =