[Bug 793333] matroskademux: Allow Matroska headers to be read more than once

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Sep 21 19:36:06 UTC 2018


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

Thibault Saunier <tsaunier at gnome.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #373732|none                        |needs-work
             status|                            |

--- Comment #18 from Thibault Saunier <tsaunier at gnome.org> ---
Review of attachment 373732:
 --> (https://bugzilla.gnome.org/review?bug=793333&attachment=373732)

Looks good otherwise.

::: gst/matroska/matroska-demux.c
@@ +3138,3 @@

+static GstFlowReturn
+gst_matroska_demux_reparse_tracks (GstMatroskaDemux * demux, GstEbmlRead *
ebml)

update_tracks?

@@ +3178,3 @@
+            gst_matroska_read_common_stream_from_num (&demux->common,
+            new_track->num);
+        g_assert (old_track_index != -1);

That should not be an assertion, if there is a mismatch the track might not be
found, and we should cleanly error out.

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