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 384060 - gst_xoverlay_set_xwindow_id() causing lockups with x(v)imagesink
gst_xoverlay_set_xwindow_id() causing lockups with x(v)imagesink
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.12
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-12-09 14:39 UTC by Jorn Baayen
Modified: 2007-01-08 13:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to ximagesink (724 bytes, patch)
2006-12-09 14:40 UTC, Jorn Baayen
none Details | Review
Patch to xvimagesink (755 bytes, patch)
2006-12-09 14:41 UTC, Jorn Baayen
none Details | Review

Description Jorn Baayen 2006-12-09 14:39:50 UTC
When calling gst_xoverlay_set_xwindow_id() during the 'prepare-xwindow-id' signal in a thread different from the one where x(v)imagesink lives, occasional lockups and/or X errors occur. I noticed flow_lock is locked whenever xwindow is being touched, except in setcaps(), just after firing the 'prepare-xwindow-id' signal. Locking flow_lock there as well appears to fix the problems.
Comment 1 Jorn Baayen 2006-12-09 14:40:47 UTC
Created attachment 78024 [details] [review]
Patch to ximagesink
Comment 2 Jorn Baayen 2006-12-09 14:41:10 UTC
Created attachment 78025 [details] [review]
Patch to xvimagesink
Comment 3 René Stadler 2006-12-09 14:55:31 UTC
See also: Bug #384008
Comment 4 Julien MOUTTE 2007-01-07 18:51:54 UTC
Fixed in CVS HEAD. Please try it and reopen the bug if it is still not ok for you.