From: Steve Holme Date: Sun, 6 Apr 2014 13:10:18 +0000 (+0100) Subject: sasl: Corrected missing free of decoded challenge message from 607883f13c X-Git-Tag: upstream/7.37.1~348 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f700eb7d9a7b1947b8466b2a3c325d5fbaaee2a2;p=platform%2Fupstream%2Fcurl.git sasl: Corrected missing free of decoded challenge message from 607883f13c --- diff --git a/lib/curl_sasl_sspi.c b/lib/curl_sasl_sspi.c index d17e08c..3b2277d 100644 --- a/lib/curl_sasl_sspi.c +++ b/lib/curl_sasl_sspi.c @@ -95,14 +95,18 @@ CURLcode Curl_sasl_create_digest_md5_message(struct SessionHandle *data, /* Ensure we have some login credientials as DigestSSP cannot use the current Windows user like NTLMSSP can */ - if(!userp || !*userp) + if(!userp || !*userp) { + Curl_safefree(chlg); return CURLE_LOGIN_DENIED; + } /* Query the security package for DigestSSP */ status = s_pSecFn->QuerySecurityPackageInfo((TCHAR *) TEXT("WDigest"), &SecurityPackage); - if(status != SEC_E_OK) + if(status != SEC_E_OK) { + Curl_safefree(chlg); return CURLE_NOT_BUILT_IN; + } /* Calculate our SPN */ spn = aprintf("%s/%s", service, data->easy_conn->host); @@ -113,6 +117,7 @@ CURLcode Curl_sasl_create_digest_md5_message(struct SessionHandle *data, result = Curl_create_sspi_identity(userp, passwdp, &identity); if(result) { Curl_safefree(spn); + Curl_safefree(chlg); return result; } @@ -127,6 +132,7 @@ CURLcode Curl_sasl_create_digest_md5_message(struct SessionHandle *data, if(status != SEC_E_OK) { Curl_sspi_free_identity(&identity); Curl_safefree(spn); + Curl_safefree(chlg); return CURLE_OUT_OF_MEMORY; } @@ -164,6 +170,7 @@ CURLcode Curl_sasl_create_digest_md5_message(struct SessionHandle *data, s_pSecFn->FreeCredentialsHandle(&handle); Curl_sspi_free_identity(&identity); Curl_safefree(spn); + Curl_safefree(chlg); return CURLE_RECV_ERROR; } @@ -182,6 +189,9 @@ CURLcode Curl_sasl_create_digest_md5_message(struct SessionHandle *data, /* Free the SPN */ Curl_safefree(spn); + /* Free the decoeded challenge message */ + Curl_safefree(chlg); + return result; }