After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 618874 - [pulse] regression: broken mainloop locking
[pulse] regression: broken mainloop locking
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal blocker
: 0.10.23
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-05-17 12:26 UTC by Tim-Philipp Müller
Modified: 2010-05-17 15:59 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
pulse: Don't lock the mainloop in NULL (3.98 KB, patch)
2010-05-17 15:17 UTC, Sebastian Dröge (slomo) 		committed Details | Review

Description Tim-Philipp Müller 2010-05-17 12:26:37 UTC 
$ gst-inspect-0.10 pulsesink
       (snip)
  device-name         : Human-readable name of the sound device
Assertion 'm' failed at pulse/thread-mainloop.c:171, function pa_threaded_mainloop_lock(). Aborting.
Comment 1 Tim-Philipp Müller 2010-05-17 12:27:22 UTC 
Tries to lock a NULL main loop.
Comment 2 Sebastian Dröge (slomo) 2010-05-17 12:53:02 UTC 
Same for pulsesrc
Comment 3 Sebastian Dröge (slomo) 2010-05-17 12:57:11 UTC 
The property getters/setters should all check if a mainloop was allocated already... if nobody is faster I'll create patch later.
Comment 4 Sebastian Dröge (slomo) 2010-05-17 15:17:40 UTC 
Created attachment 161232 [details] [review]
pulse: Don't lock the mainloop in NULL
Comment 5 Sebastian Dröge (slomo) 2010-05-17 15:58:24 UTC 
commit a8103facc59b9dde07ee95d026c3fd2323e278bb
Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
Date:   Mon May 17 17:17:01 2010 +0200

    pulse: Don't lock the mainloop in NULL