Re: [FFmpeg-devel] [PATCH] xface: Fix encoder crashes due to too small on-stack array.

2014-11-23 Thread Michael Niedermayer
On Sun, Nov 23, 2014 at 12:01:57PM +0100, Stefano Sabatini wrote: > On date Sunday 2014-11-23 06:53:04 +0100, Reimar Döffinger encoded: > > On Sat, Nov 22, 2014 at 11:14:45PM +0100, Reimar Döffinger wrote: > > > Also add a FATE test. > > > > > > Signed-off-by: Reimar Döffinger > > > --- > > > li

Re: [FFmpeg-devel] [PATCH] xface: Fix encoder crashes due to too small on-stack array.

2014-11-23 Thread Stefano Sabatini
On date Sunday 2014-11-23 06:53:04 +0100, Reimar Döffinger encoded: > On Sat, Nov 22, 2014 at 11:14:45PM +0100, Reimar Döffinger wrote: > > Also add a FATE test. > > > > Signed-off-by: Reimar Döffinger > > --- > > libavcodec/xface.h | 9 + > > libavcodec/xfaceenc.c |

Re: [FFmpeg-devel] [PATCH] xface: Fix encoder crashes due to too small on-stack array.

2014-11-22 Thread Reimar Döffinger
On Sat, Nov 22, 2014 at 11:14:45PM +0100, Reimar Döffinger wrote: > Also add a FATE test. > > Signed-off-by: Reimar Döffinger > --- > libavcodec/xface.h | 9 + > libavcodec/xfaceenc.c | 3 +++ > libavformat/nut.c | 1 + I forgot this part: > const AVCo

Re: [FFmpeg-devel] [PATCH] xface: Fix encoder crashes due to too small on-stack array.

2014-11-22 Thread Stefano Sabatini
On date Saturday 2014-11-22 23:14:45 +0100, Reimar Döffinger encoded: > Also add a FATE test. > > Signed-off-by: Reimar Döffinger > --- > libavcodec/xface.h | 9 + > libavcodec/xfaceenc.c | 3 +++ > libavformat/nut.c | 1 + > tests/fate/vcodec.mak

[FFmpeg-devel] [PATCH] xface: Fix encoder crashes due to too small on-stack array.

2014-11-22 Thread Reimar Döffinger
Also add a FATE test. Signed-off-by: Reimar Döffinger --- libavcodec/xface.h | 9 + libavcodec/xfaceenc.c | 3 +++ libavformat/nut.c | 1 + tests/fate/vcodec.mak | 5 + tests/ref/vsynth/vsynth1-xface | 4 tests/ref/vsynth/vsynth2-xface