From 4170ed17212b5e0502af778ea48b3a610bbdd51c Mon Sep 17 00:00:00 2001
From: Dan Williams <dcbw@redhat.com>
Date: Fri, 7 Mar 2014 09:28:16 -0600
Subject: [PATCH] Fix use-after-free in _free_pending_clip()

https://bugzilla.gnome.org/show_bug.cgi?id=725855
---
 ges/ges-base-xml-formatter.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ges/ges-base-xml-formatter.c b/ges/ges-base-xml-formatter.c
index 5edb89dfa0..3758a78b7e 100644
--- a/ges/ges-base-xml-formatter.c
+++ b/ges/ges-base-xml-formatter.c
@@ -537,7 +537,6 @@ _free_pending_effect (PendingEffects * pend)
 static void
 _free_pending_clip (GESBaseXmlFormatterPrivate * priv, PendingClip * pend)
 {
-  g_free (pend->id);
   gst_object_unref (pend->layer);
   if (pend->properties)
     gst_structure_free (pend->properties);
@@ -545,6 +544,7 @@ _free_pending_clip (GESBaseXmlFormatterPrivate * priv, PendingClip * pend)
   g_list_free_full (pend->pending_bindings,
       (GDestroyNotify) _free_pending_binding);
   g_hash_table_remove (priv->clipid_pendings, pend->id);
+  g_free (pend->id);
   g_slice_free (PendingClip, pend);
 }