[FFmpeg-devel] [PATCH 4/6] avformat/au: check return value of au_read_annotation()

Marton Balint cus at passwd.hu
Tue Jun 23 00:15:12 EEST 2020



On Mon, 22 Jun 2020, lance.lmwang at gmail.com wrote:

> From: Limin Wang <lance.lmwang at gmail.com>
>
> Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> ---
> libavformat/au.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/libavformat/au.c b/libavformat/au.c
> index ff9176a..b6df63e 100644
> --- a/libavformat/au.c
> +++ b/libavformat/au.c
> @@ -145,6 +145,7 @@ static int au_read_header(AVFormatContext *s)
>     int bps, ba = 0;
>     enum AVCodecID codec;
>     AVStream *st;
> +    int ret;
>
>     tag = avio_rl32(pb);
>     if (tag != MKTAG('.', 's', 'n', 'd'))
> @@ -163,7 +164,8 @@ static int au_read_header(AVFormatContext *s)
>
>     if (size > 24) {
>         /* parse annotation field to get metadata */
> -        au_read_annotation(s, size - 24);
> +        if (ret = au_read_annotation(s, size - 24) < 0)

Use two lines for the assignment and the check, otherwise sooner or later 
you will make an operator precedence mistake.

Regards,
Marton

> +            return ret;
>     }
>
>     codec = ff_codec_get_id(codec_au_tags, id);
> -- 
> 1.8.3.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list