[FFmpeg-cvslog] fftools/ffmpeg: make attachment filenames dynamically allocated

Anton Khirnov git at videolan.org
Sat Jan 20 11:34:09 EET 2024


ffmpeg | branch: master | Anton Khirnov <anton at khirnov.net> | Mon Mar 14 12:30:31 2022 +0100| [8aed3911fc454e79697e183660bf30d31334a64b] | committer: Anton Khirnov

fftools/ffmpeg: make attachment filenames dynamically allocated

Do not store the supplied arg pointer directly. While that is valid for
now, it will become ephemeral in the future commits.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=8aed3911fc454e79697e183660bf30d31334a64b
---

 fftools/ffmpeg_opt.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/fftools/ffmpeg_opt.c b/fftools/ffmpeg_opt.c
index ea995f2b5f..563f443bd8 100644
--- a/fftools/ffmpeg_opt.c
+++ b/fftools/ffmpeg_opt.c
@@ -124,6 +124,9 @@ static void uninit_options(OptionsContext *o)
 #if FFMPEG_OPT_MAP_CHANNEL
     av_freep(&o->audio_channel_maps);
 #endif
+
+    for (i = 0; i < o->nb_attachments; i++)
+        av_freep(&o->attachments[i]);
     av_freep(&o->attachments);
 
     av_dict_free(&o->streamid);
@@ -494,7 +497,10 @@ static int opt_attach(void *optctx, const char *opt, const char *arg)
     if (ret < 0)
         return ret;
 
-    o->attachments[o->nb_attachments - 1] = arg;
+    o->attachments[o->nb_attachments - 1] = av_strdup(arg);
+    if (!o->attachments[o->nb_attachments - 1])
+        return AVERROR(ENOMEM);
+
     return 0;
 }
 



More information about the ffmpeg-cvslog mailing list