[Bug 735663] dashdemux: synchronize with the download loop thread before signalling it

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=735663
  GStreamer | gst-plugins-bad | git

--- Comment #14 from Matthieu Bouron <matthieu.bouron at collabora.com> 2014-09-18 13:50:33 UTC ---
(In reply to comment #13)
> Review of attachment 286138 [details]:
> 
> ::: ext/dash/gstdashdemux.c
> @@ -2005,3 @@
>    }
> 
> -  stream->last_ret = ret;
> 
> Shouldn't this be kept with some locking, to assign stream->last_ret to
> GST_FLOW_NOT_LINKED when this is the case ?

I've test the patch and it seems to work fine with the previous change i've
mentionned. If that can help you here is my branch that includes all the
remaining dash fixes rebased on top of master:

http://cgit.collabora.com/git/user/mateo/gst-plugins-bad.git/log/?h=dash-fixes

-- 
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.