[FFmpeg-devel] [PATCH] avfilter/gblur_vulkan: fix memory leak

Lynne dev at lynne.ee
Wed Nov 17 05:28:08 EET 2021


17 Nov 2021, 04:04 by jianhua.wu at intel.com:

> Whether failed or not, the block of codes labeled with fail should
> be always run to ensure the av_free(kernel_def) is executed.
>
> Signed-off-by: Wu Jianhua <jianhua.wu at intel.com>
> ---
>  libavfilter/vf_gblur_vulkan.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/libavfilter/vf_gblur_vulkan.c b/libavfilter/vf_gblur_vulkan.c
> index b9e2c4fbc7..d4f391b25d 100644
> --- a/libavfilter/vf_gblur_vulkan.c
> +++ b/libavfilter/vf_gblur_vulkan.c
> @@ -296,8 +296,6 @@ static av_cold int init_filter(AVFilterContext *ctx, AVFrame *in)
>  
>  s->initialized = 1;
>  
> -    return 0;
> -
>  fail:
>  av_free(kernel_def);
>  return err;
>

Pushed, thanks.


More information about the ffmpeg-devel mailing list