[FFmpeg-devel] [PATCH 4/5] fftools/ffmpeg_filter: do not leak graph_desc on fg_create() failure
Anton Khirnov
anton at khirnov.net
Mon Oct 7 13:50:15 EEST 2024
fg_create() is passed a malloced string and assumed to always tak
ownership of it.
---
fftools/ffmpeg_filter.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/fftools/ffmpeg_filter.c b/fftools/ffmpeg_filter.c
index faab003e71..4d444c161f 100644
--- a/fftools/ffmpeg_filter.c
+++ b/fftools/ffmpeg_filter.c
@@ -1058,8 +1058,10 @@ int fg_create(FilterGraph **pfg, char *graph_desc, Scheduler *sch)
int ret = 0;
fgp = av_mallocz(sizeof(*fgp));
- if (!fgp)
+ if (!fgp) {
+ av_freep(&graph_desc);
return AVERROR(ENOMEM);
+ }
fg = &fgp->fg;
if (pfg) {
@@ -1068,6 +1070,7 @@ int fg_create(FilterGraph **pfg, char *graph_desc, Scheduler *sch)
} else {
ret = av_dynarray_add_nofree(&filtergraphs, &nb_filtergraphs, fgp);
if (ret < 0) {
+ av_freep(&graph_desc);
av_freep(&fgp);
return ret;
}
--
2.43.0
More information about the ffmpeg-devel
mailing list