[Bug 708636] collectpads: Should set *all* its pads to flushing when set_flushing is called, not only the ones in the public list
GStreamer (bugzilla.gnome.org)
bugzilla at gnome.org
Tue Sep 24 01:44:59 PDT 2013
https://bugzilla.gnome.org/show_bug.cgi?id=708636
GStreamer | gstreamer (core) | git
Sebastian Dröge (slomo) <slomo> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
Target Milestone|HEAD |1.2.0
--- Comment #6 from Sebastian Dröge (slomo) <slomo at circular-chaos.org> 2013-09-24 08:44:53 UTC ---
commit 20f1c96c89e9b5d57a83f81eee4191938c952764
Author: Sebastian Dröge <slomo at circular-chaos.org>
Date: Tue Sep 24 10:42:06 2013 +0200
collectpads: Make sure that the object lock is always taken when accessing
the private pad list
https://bugzilla.gnome.org/show_bug.cgi?id=708636
commit c79e5bbcad3f73fd231d4571d95506fd516f7a98
Author: Mathieu Duponchelle <mathieu.duponchelle at epitech.eu>
Date: Tue Sep 17 23:23:34 2013 +0200
collectpads: Use private pad list in set_flushing_unlocked
pads->data is the public list. It is dynamically rebuilt at each call to
check_collected, in check_pads to be specific. When you add a pad and
collectpads have been started, it is not added to the public list.
Thus there exists a possible race where :
1) You would add a pad to collectpads while running.
2) You set collectpads to flushing before check_collected has been called
again
-> the pad is not set to flushing
3) the pad starts pushing data as downstream might not be prepared, in the
case
of adder it then returns FLOW_FLUSHING.
4) elements like demuxers, when they get a FLOW_FLUSHING, stop their tasks,
never to be seen again.
https://bugzilla.gnome.org/show_bug.cgi?id=708636
--
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.
More information about the gstreamer-bugs
mailing list