[FFmpeg-devel] [PATCH] avfilter/vf_dedot: Fix leak of AVFrame if making it writable fails

Paul B Mahol onemda at gmail.com
Sun Feb 9 23:56:03 EET 2020


lgtm

On 2/9/20, Andreas Rheinhardt <andreas.rheinhardt at gmail.com> wrote:
> Even in this scenario, the frame still contains references to data that
> won't be freed if the frame isn't unreferenced. And the AVFrame itself
> will leak, too.
>
> Fixes Coverity issue #1441422.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> ---
>  libavfilter/vf_dedot.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/libavfilter/vf_dedot.c b/libavfilter/vf_dedot.c
> index 1fb7bfe866..993ac8f74f 100644
> --- a/libavfilter/vf_dedot.c
> +++ b/libavfilter/vf_dedot.c
> @@ -313,7 +313,8 @@ static int activate(AVFilterContext *ctx)
>                                                 FFMIN(s->planeheight[2],
>
> ff_filter_get_nb_threads(ctx)));
>                      }
> -                }
> +                } else
> +                    av_frame_free(&out);
>              } else if (!out) {
>                  ret = AVERROR(ENOMEM);
>              }
> --
> 2.20.1
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list