[FFmpeg-devel] [PATCH] avformat/mvdec: propagate the errors returned by parse_audio_var()

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Thu Sep 16 22:39:53 EEST 2021


James Almer:
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  libavformat/mvdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/mvdec.c b/libavformat/mvdec.c
> index 7573087c7c..0b4aa1f18f 100644
> --- a/libavformat/mvdec.c
> +++ b/libavformat/mvdec.c
> @@ -387,7 +387,7 @@ static int mv_read_header(AVFormatContext *avctx)
>              if (!ast)
>                  return AVERROR(ENOMEM);
>              ast->codecpar->codec_type = AVMEDIA_TYPE_AUDIO;
> -            if ((read_table(avctx, ast, parse_audio_var)) < 0)
> +            if ((ret = read_table(avctx, ast, parse_audio_var)) < 0)
>                  return ret;
>              if (mv->acompression == 100 &&
>                  mv->aformat == AUDIO_FORMAT_SIGNED &&
> 

The title is misleading: it should be something like "Don't signal
success upon error". Just reading your title would indicate that we are
just making error codes up instead of forwarding them.

- Andreas


More information about the ffmpeg-devel mailing list