On Thu, Nov 16, 2017 at 16:28:22 -0800, Jacob Trimble wrote:
> -    if (mov_read_default(mov, s->pb, (MOVAtom){ AV_RL32("root"), INT64_MAX 
> }) < 0 ||
> -        avio_feof(s->pb))
> +    if ((ret = mov_read_default(mov, s->pb, (MOVAtom){ AV_RL32("root"), 
> INT64_MAX })) < 0)
> +        return ret;
> +    if (avio_feof(s->pb))
>          return AVERROR_EOF;

The recommendation is to use:

    ret = mov_read_default(mov, s->pb, (MOVAtom){ AV_RL32("root"), INT64_MAX });
    if (ret < 0)
        return ret;

in new code.

Moritz
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to