[FFmpeg-devel] [PATCH] avcodec/h264_metadata_bsf: Don't use unitialized value

Mark Thompson sw at jkqxz.net
Fri Jan 22 15:36:30 EET 2021


On 22/01/2021 11:43, Andreas Rheinhardt wrote:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>   libavcodec/h264_metadata_bsf.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/h264_metadata_bsf.c b/libavcodec/h264_metadata_bsf.c
> index 30c0aa80d9..6673a32601 100644
> --- a/libavcodec/h264_metadata_bsf.c
> +++ b/libavcodec/h264_metadata_bsf.c
> @@ -528,7 +528,7 @@ static int h264_metadata_update_fragment(AVBSFContext *bsf, AVPacket *pkt,
>       if (pkt)
>           ctx->done_first_au = 1;
>   
> -    return err;
> +    return 0;
>   }
>   
>   static const CBSBSFType h264_metadata_type = {
> 

LGTM, thanks.  (Somewhat disappointed that gcc didn't catch that for me, though.)

- Mark


More information about the ffmpeg-devel mailing list