Re: [FFmpeg-devel] [PATCH] Fix to prevent runaway ac3 detection

2017-01-28 Thread Marijn Meijles
On Sat, Jan 28, 2017 at 06:04:02PM +, Kieran Kunhya wrote: > On Sat, 28 Jan 2017 at 12:53 Marijn Meijles wrote: > > > This is a fix for https://trac.ffmpeg.org/ticket/6106 . > > > > The issue is that when detecting a swapped AC3 marker the data of the > >

Re: [FFmpeg-devel] [PATCH] Fix to prevent runaway ac3 detection by looking at the actual frame rather than the first detected frame.

2017-01-28 Thread Marijn Meijles
On Sat, Jan 28, 2017 at 07:56:51PM +0100, Michael Niedermayer wrote: > On Sat, Jan 28, 2017 at 01:52:30PM +0100, Marijn Meijles wrote: > > Signed-off-by: Marijn Meijles > > --- > > libavformat/ac3dec.c | 8 > > 1 file changed, 4 insertions(+), 4 deletion

[FFmpeg-devel] [PATCH] Fix to prevent runaway ac3 detection by looking at the actual frame rather than the first detected frame.

2017-01-28 Thread Marijn Meijles
Signed-off-by: Marijn Meijles --- libavformat/ac3dec.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/ac3dec.c b/libavformat/ac3dec.c index 363a32e..e85b0ac 100644 --- a/libavformat/ac3dec.c +++ b/libavformat/ac3dec.c @@ -49,8 +49,8 @@ static int

[FFmpeg-devel] [PATCH] Fix to prevent runaway ac3 detection

2017-01-28 Thread Marijn Meijles
This is a fix for https://trac.ffmpeg.org/ticket/6106 . The issue is that when detecting a swapped AC3 marker the data of the frame is swapped. However, in subsequent frames the data swapped is taken from the first frame rather than the current frame. So all frames are considered to be correct eve