[FFmpeg-cvslog] indeo3: fix motion vector validation

[Reimar Döffinger]

On 25 Jan 2012, at 21:27, git at videolan.org (Hendrik Leppkes) wrote:

> ffmpeg | branch: master | Hendrik Leppkes <h.leppkes at gmail.com> | Wed Jan 25 17:37:26 2012 +0100| [6071644287d2a7471d906b688cb7253a5ceaaa8a] | committer: Michael Niedermayer
> 
> indeo3: fix motion vector validation
> 
> The index of the motion vector has to be checked before being
> multiplied by 2 for the array index.
> 
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> 
>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=6071644287d2a7471d906b688cb7253a5ceaaa8a
> ---
> 
> libavcodec/indeo3.c |    5 ++---
> 1 files changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/libavcodec/indeo3.c b/libavcodec/indeo3.c
> index ce84d72..fc38f5e 100644
> --- a/libavcodec/indeo3.c
> +++ b/libavcodec/indeo3.c
> @@ -772,13 +772,12 @@ static int parse_bintree(Indeo3DecodeContext *ctx, AVCodecContext *avctx,
>                 /* get motion vector index and setup the pointer to the mv set */
>                 if (!ctx->need_resync)
>                     ctx->next_cell_data = &ctx->gb.buffer[(get_bits_count(&ctx->gb) + 7) >> 3];
> -                if(ctx->mc_vectors)
> -                    mv_idx = *(ctx->next_cell_data++) << 1;
> +                mv_idx = *(ctx->next_cell_data++);

The commit message doesn't quite explain why the if was removed though.