Re: [FFmpeg-devel] [PATCH 3/3] http: fix memory leak in parse_cookie.
On Thu, Jan 11, 2018 at 03:15:41PM -0800, Richard Shaffer wrote: > On Thu, Jan 11, 2018 at 2:49 PM, Moritz Barsnick wrote: > > > On Thu, Jan 11, 2018 at 14:28:52 -0800, rshaf...@tunein.com wrote: > > > > > [PATCH 3/3] http: fix memory leak in parse_cookie.g > > > > > This is my first time contributing. > > > > Where are the other two of the three ("3/3") patches then? (Not > > important, but it's confusing on the mailing list.) > > > > I apologize for the 3/3 in the subject. Patches 1 and 2 deal with something > else, and I'll submit those for review later. > > > > > The problem and the solution seem pretty straightforward, but if I'm > > missing something, please do point me in the right direction. > > > > I believe the way you submitted this, this text would be part of your > > commit message. If you desire to type it into the email, please put it > > below the three dashes: > > > Good to know. If someone is willing to review the substance of the patch, > hopefully they'll remove my introduction before committing. will apply with the commit messages fixed thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB When you are offended at any man's fault, turn to yourself and study your own failings. Then you will forget your anger. -- Epictetus signature.asc Description: PGP signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 3/3] http: fix memory leak in parse_cookie.
On Thu, Jan 11, 2018 at 2:49 PM, Moritz Barsnick wrote: > On Thu, Jan 11, 2018 at 14:28:52 -0800, rshaf...@tunein.com wrote: > > > [PATCH 3/3] http: fix memory leak in parse_cookie.g > > > This is my first time contributing. > > Where are the other two of the three ("3/3") patches then? (Not > important, but it's confusing on the mailing list.) > I apologize for the 3/3 in the subject. Patches 1 and 2 deal with something else, and I'll submit those for review later. > > The problem and the solution seem pretty straightforward, but if I'm > missing something, please do point me in the right direction. > > I believe the way you submitted this, this text would be part of your > commit message. If you desire to type it into the email, please put it > below the three dashes: Good to know. If someone is willing to review the substance of the patch, hopefully they'll remove my introduction before committing. > --- > > libavformat/http.c | 1 + > > 1 file changed, 1 insertion(+) > > I can't comment on the actual change. > > Cheers, > Moritz > ___ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
Re: [FFmpeg-devel] [PATCH 3/3] http: fix memory leak in parse_cookie.
On Thu, Jan 11, 2018 at 14:28:52 -0800, rshaf...@tunein.com wrote: > [PATCH 3/3] http: fix memory leak in parse_cookie.g > This is my first time contributing. Where are the other two of the three ("3/3") patches then? (Not important, but it's confusing on the mailing list.) > The problem and the solution seem pretty straightforward, but if I'm missing > something, please do point me in the right direction. I believe the way you submitted this, this text would be part of your commit message. If you desire to type it into the email, please put it below the three dashes: > --- > libavformat/http.c | 1 + > 1 file changed, 1 insertion(+) I can't comment on the actual change. Cheers, Moritz ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH 3/3] http: fix memory leak in parse_cookie.
From: Richard Shaffer This is my first time contributing. The problem and the solution seem pretty straightforward, but if I'm missing something, please do point me in the right direction. Thanks, -Richard --- libavformat/http.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/http.c b/libavformat/http.c index 4806b1e59b..510b23375a 100644 --- a/libavformat/http.c +++ b/libavformat/http.c @@ -824,6 +824,7 @@ static int parse_cookie(HTTPContext *s, const char *p, AVDictionary **cookies) } } } +av_dict_free(&new_params); // duplicate the cookie name (dict will dupe the value) if (!(eql = strchr(p, '='))) return AVERROR(EINVAL); -- 2.14.3 (Apple Git-98) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel