[Bug 786111] tsdemux: incorrectly parsing non-timestamp byte sequence in PES header as PTS time stamp

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Sep 1 08:56:57 UTC 2017


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

--- Comment #18 from Edward Hervey <bilboed at bilboed.com> ---
commit 52d0ef2665e0c620e503903d6eebe0771f92c5ba (HEAD -> master, origin/master,
origin/HEAD)
Author: Edward Hervey <edward at centricular.com>
Date:   Wed Aug 30 08:37:04 2017 +0200

    tsdemux: Make jp2k handling more robust and efficient

    * Avoid copying the pending data and instead create a buffer directly from
      that data with the appropriate offset.
    * Locate the jp2k magic to determine the exact location of the (first)
frame
      data instead of assuming that the header is of an expected size

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

commit c393f0d768a5c86dade48d991fb7d890f6dc5221
Author: Edward Hervey <edward at centricular.com>
Date:   Tue Aug 29 11:14:59 2017 +0200

    tsdemux: Handle quirk in jp2k es header handling

    The jp2k specification (ITU-T T.800) specifies that the 'brat' box
    has two fields and the second one (AUF2) can be set to 0 for progressive
    streams.

    The problem is that the mpeg-ts specification (ITU-T H.222.0 06/2012)
    says that the AUF2 field is only present if the stream is interlaced

    In order to cope with both situation, accept those next 32bit if the
    stream is marked as progressive and those bits contain 0

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

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