[Bug 734716] New: bus: signal watched added in a new thread-default context can't be removed

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=734716
  GStreamer | gstreamer (core) | git

           Summary: bus: signal watched added in a new thread-default
                    context can't be removed
    Classification: Platform
           Product: GStreamer
           Version: git
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
        AssignedTo: gstreamer-bugs at lists.freedesktop.org
        ReportedBy: phil at base-art.net
         QAContact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---


The watch is attached to the current thread-default main context but
gst_bus_remove_signal_watch() invokes g_source_remove() which operates on the
default main context.

So in that configuration the source won't be removed and GLib emits a critical
warning.

I think the solution would be to get the thread-default context in
_remove_signal_watch(), look for the source in that context and destroy it.
I'll post a patch :)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.