gldisplay: fix list leak
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Fri, 7 Apr 2017 09:24:19 +0000 (10:24 +0100)
committerTim-Philipp Müller <tim@centricular.com>
Sat, 9 Dec 2017 19:32:27 +0000 (19:32 +0000)
Windows aren't always removed in time, and it turns out to be
very, very hard to remove a window in a way that's not racy and
not deadlocky. Since the window itself doesn't leak, freeing
the list on object destruction is enough.

https://bugzilla.gnome.org/show_bug.cgi?id=781018

gst-libs/gst/gl/gstgldisplay.c

index 299eca63c45f8c79e9e6668d1ff95b3b8b8c3ccd..54a135d92de2db3a1598427a962b89d7bd1a330c 100644 (file)
@@ -251,6 +251,7 @@ gst_gl_display_finalize (GObject * object)
     g_free (l->data);
   }
 
+  g_list_free (display->windows);
   g_list_free (display->priv->contexts);
 
   g_cond_clear (&display->priv->thread_cond);