Re: [FFmpeg-devel] [PATCH/RFC]lavf/avidec: Do not fail for very large idx1 tags

2016-10-17 Thread Carl Eugen Hoyos
2016-10-17 22:32 GMT+02:00 Michael Niedermayer :
> On Mon, Oct 17, 2016 at 06:29:42PM +0200, Carl Eugen Hoyos wrote:
>> Hi!
>>
>> Attached patch fixes the file mentioned in github pull request 197,
>> it can be played with MPlayer, vlc, xine and totem.
>>
>> Please comment, Carl Eugen
>
>>  avidec.c |2 ++
>>  1 file changed, 2 insertions(+)
>> 0b5d5e672b7ec13a8c8aa1597ceb21b16b5e61b8  
>> 0001-lavf-avidec-Do-not-fail-for-very-large-idx1-tags.patch
>> From 646a765749e08bb48f74488ad9f539cbc736673c Mon Sep 17 00:00:00 2001
>> From: Carl Eugen Hoyos 
>> Date: Mon, 17 Oct 2016 18:25:48 +0200
>> Subject: [PATCH] lavf/avidec: Do not fail for very large idx1 tags.
>>
>> Fixes demuxing the sample file from github pull request 197,
>> the size of its idx1 tag is 6171936 bytes, followed by a JUNK
>> tag of 9505704 bytes.
>> ---
>>  libavformat/avidec.c |2 ++
>>  1 file changed, 2 insertions(+)
>
> LGTM

Patch applied.

Thank you, Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


Re: [FFmpeg-devel] [PATCH/RFC]lavf/avidec: Do not fail for very large idx1 tags

2016-10-17 Thread Michael Niedermayer
On Mon, Oct 17, 2016 at 06:29:42PM +0200, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch fixes the file mentioned in github pull request 197, 
> it can be played with MPlayer, vlc, xine and totem.
> 
> Please comment, Carl Eugen

>  avidec.c |2 ++
>  1 file changed, 2 insertions(+)
> 0b5d5e672b7ec13a8c8aa1597ceb21b16b5e61b8  
> 0001-lavf-avidec-Do-not-fail-for-very-large-idx1-tags.patch
> From 646a765749e08bb48f74488ad9f539cbc736673c Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos 
> Date: Mon, 17 Oct 2016 18:25:48 +0200
> Subject: [PATCH] lavf/avidec: Do not fail for very large idx1 tags.
> 
> Fixes demuxing the sample file from github pull request 197,
> the size of its idx1 tag is 6171936 bytes, followed by a JUNK
> tag of 9505704 bytes.
> ---
>  libavformat/avidec.c |2 ++
>  1 file changed, 2 insertions(+)

LGTM

thx

[...]

-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus


signature.asc
Description: Digital signature
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel