On Wed, Jan 18, 2012 at 1:36 AM, Dustin Brody <li...@parsoma.net> wrote:
> ---
>  libavformat/avidec.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavformat/avidec.c b/libavformat/avidec.c
> index b45224f..3f0d1a3 100644
> --- a/libavformat/avidec.c
> +++ b/libavformat/avidec.c
> @@ -666,7 +666,7 @@ static int avi_read_header(AVFormatContext *s, 
> AVFormatParameters *ap)
>         case MKTAG('i', 'n', 'd', 'x'):
>             i= avio_tell(pb);
>             if(pb->seekable && !(s->flags & AVFMT_FLAG_IGNIDX) &&
> -               read_braindead_odml_indx(s, 0) < 0 && s->error_recognition >= 
> FF_ER_EXPLODE){
> +               read_braindead_odml_indx(s, 0) < 0 && (s->error_recognition & 
> AV_EF_EXPLODE)){

> 80 Chars , and we don't need the braces.

>                 goto fail;
>             }
>             avio_seek(pb, i+size, SEEK_SET);
> @@ -705,7 +705,7 @@ static int avi_read_header(AVFormatContext *s, 
> AVFormatParameters *ap)
>             if(size > 1000000){
>                 av_log(s, AV_LOG_ERROR, "Something went wrong during header 
> parsing, "
>                                         "I will ignore it and try to continue 
> anyway.\n");
> -                if (s->error_recognition >= FF_ER_EXPLODE) goto fail;
> +                if (s->error_recognition & AV_EF_EXPLODE) goto fail;

if (condition)
    goto fail;

>                 avi->movi_list = avio_tell(pb) - 4;
>                 avi->movi_end  = avio_size(pb);
>                 goto end_of_header;
> --
> 1.7.2.5
>
> _______________________________________________
> libav-devel mailing list
> libav-devel@libav.org
> https://lists.libav.org/mailman/listinfo/libav-devel

I don't know if the patch is OK, someone else have to review it first.


-- 
Thanks
Aneesh Dogra (lionaneesh)
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to