[Bug 643087] pulsesink: deadlock in gst_pulseringbuffer_open_device

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=643087
  GStreamer | gst-plugins-good | git

--- Comment #17 from Philip Jägenstedt <philipj at opera.com> 2011-02-24 14:02:47 UTC ---
Arun, what you describe is correct. Your patch is almost identical to my
<https://bugzilla.gnome.org/show_bug.cgi?id=643087#c7> on which I commented
"introduces another deadlock, probably".

The problem is that your code takes the lock in order "resource lock, mainloop
lock", while there's other code that takes it in the reverse order, at least
gst_pulseringbuffer_close_device and
gst_pulseringbuffer_release+gst_pulsering_destroy_stream.

In other words, that fix will introduce another deadlock which will happen if
you're creating and destroying pulsesinks at the same time, which will
eventually happen.

-- 
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.