[Bug 755169] dashdemux: can we have multiple seek events at the same time?

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=755169

--- Comment #23 from Florin Apostol <florin.apostol at oregan.net> ---
Review of attachment 311886:
 --> (https://bugzilla.gnome.org/review?bug=755169&attachment=311886)

::: gst-libs/gst/adaptivedemux/gstadaptivedemux.c
@@ +1387,3 @@
+  }
+
+  GST_MANIFEST_UNLOCK (demux);

copying the list and refcounting the streams does not solve all problems and I
don't think it could be a solution

gst_adaptive_demux_stream_download_loop can call
gst_adaptive_demux_advance_period which calls gst_adaptive_demux_expose_streams
which sets old_streams = demux->streams; demux->streams = demux->next_streams;
If you release the lock in gst_adaptive_demux_stop_tasks, when you join tasks
you are killing streams that were moved to old_streams!

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.