[Bug 759615] New: DTS > PTS creates invalid mp4 files

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 759615
           Summary: DTS > PTS creates invalid mp4 files
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: thaytan at noraisin.net
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

I think there are other bugs related to this, but not sure where - feel free to
dup.

When running a pipeline like this: rtspsrc location=... ! rtph264depay !
h264parse ! mp4mux name=mux ! filesink it seems common that at the start
there's a packet with DTS > PTS, possibly due to udpsrc DTS timestamping stuff,
I'm not sure.

When it happens, the logic inside mp4mux for collecting first_ts and first_dts
ends up with first_dts > first_ts, and the in gst_qt_mux_update_edit_lists() it
does ctts = qtpad->first_ts - qtpad->first_dts and then writes a very large
uint32 into the media_start time for the edit list - creating a file that
GStreamer doesn't play (although other apps VLC/mplayer that I guess don't
handle edit lists work).

Attaching a patch that makes it work, but I think ignores a more fundamental
problem that needs addressing.

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