[FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

Derek Buitenhuis derek.buitenhuis at gmail.com
Thu Nov 3 00:26:35 EET 2016


On 10/24/2016 6:43 AM, Sasi Inguva wrote:
> Just sent a patch, correcting a bug in the edit list code. PTAL.

Hi again, it's me. Back with more files this patch set broke :).

This time it's very easy to reproduce:

    $ ffmpeg -i sample.mp4 -f null -

This works before your patchset, and also if I use -ignore_editlist 1.

After your patchset, it fails with:

    [mov,mp4,m4a,3gp,3g2,mj2 @ 0x34a1380] Missing key frame while reordering index according to edit list
    [mov,mp4,m4a,3gp,3g2,mj2 @ 0x34a1380] Could not find codec parameters for stream 0 (Video: h264 (avc1 / 0x31637661), none, 1280x720, 2000 kb/s): unspecified pixel format
    [...]
    [buffer @ 0x38b0760] Unable to parse option value "-1" as pixel format
        Last message repeated 1 times
    [buffer @ 0x38b0760] Error setting option pix_fmt to value -1.
    [graph 0 input from stream 0:0 @ 0x34a11a0] Error applying options to the filter.
    Error opening filters!

I will forward you the sample privately.

I've seen a lot (hundreds/thousands) of files broken like this so far.

Cheers,
- Derek



More information about the ffmpeg-devel mailing list