[Bug 752714] dashdemux: index information returned by gst_mpd_client_get_next_fragment is not used

[GStreamer (GNOME Bugzilla)]

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

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

::: ext/dash/gstdashdemux.c
@@ +934,3 @@
+    } else {
+      /* header did not configure index, take values from fragment */
+      stream->fragment.index_uri = fragment.index_uri;

no, it doesn't.
fragment variable is being used to get the data from
gst_mpd_client_get_next_fragment function. When the fragment variable is
destroyed, it does not free the contained strings. The caller of
gst_mpd_client_get_next_fragment function (in this case
gst_dash_demux_stream_update_fragment_info) will take 
ownership of the data from fragment variable.

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