[FFmpeg-devel] [PATCH] avformat/sapdec: check return value of avcodec_parameters_copy()

[Marth64]

Ping if anyone is familiar with this format, thanks

On Sat, Oct 12, 2024 at 12:32 Marth64 <marth64 at proxyid.net> wrote:

> Written in the dominant style of the surrounding code block.
>
> Signed-off-by: Marth64 <marth64 at proxyid.net>
> ---
>  libavformat/sapdec.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/libavformat/sapdec.c b/libavformat/sapdec.c
> index d5b5d71c02..22db3803c9 100644
> --- a/libavformat/sapdec.c
> +++ b/libavformat/sapdec.c
> @@ -179,7 +179,9 @@ static int sap_read_header(AVFormatContext *s)
>              goto fail;
>          }
>          st->id = i;
> -        avcodec_parameters_copy(st->codecpar,
> sap->sdp_ctx->streams[i]->codecpar);
> +        ret = avcodec_parameters_copy(st->codecpar,
> sap->sdp_ctx->streams[i]->codecpar);
> +        if (ret < 0)
> +            goto fail;
>          st->time_base = sap->sdp_ctx->streams[i]->time_base;
>      }
>
> --
> 2.34.1
>
>