romenskiy2012/curl
1
0
Fork 0
mirror of https://github.com/curl/curl.git synced 2026-05-30 06:17:28 +03:00
Code Issues Projects Releases Packages Wiki Activity

c-hyper: fix another memory leak in Curl_http.

Browse source 
There is a `hyper_clientconn_free` call on the happy path, but not one
on the error path. This commit adds one.

Fixes the second memory leak reported by Valgrind in #10803.

Fixes #10803
Closes #11729
This commit is contained in:
Nicholas Nethercote 2023-08-25 18:40:31 +10:00 • committed by Daniel Stenberg
parent c61dd5fed2
commit e854166472
No known key found for this signature in database
GPG key ID: 5CC908FDB71E12C2
1 changed files with 4 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lib/c-hyper.c
View file

@ -1217,6 +1217,7 @@ CURLcode Curl_http(struct Curl_easy *data, bool *done)
}
hyper_clientconn_free(client);
client = NULL;
if((httpreq == HTTPREQ_GET) || (httpreq == HTTPREQ_HEAD)) {
/* HTTP GET/HEAD download */
@ -1245,6 +1246,9 @@ error:
if(handshake)
hyper_task_free(handshake);
if(client)
hyper_clientconn_free(client);
if(req)
hyper_request_free(req);