Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-25 Thread Andreas Cadhalpun
On 25.11.2016 02:59, Michael Niedermayer wrote: > On Fri, Nov 25, 2016 at 02:26:24AM +0100, Andreas Cadhalpun wrote: >> mss2.c |8 +++- >> 1 file changed, 7 insertions(+), 1 deletion(-) >> 958ee0811485404a0662a1540fcb8b131423947b >> 0001-mss2-only-use-error-correction-for-matching-block-

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-24 Thread Michael Niedermayer
On Fri, Nov 25, 2016 at 02:26:24AM +0100, Andreas Cadhalpun wrote: > On 25.11.2016 01:38, Michael Niedermayer wrote: > > On Fri, Nov 25, 2016 at 12:03:30AM +0100, Andreas Cadhalpun wrote: > >> mss2.c | 13 ++--- > >> 1 file changed, 10 insertions(+), 3 deletions(-) > >> 884b912643244a420

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-24 Thread Andreas Cadhalpun
On 25.11.2016 01:38, Michael Niedermayer wrote: > On Fri, Nov 25, 2016 at 12:03:30AM +0100, Andreas Cadhalpun wrote: >> mss2.c | 13 ++--- >> 1 file changed, 10 insertions(+), 3 deletions(-) >> 884b912643244a4205bac63faedfa0c048bcc97a >> 0001-mss2-only-use-error-correction-for-matching

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-24 Thread Michael Niedermayer
On Fri, Nov 25, 2016 at 12:03:30AM +0100, Andreas Cadhalpun wrote: > On 24.11.2016 17:57, Michael Niedermayer wrote: > > On Thu, Nov 24, 2016 at 05:45:38PM +0100, Michael Niedermayer wrote: > >> Is it correct that your cases uses > >> decode_wmv9() -> vc1_decode_i_blocks() ? > > Yes. > > >> these

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-24 Thread Andreas Cadhalpun
On 24.11.2016 17:57, Michael Niedermayer wrote: > On Thu, Nov 24, 2016 at 05:45:38PM +0100, Michael Niedermayer wrote: >> Is it correct that your cases uses >> decode_wmv9() -> vc1_decode_i_blocks() ? Yes. >> these decode a rectangele up to end_mb_y, end_mb_x >> does this mismatch with what later

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-24 Thread Michael Niedermayer
On Thu, Nov 24, 2016 at 05:45:38PM +0100, Michael Niedermayer wrote: > On Thu, Nov 24, 2016 at 02:14:59AM +0100, Andreas Cadhalpun wrote: > > On 23.11.2016 15:01, Michael Niedermayer wrote: > > > On Mon, Nov 07, 2016 at 10:32:29PM +0100, Andreas Cadhalpun wrote: > > >> This fixes a heap-buffer-over

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-24 Thread Michael Niedermayer
On Thu, Nov 24, 2016 at 02:14:59AM +0100, Andreas Cadhalpun wrote: > On 23.11.2016 15:01, Michael Niedermayer wrote: > > On Mon, Nov 07, 2016 at 10:32:29PM +0100, Andreas Cadhalpun wrote: > >> This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 > >> with > >> coded_width/coded_

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-23 Thread Andreas Cadhalpun
On 23.11.2016 15:01, Michael Niedermayer wrote: > On Mon, Nov 07, 2016 at 10:32:29PM +0100, Andreas Cadhalpun wrote: >> This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 with >> coded_width/coded_height larger than width/height. >> >> Signed-off-by: Andreas Cadhalpun >> --- >

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-23 Thread Michael Niedermayer
On Mon, Nov 07, 2016 at 10:32:29PM +0100, Andreas Cadhalpun wrote: > This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 with > coded_width/coded_height larger than width/height. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/mpegpicture.c | 12 ++-- > 1 file

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-22 Thread Andreas Cadhalpun
On 07.11.2016 22:32, Andreas Cadhalpun wrote: > This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 with > coded_width/coded_height larger than width/height. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/mpegpicture.c | 12 ++-- > 1 file changed, 6 insertion

[FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-07 Thread Andreas Cadhalpun
This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 with coded_width/coded_height larger than width/height. Signed-off-by: Andreas Cadhalpun --- libavcodec/mpegpicture.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavcodec/mpegpicture.c