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

Michael Niedermayer michael at niedermayer.cc
Fri Feb 8 13:00:25 EET 2019


On Thu, Feb 07, 2019 at 08:03:01PM +0000, Derek Buitenhuis wrote:
> 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.

will apply

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190208/c44d426c/attachment.sig>


More information about the ffmpeg-devel mailing list