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.rheinha...@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@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe". -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB No great genius has ever existed without some touch of madness. -- Aristotle
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
