[Bug 752213] tee: Avoid race condition while forwarding sticky events
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Wed Mar 30 12:43:12 UTC 2016
https://bugzilla.gnome.org/show_bug.cgi?id=752213
Sebastian Dröge (slomo) <slomo at coaxion.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #324940|none |needs-work
status| |
--- Comment #31 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 324940:
--> (https://bugzilla.gnome.org/review?bug=752213&attachment=324940)
::: plugins/elements/gsttee.c
@@ +567,3 @@
gboolean res;
+ res = gst_pad_event_default (pad, parent, event);
You can't use gst_pad_event_default() here, that's racy and why I didn't do
that :)
But in the forward function you will *also* need to check the current pad
cookie and if it's wrong you additionally need to forward all events there.
That should fix the sticky event misordering problem
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
More information about the gstreamer-bugs
mailing list