[Bug 774462] New: glwindow: found memory leak of navigation_thread

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 774462
           Summary: glwindow: found memory leak of navigation_thread
    Classification: Platform
           Product: GStreamer
           Version: 1.10.0
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: jared.hu at nxp.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

Hi, navigation thread memory leak has been fixed in below bugzilla:
https://bugzilla.gnome.org/show_bug.cgi?id=758820

But the g_thread_join() has been removed again in gstreamer 1.10.0 which will
cause memory leak again.

It was removed in this commit:

>From 9ad4c67578e95a48e2969c8272e2c94ff3fc50b5 Mon Sep 17 00:00:00 2001
From: Matthew Waters <matthew at centricular.com>
Date: Wed, 5 Oct 2016 18:32:09 +1100
Subject: glwindow: don't use g_thread_join() to join the navigation thread

Using g_thread_join() in _finalize() handlers may result in a deadlock
joining the current thread when the last reference is held by a signal
handler.

I think if g_thread_join() will cause deadlock, then we also need use
g_thread_unref() to release this thread. Is that right?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.