[FFmpeg-devel] [PATCH 07/21] avfilter/af_afir: Fix leak of AVFilterChannelLayout in case of error

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Tue Aug 18 22:15:07 EEST 2020


Andreas Rheinhardt:
> If an error happens between the allocation of an AVFilterChannelLayout
> and its usage (which involves attaching said object to a more permanent
> object), the channel layout array leaks. This can simply be fixed by
> making sure that nothing is between the allocation and the
> aforementioned usage.
> 
> Fixes Coverity issue #1250334.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavfilter/af_afir.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/libavfilter/af_afir.c b/libavfilter/af_afir.c
> index 5ba880f10b..6cbc7a00a1 100644
> --- a/libavfilter/af_afir.c
> +++ b/libavfilter/af_afir.c
> @@ -742,14 +742,14 @@ static int query_formats(AVFilterContext *ctx)
>      } else {
>          AVFilterChannelLayouts *mono = NULL;
>  
> -        ret = ff_add_channel_layout(&mono, AV_CH_LAYOUT_MONO);
> -        if (ret)
> -            return ret;
> -
>          if ((ret = ff_channel_layouts_ref(layouts, &ctx->inputs[0]->out_channel_layouts)) < 0)
>              return ret;
>          if ((ret = ff_channel_layouts_ref(layouts, &ctx->outputs[0]->in_channel_layouts)) < 0)
>              return ret;
> +
> +        ret = ff_add_channel_layout(&mono, AV_CH_LAYOUT_MONO);
> +        if (ret)
> +            return ret;
>          for (int i = 1; i < ctx->nb_inputs; i++) {
>              if ((ret = ff_channel_layouts_ref(mono, &ctx->inputs[i]->out_channel_layouts)) < 0)
>                  return ret;
> 
Ping for this and the other outstanding patches of this patchset.

- Andreas


More information about the ffmpeg-devel mailing list