----- 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.

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

Reply via email to