[libav-devel] [PATCH 5/5] cinepak: remove redundant coordinate checks

[Kostya Shishkov]

On Fri, Nov 18, 2011 at 11:34:52AM +0200, Martin Storsjö wrote:
> From: Michael Niedermayer <michaelni at gmx.at>
> 
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  libavcodec/cinepak.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/cinepak.c b/libavcodec/cinepak.c
> index a858d6e..8ae1bb3 100644
> --- a/libavcodec/cinepak.c
> +++ b/libavcodec/cinepak.c
> @@ -269,8 +269,8 @@ static int cinepak_decode_strip (CinepakContext *s,
>      int      chunk_id, chunk_size;
>  
>      /* coordinate sanity checks */
> -    if (strip->x1 >= s->width  || strip->x2 > s->width  ||
> -        strip->y1 >= s->height || strip->y2 > s->height ||
> +    if (strip->x2 > s->width   ||
> +        strip->y2 > s->height  ||
>          strip->x1 >= strip->x2 || strip->y1 >= strip->y2)
>          return -1;
>  
> -- 

probably OK