Re: [FFmpeg-devel] [PATCH] avformat/tta: fix crash with corrupted files

2015-02-03 Thread Michael Niedermayer
On Tue, Feb 03, 2015 at 02:40:58PM +, Paul B Mahol wrote: > On 2/3/15, wm4 wrote: > > av_add_index_entry() can fail, for example because the parameters are > > invalid, or because memory allocation fails. Check this; it can actually > > happen with corrupted files. > > > > The second hunk is j

Re: [FFmpeg-devel] [PATCH] avformat/tta: fix crash with corrupted files

2015-02-03 Thread Paul B Mahol
On 2/3/15, wm4 wrote: > av_add_index_entry() can fail, for example because the parameters are > invalid, or because memory allocation fails. Check this; it can actually > happen with corrupted files. > > The second hunk is just for robustness. Just in case functions like > ff_reduce_index() remove

[FFmpeg-devel] [PATCH] avformat/tta: fix crash with corrupted files

2015-02-03 Thread wm4
av_add_index_entry() can fail, for example because the parameters are invalid, or because memory allocation fails. Check this; it can actually happen with corrupted files. The second hunk is just for robustness. Just in case functions like ff_reduce_index() remove entries. (Not sure if this can ac