Re: [FFmpeg-devel] [PATCH] avformat/icodec: Fix crash probing fuzzed file

2016-02-15 Thread Michael Niedermayer
On Mon, Feb 15, 2016 at 11:27:20AM -0800, Mark Harris wrote: > On Mon, Feb 15, 2016 at 11:02 AM, Michael Niedermayer > wrote: > > On Mon, Feb 15, 2016 at 09:57:51AM -0800, Mark Harris wrote: > >> Avoid invalid memory read/crash when ico offset >= 0xfff8. > >> Base64-encoded example: AAABADAwMD

Re: [FFmpeg-devel] [PATCH] avformat/icodec: Fix crash probing fuzzed file

2016-02-15 Thread Mark Harris
On Mon, Feb 15, 2016 at 11:02 AM, Michael Niedermayer wrote: > On Mon, Feb 15, 2016 at 09:57:51AM -0800, Mark Harris wrote: >> Avoid invalid memory read/crash when ico offset >= 0xfff8. >> Base64-encoded example: AAABADAwMDAwMAAAMAAwMDAw/P///w== >> --- >> libavformat/icodec.c | 2 +- >> 1 fil

Re: [FFmpeg-devel] [PATCH] avformat/icodec: Fix crash probing fuzzed file

2016-02-15 Thread Michael Niedermayer
On Mon, Feb 15, 2016 at 09:57:51AM -0800, Mark Harris wrote: > Avoid invalid memory read/crash when ico offset >= 0xfff8. > Base64-encoded example: AAABADAwMDAwMAAAMAAwMDAw/P///w== > --- > libavformat/icodec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavforma

Re: [FFmpeg-devel] [PATCH] avformat/icodec: Fix crash probing fuzzed file

2016-02-15 Thread Michael Bradshaw
On Mon, Feb 15, 2016 at 9:57 AM, Mark Harris wrote: > Avoid invalid memory read/crash when ico offset >= 0xfff8. > Base64-encoded example: AAABADAwMDAwMAAAMAAwMDAw/P///w== > --- > libavformat/icodec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/icodec.

[FFmpeg-devel] [PATCH] avformat/icodec: Fix crash probing fuzzed file

2016-02-15 Thread Mark Harris
Avoid invalid memory read/crash when ico offset >= 0xfff8. Base64-encoded example: AAABADAwMDAwMAAAMAAwMDAw/P///w== --- libavformat/icodec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/icodec.c b/libavformat/icodec.c index 6ddb901..8f84337 100644 --- a/libav