[FFmpeg-devel] [PATCH] IFF: Add error checking to byterun1 decoder

Sebastian Vater cdgs.basty
Wed May 26 23:09:58 CEST 2010


Ronald S. Bultje a ?crit :
> Hi,
>
> On Wed, May 26, 2010 at 3:20 PM, Sebastian Vater
> <cdgs.basty at googlemail.com> wrote:
>   
>> -    for (x = 0; x < dst_size && buf < buf_end;) {
>> -        unsigned length;
>> -        const int8_t value = *buf++;
>> -        if (value >= 0) {
>> -            length = value + 1;
>> -            memcpy(dst + x, buf, FFMIN3(length, dst_size - x, buf_end - buf));
>> -            buf += length;
>> -        } else if (value > -128) {
>> -            length = -value + 1;
>> -            memset(dst + x, *buf++, FFMIN(length, dst_size - x));
>> -        } else { // noop
>> -            continue;
>> -        }
>> -        x += length;
>> +    if (buf < buf_end) {
>> +        do {
>> +            const int8_t value = *buf++;
>>     
>
> Your patch now mixes indentation and functional changes.
>   

Fixed. Reindent patch will follow...

-- 

Best regards,
                   :-) Basty/CDGS (-:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: iff-byterun1-error.patch
Type: text/x-patch
Size: 2505 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100526/adc59dd0/attachment.bin>



More information about the ffmpeg-devel mailing list