Re: [FFmpeg-devel] [PATCH] lavf/wavdec: add sanity check for AVCodecContext.channels

2014-08-18 Thread Michael Niedermayer
On Mon, Aug 18, 2014 at 07:44:39PM -0300, James Almer wrote: > On 18/08/14 4:42 AM, Michael Niedermayer wrote: > > On Mon, Aug 18, 2014 at 03:14:01AM -0300, James Almer wrote: > >> Fixes ticket #3862. > >> As a side effect, this also fixes aac_latm in wav. > >> > >> Signed-off-by: James Almer > >

Re: [FFmpeg-devel] [PATCH] lavf/wavdec: add sanity check for AVCodecContext.channels

2014-08-18 Thread James Almer
On 18/08/14 4:42 AM, Michael Niedermayer wrote: > On Mon, Aug 18, 2014 at 03:14:01AM -0300, James Almer wrote: >> Fixes ticket #3862. >> As a side effect, this also fixes aac_latm in wav. >> >> Signed-off-by: James Almer > > applied > > >> --- >> Maybe a check for channels <= 0 should be also a

Re: [FFmpeg-devel] [PATCH] lavf/wavdec: add sanity check for AVCodecContext.channels

2014-08-18 Thread Michael Niedermayer
On Mon, Aug 18, 2014 at 03:14:01AM -0300, James Almer wrote: > Fixes ticket #3862. > As a side effect, this also fixes aac_latm in wav. > > Signed-off-by: James Almer applied > --- > Maybe a check for channels <= 0 should be also added to ff_get_wav_header() > right after the sample_rate one?

[FFmpeg-devel] [PATCH] lavf/wavdec: add sanity check for AVCodecContext.channels

2014-08-17 Thread James Almer
Fixes ticket #3862. As a side effect, this also fixes aac_latm in wav. Signed-off-by: James Almer --- Maybe a check for channels <= 0 should be also added to ff_get_wav_header() right after the sample_rate one? libavformat/wavdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff -