[FFmpeg-cvslog] r12434 - trunk/libavcodec/mpeg12.c

[Måns Rullgård]

bcoudurier <subversion at mplayerhq.hu> writes:

> Author: bcoudurier
> Date: Thu Mar 13 19:26:53 2008
> New Revision: 12434
>
> Log:
> do not fail badly if aspect ratio is 0/forbidden, fix issue 369
>
> Modified:
>    trunk/libavcodec/mpeg12.c
>
> Modified: trunk/libavcodec/mpeg12.c
> ==============================================================================
> --- trunk/libavcodec/mpeg12.c	(original)
> +++ trunk/libavcodec/mpeg12.c	Thu Mar 13 19:26:53 2008
> @@ -1970,8 +1970,6 @@ static int mpeg1_decode_sequence(AVCodec
>          (width % 2) != 0 || (height % 2) != 0)
>          return -1;
>      s->aspect_ratio_info= get_bits(&s->gb, 4);
> -    if (s->aspect_ratio_info == 0)
> -        return -1;

IMHO a warning should be printed in this case.  Users have right to
know (or deserve to be annoyed).

-- 
M?ns Rullg?rd
mans at mansr.com