Re: [FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-03 Thread Hein-Pieter van Braam
On Mon, 2017-07-03 at 11:59 +0200, Paul B Mahol wrote: > > > You are comparing pointer with same pointer increased by some value. > > This is wrong way to fix it. Urgh, you're right, of course. I don't know what I was thinking. ___ ffmpeg-devel mailin

Re: [FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-03 Thread Paul B Mahol
On 7/3/17, Hein-Pieter van Braam wrote: > On Mon, 2017-07-03 at 11:43 +0200, Paul B Mahol wrote: >> On 7/2/17, Hein-Pieter van Braam wrote: >> > Fixes: 6503 crash with fuzzed file >> > >> > Signed-off-by: Hein-Pieter van Braam >> > --- >> > libavcodec/interplayvideo.c | 8 >> > 1 file

Re: [FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-03 Thread Hein-Pieter van Braam
On Mon, 2017-07-03 at 11:43 +0200, Paul B Mahol wrote: > On 7/2/17, Hein-Pieter van Braam wrote: > > Fixes: 6503 crash with fuzzed file > > > > Signed-off-by: Hein-Pieter van Braam > > --- > >  libavcodec/interplayvideo.c | 8 > >  1 file changed, 8 insertions(+) > > > > diff --git a/li

Re: [FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-03 Thread Paul B Mahol
On 7/2/17, Hein-Pieter van Braam wrote: > Fixes: 6503 crash with fuzzed file > > Signed-off-by: Hein-Pieter van Braam > --- > libavcodec/interplayvideo.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/libavcodec/interplayvideo.c b/libavcodec/interplayvideo.c > index d6f484a..86

[FFmpeg-devel] [PATCH v2] avcodec/interplayvideo: Check sizes of decode buffers

2017-07-02 Thread Hein-Pieter van Braam
Fixes: 6503 crash with fuzzed file Signed-off-by: Hein-Pieter van Braam --- libavcodec/interplayvideo.c | 8 1 file changed, 8 insertions(+) diff --git a/libavcodec/interplayvideo.c b/libavcodec/interplayvideo.c index d6f484a..86530e6 100644 --- a/libavcodec/interplayvideo.c +++ b/liba