[FFmpeg-devel] [PATCH] avformat/http: fix memleak
mypopy at gmail.com
mypopy at gmail.com
Thu Sep 24 06:13:49 EEST 2020
On Thu, Sep 24, 2020 at 2:10 AM Zhao Zhili <quinkblack at foxmail.com> wrote:
>
> Ping for review.
>
> > On Aug 24, 2020, at 12:58 AM, Zhao Zhili <quinkblack at foxmail.com> wrote:
> >
> > ---
> > libavformat/http.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/libavformat/http.c b/libavformat/http.c
> > index 6c39da1a8b..3d25d652d3 100644
> > --- a/libavformat/http.c
> > +++ b/libavformat/http.c
> > @@ -577,7 +577,7 @@ static int http_open(URLContext *h, const char *uri, int flags,
> > "No trailing CRLF found in HTTP header. Adding it.\n");
> > ret = av_reallocp(&s->headers, len + 3);
> > if (ret < 0)
> > - return ret;
> > + goto bail_out;
> > s->headers[len] = '\r';
> > s->headers[len + 1] = '\n';
> > s->headers[len + 2] = '\0';
> > @@ -588,6 +588,7 @@ static int http_open(URLContext *h, const char *uri, int flags,
> > return http_listen(h, uri, flags, options);
> > }
> > ret = http_open_cnx(h, options);
> > +bail_out:
> > if (ret < 0)
> > av_dict_free(&s->chained_options);
> > return ret;
> > --
> > 2.25.1
> >
>
LGTM
More information about the ffmpeg-devel
mailing list