[FFmpeg-cvslog] avformat/http: fix memleak

Zhao Zhili git at videolan.org
Fri Sep 25 17:37:36 EEST 2020


ffmpeg | branch: master | Zhao Zhili <quinkblack at foxmail.com> | Mon Aug 24 00:58:09 2020 +0800| [90351b5f11079a86230f131d30df173d4c737676] | committer: Michael Niedermayer

avformat/http: fix memleak

Reviewed-by: "mypopy at gmail.com" <mypopy at gmail.com>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

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

 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;



More information about the ffmpeg-cvslog mailing list