mirror/transmission
1
0
Fork 0
mirror of https://github.com/transmission/transmission synced 2024-12-23 08:13:27 +00:00
Code Issues Releases Wiki Activity

Fix some leaks in tr-remote reported by GCC's sanitizer

Browse source
This commit is contained in:
Mike Gelfand 2018-02-03 12:39:24 +03:00
parent 30a6a3244a
commit ee259c617f
1 changed files with 19 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
daemon/remote.c
View file

@ -2055,13 +2055,28 @@ static CURL* tr_curl_easy_init(struct evbuffer* writebuf)
{
char* h = tr_strdup_printf("%s: %s", TR_RPC_SESSION_ID_HEADER, sessionId);
struct curl_slist* custom_headers = curl_slist_append(NULL, h);
tr_free(h);
curl_easy_setopt(curl, CURLOPT_HTTPHEADER, custom_headers);
/* fixme: leaks */
curl_easy_setopt(curl, CURLOPT_PRIVATE, custom_headers);
}
return curl;
}
static void tr_curl_easy_cleanup(CURL* curl)
{
struct curl_slist* custom_headers = NULL;
curl_easy_getinfo(curl, CURLINFO_PRIVATE, &custom_headers);
curl_easy_cleanup(curl);
if (custom_headers != NULL)
{
curl_slist_free_all(custom_headers);
}
}
static int flush(char const* rpcurl, tr_variant** benc)
{
CURLcode res;
@ -2101,7 +2116,7 @@ static int flush(char const* rpcurl, tr_variant** benc)
/* Session id failed. Our curl header func has already
* pulled the new session id from this response's headers,
* build a new CURL* and try again */
curl_easy_cleanup(curl);
tr_curl_easy_cleanup(curl);
curl = NULL;
status |= flush(rpcurl, benc);
benc = NULL;
@ -2121,12 +2136,13 @@ static int flush(char const* rpcurl, tr_variant** benc)
if (curl != NULL)
{
curl_easy_cleanup(curl);
tr_curl_easy_cleanup(curl);
}
if (benc != NULL)
{
tr_variantFree(*benc);
tr_free(*benc);
*benc = NULL;
}