[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