[FFmpeg-devel] [PATCH] avformat/mov.c: require tfhd to begin parsing trun

Derek Buitenhuis derek.buitenhuis at gmail.com
Thu Feb 7 22:03:01 EET 2019


On 07/02/2019 19:30, Chris Cunningham wrote:
> This will reject the file entirely. The testcase I have (can share once
> chromium bug is fixed) was previously hitting an av_assert0 in
> mov_read_trun, arguably also a total rejection ;). Recovery for this could
> be pretty tricky since the dropped trun will break decode dependencies. I
> would be surprised to find files with missing tfhd's in the wild (outside
> of fuzzer / maliciously crafted sorts).

You would be surprised what kind of garbage things can produce...

In any case, sounds OK to me.

- Derek


More information about the ffmpeg-devel mailing list