Hi,

On Thu, Oct 13, 2011 at 6:39 AM, shahriman AMS <[email protected]> wrote:
> ----- Original Message -----
>> From: Kostya Shishkov <[email protected]>
>> To: libav development <[email protected]>
>> Cc:
>> Sent: Thursday, October 13, 2011 2:12 PM
>> Subject: [libav-devel] [PATCH] vc1: explicitly zero interlaced mode coding 
>> variables for progressive mode
>>
>> Both v->fcm and v->field_mode are used in common code, now they won't
>> be
>> reset for progressive frame after interlaced one causing writing past the
>> frame end for example.
>> ---
>> libavcodec/vc1.c |    5 +++--
>> 1 files changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavcodec/vc1.c b/libavcodec/vc1.c
>> index 9d36811..cb228e6 100644
>> --- a/libavcodec/vc1.c
>> +++ b/libavcodec/vc1.c
>> @@ -832,17 +832,18 @@ int vc1_parse_frame_header_adv(VC1Context *v,
>> GetBitContext* gb)
>>              goto parse_common_info;
>>      }
>>
>> +    v->field_mode = 0;
>>      if (v->interlace) {
>>          v->fcm = decode012(gb);
>>          if (v->fcm) {
>>              if (v->fcm == 2)
>>                  v->field_mode = 1;
>> -            else
>> -                v->field_mode = 0;
>>              if (!v->warn_interlaced++)
>>                  av_log(v->s.avctx, AV_LOG_ERROR,
>>                         "Interlaced frames/fields support is
>> incomplete\n");
>>          }
>> +    } else {
>> +        v->fcm = 0;
>>      }
>>
>>      if (v->field_mode) {
>> --
>> 1.7.0.4
>>
>
> certainly OK.

Queued.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to