]> granicus.if.org Git - curl/commit
ntlm: fix memory leak
authorGisle Vanem <gvanem@broadpark.no>
Fri, 8 Feb 2013 14:50:23 +0000 (15:50 +0100)
committerDaniel Stenberg <daniel@haxx.se>
Fri, 8 Feb 2013 14:51:27 +0000 (15:51 +0100)
commit463082bea42d8bea751303da340218a18fb67e85
treeda47b95c17444f87c5d642904d9a3bc728769b98
parent72688317adcedb9508fd2189e6c6d3945e06a004
ntlm: fix memory leak

Running tests\libtest\libntlmconnect.exe reveals a 1 byte (!) leak in
./lib/curl_ntlm_msgs.c:

perl ..\memanalyze.pl c:memdebug.curl
Leak detected: memory still allocated: 1 bytes
At 9771e8, there's 1 bytes.
allocated by curl_ntlm_msgs.c:399

Snippet from curl_ntlm_msgs.c:
   /* setup ntlm identity's domain and length */
   dup_domain.tchar_ptr = malloc(sizeof(TCHAR) * (domlen + 1));

(my domlen == 0).

'dup_domain.tbyte_ptr' looks to be freed in Curl_ntlm_sspi_cleanup() via
'ntlm->identity.Domain'. But I see no freeing of 'dup_domain.tchar_ptr'.
lib/curl_ntlm_msgs.c