[FFmpeg-cvslog] lavfi/afir: fix vpad.name leak
Jun Zhao
git at videolan.org
Sat May 30 14:03:43 EEST 2020
ffmpeg | branch: master | Jun Zhao <barryjzhao at tencent.com> | Mon May 25 08:48:12 2020 +0800| [ff8329a73054bc6055a69860507fe45386e94798] | committer: Jun Zhao
lavfi/afir: fix vpad.name leak
Fix vpad.name leak in error path, move the vpad related operation
only if enabled show IR frequency response.
Signed-off-by: Jun Zhao <barryjzhao at tencent.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=ff8329a73054bc6055a69860507fe45386e94798
---
libavfilter/af_afir.c | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/libavfilter/af_afir.c b/libavfilter/af_afir.c
index 7c7e8458d4..5ba880f10b 100644
--- a/libavfilter/af_afir.c
+++ b/libavfilter/af_afir.c
@@ -876,6 +876,12 @@ static av_cold int init(AVFilterContext *ctx)
if (!pad.name)
return AVERROR(ENOMEM);
+ ret = ff_insert_outpad(ctx, 0, &pad);
+ if (ret < 0) {
+ av_freep(&pad.name);
+ return ret;
+ }
+
if (s->response) {
vpad = (AVFilterPad){
.name = av_strdup("filter_response"),
@@ -884,15 +890,7 @@ static av_cold int init(AVFilterContext *ctx)
};
if (!vpad.name)
return AVERROR(ENOMEM);
- }
- ret = ff_insert_outpad(ctx, 0, &pad);
- if (ret < 0) {
- av_freep(&pad.name);
- return ret;
- }
-
- if (s->response) {
ret = ff_insert_outpad(ctx, 1, &vpad);
if (ret < 0) {
av_freep(&vpad.name);
More information about the ffmpeg-cvslog
mailing list