[FFmpeg-cvslog] fftools/ffmpeg_opt: Fix leak on error
Andreas Rheinhardt
git at videolan.org
Sun Jul 23 18:55:00 EEST 2023
ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Sat Jul 22 20:34:18 2023 +0200| [2654347d7adba61ab8a8cce470d657958f6f00f9] | committer: Andreas Rheinhardt
fftools/ffmpeg_opt: Fix leak on error
Fixes Coverity issue #1539097.
Reviewed-by: Anton Khirnov <anton at khirnov.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=2654347d7adba61ab8a8cce470d657958f6f00f9
---
fftools/ffmpeg_opt.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
index 700db706a1..f7606ae6f6 100644
--- a/fftools/ffmpeg_opt.c
+++ b/fftools/ffmpeg_opt.c
@@ -497,7 +497,7 @@ static int opt_map_channel(void *optctx, const char *opt, const char *arg)
ret = GROW_ARRAY(o->audio_channel_maps, o->nb_audio_channel_maps);
if (ret < 0)
- return ret;
+ goto end;
m = &o->audio_channel_maps[o->nb_audio_channel_maps - 1];
@@ -559,11 +559,13 @@ static int opt_map_channel(void *optctx, const char *opt, const char *arg)
}
}
+ ret = 0;
+end:
av_free(mapchan);
- return 0;
+ return ret;
fail:
- av_free(mapchan);
- return AVERROR(EINVAL);
+ ret = AVERROR(EINVAL);
+ goto end;
}
#endif
More information about the ffmpeg-cvslog
mailing list