[Bug 729371] dash/qtdemux: Reported video size does not correspond to what avdec_h264 states

[GStreamer (GNOME Bugzilla)]

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

Edward Hervey <bilboed at bilboed.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|git master                  |1.11.91

--- Comment #14 from Edward Hervey <bilboed at bilboed.com> ---
commit 26f9869668ec5f18e011cd6f936b6f62778b051f
Author: Edward Hervey <edward at centricular.com>
Date:   Wed Apr 12 11:03:24 2017 +0200

    qtdemux: Add out-of-bound check

    Make sure we don't read invalid memory

commit 9ac3861a9bd91d534ba29933f05595e7416dfab7
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Wed Apr 27 12:17:37 2016 -0300

    qtdemux: move parsing of tkhd out of stsd entry loop

    It needs only to be read once.

commit cf6733a55eacf03dd096d95f74f224d8bc7d427b
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Thu Apr 7 12:23:35 2016 -0300

    qtdemux: check for a different stsd entry before pushing a sample

    Before pushing a sample, check if there was a change in the current
    stsd entry. This patch also assumes that the first stsd entry is
    used as default for the first sample. It might cause an uneeded
    caps renegotiation when this isn't the case.

commit 86b427dc70562f891a551ffc9f96cefe1cafcddd
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Wed Apr 6 12:55:18 2016 -0300

    qtdemux: parse all stsd entries

    stsd can have multiple format entries, parse them all.

    This is required to play DVB DASH profile that uses multiple entries
    to identify the different available bitrates/options on dash streams

    The stream format-specific data is not stored into QtDemuxStreamStsdEntry

commit 54e252e0955287bf6971fba89666e9fa4901e118
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Tue Apr 5 14:34:00 2016 -0300

    qtdemux: rework stsd sample entries access

    Instead of using the stsd as a base pointer, use the actual stsd
    entry as the stsd can have multiple entries. This is rarely used
    for file playback but is a possible profile with in DVB DASH specs.

    This still doesn't support stsd with multiple entries but makes it
    easier to do so.

commit bd32bcc36cc7b08a0aecbad3bcc5dbd7308b9e19
Author: Thiago Santos <thiagoss at osg.samsung.com>
Date:   Tue Apr 5 18:00:10 2016 -0300

    qtdemux: get stsd child by index instead of type

    There might be multiple children with the same type

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