Re: [FFmpeg-devel] [mov] Error on too large stsd entry counts.

2018-08-31 Thread Michael Niedermayer
On Thu, Aug 30, 2018 at 03:21:02PM -0700, Dale Curtis wrote: > Entries are always at least 8 bytes per the parsing code, so if we > see an impossible entry count avoid massive allocations. This is > similar to an existing check in mov_read_stsc(). > > Since ff_mov_read_stsd_entries() does eof chec

[FFmpeg-devel] [mov] Error on too large stsd entry counts.

2018-08-30 Thread Dale Curtis
Entries are always at least 8 bytes per the parsing code, so if we see an impossible entry count avoid massive allocations. This is similar to an existing check in mov_read_stsc(). Since ff_mov_read_stsd_entries() does eof checks, an alternative approach could be to clamp the entry count to atom.s