diff mbox series

[FFmpeg-devel,v3,24/25] avfilter/vf_xmedian: Free inpads' names generically

Message ID PR3PR03MB6665C6468687963B53324FA48FC39@PR3PR03MB6665.eurprd03.prod.outlook.com
State Accepted
Commit 845579c6e2ee4c729a56c4daf819e5690d858570
Headers show
Series [FFmpeg-devel,v3,01/25] avfilter/internal: Uninline ff_insert_(in|out)pad() | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Andreas Rheinhardt Aug. 22, 2021, 12:47 a.m. UTC 
This affects only the xmedian filter, not tmedian.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavfilter/vf_xmedian.c | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)
diff mbox series

Patch

diff --git a/libavfilter/vf_xmedian.c b/libavfilter/vf_xmedian.c
index f900720012..f8e4e4adb2 100644
--- a/libavfilter/vf_xmedian.c
+++ b/libavfilter/vf_xmedian.c
@@ -118,10 +118,8 @@  static av_cold int init(AVFilterContext *ctx)
         if (!pad.name)
             return AVERROR(ENOMEM);
 
-        if ((ret = ff_append_inpad(ctx, &pad)) < 0) {
-            av_freep(&pad.name);
+        if ((ret = ff_append_inpad_free_name(ctx, &pad)) < 0)
             return ret;
-        }
     }
 
     return 0;
@@ -335,8 +333,6 @@  static av_cold void uninit(AVFilterContext *ctx)
 
     ff_framesync_uninit(&s->fs);
 
-    for (int i = 0; i < ctx->nb_inputs && !s->tmedian; i++)
-        av_freep(&ctx->input_pads[i].name);
     for (int i = 0; i < s->nb_frames && s->frames && s->tmedian; i++)
         av_frame_free(&s->frames[i]);
     av_freep(&s->frames);