[FFmpeg-devel] [PATCH 4/7] avformat/icecast: Use AV_DICT_DONT_STRDUP_VAL to save an av_strdup

Michael Niedermayer michael at niedermayer.cc
Sat Dec 21 16:03:51 EET 2019


On Fri, Dec 20, 2019 at 09:32:34AM +0100, Marvin Scholz wrote:
> On 10 Nov 2019, at 5:07, Andreas Rheinhardt wrote:
> 
> >This will probably also fix CID 1452559, a false positive where Coverity
> >claims a double-free occurs, because it thinks that av_dict_set() frees
> >its key and value arguments even when the AV_DICT_DONT_STRDUP_* flags
> >aren't used.
> >
> >Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
> >---
> > libavformat/icecast.c | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> >diff --git a/libavformat/icecast.c b/libavformat/icecast.c
> >index 052cd37f3e..7d8f92fe73 100644
> >--- a/libavformat/icecast.c
> >+++ b/libavformat/icecast.c
> >@@ -114,7 +114,7 @@ static int icecast_open(URLContext *h, const char
> >*uri, int flags)
> >     // Set options
> >     av_dict_set(&opt_dict, "method", s->legacy_icecast ? "SOURCE" :
> >"PUT", 0);
> >     av_dict_set(&opt_dict, "auth_type", "basic", 0);
> >-    av_dict_set(&opt_dict, "headers", headers, 0);
> >+    av_dict_set(&opt_dict, "headers", headers, AV_DICT_DONT_STRDUP_VAL);
> >     av_dict_set(&opt_dict, "chunked_post", "0", 0);
> >     av_dict_set(&opt_dict, "send_expect_100", s->legacy_icecast ? "-1" :
> >"1", 0);
> >     if (NOT_EMPTY(s->content_type))
> >@@ -170,7 +170,6 @@ static int icecast_open(URLContext *h, const char
> >*uri, int flags)
> >
> > cleanup:
> >     av_freep(&user);
> >-    av_freep(&headers);
> >     av_dict_free(&opt_dict);
> >
> >     return ret;
> >-- 
> >2.20.1
> >
> >
> 
> Thanks for the patch, LGTM.

will apply 

thx

> (I can’t push it though, so someone else needs to approve too and push this)
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20191221/bed55c3b/attachment.sig>


More information about the ffmpeg-devel mailing list