[FFmpeg-devel] [PATCH] wav: fix skip size at end of ds64 chunk

[Michael Niedermayer]

On Fri, Jun 03, 2011 at 01:41:22PM +0200, Tomas Härdin wrote:
> Philip de Nier skrev 2011-06-02 18:04:
>> From 955a98a4a54458e3beec08534a4203c878b18876 Mon Sep 17 00:00:00 2001
>> From: Philip de Nier <philip.denier at rd.bbc.co.uk>
>> Date: Thu, 2 Jun 2011 16:46:23 +0100
>> Subject: [PATCH] wav: fix skip size at end of ds64 chunk
>>
>> ---
>>  libavformat/wav.c |    2 +-
>>  1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/libavformat/wav.c b/libavformat/wav.c
>> index b475d0c..34b9571 100644
>> --- a/libavformat/wav.c
>> +++ b/libavformat/wav.c
>> @@ -334,7 +334,7 @@ static int wav_read_header(AVFormatContext *s,
>>                     data_size, sample_count);
>>              return AVERROR_INVALIDDATA;
>>          }
>> -        avio_skip(pb, size - 16); /* skip rest of ds64 chunk */
>> +        avio_skip(pb, size - 24); /* skip rest of ds64 chunk */
>
> Looks correct - it reads 3*8 = 24 byte before the skip. I also  
> double-checked the RF64 spec.

locally applied

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I hate to see young programmers poisoned by the kind of thinking
Ulrich Drepper puts forward since it is simply too narrow -- Roman Shaposhnik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110603/004b4ded/attachment.asc>