Commit 13468346 authored by Rob Shearman's avatar Rob Shearman Committed by Alexandre Julliard

secur32: Fix memory leaks in ntlm_InitializeSecurityContextW.

parent 693433a3
...@@ -549,6 +549,7 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW( ...@@ -549,6 +549,7 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW(
goto isc_end; goto isc_end;
} }
username = ntlm_GetUsernameArg(ui->wkui1_username, -1); username = ntlm_GetUsernameArg(ui->wkui1_username, -1);
NetApiBufferFree(ui);
TRACE("using cached credentials\n"); TRACE("using cached credentials\n");
...@@ -820,6 +821,7 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW( ...@@ -820,6 +821,7 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW(
ret = SEC_E_BUFFER_TOO_SMALL; ret = SEC_E_BUFFER_TOO_SMALL;
if ((phContext == NULL) && (pInput == NULL)) if ((phContext == NULL) && (pInput == NULL))
{ {
HeapFree(GetProcessHeap(), 0, helper->session_key);
cleanup_helper(helper); cleanup_helper(helper);
phNewContext->dwUpper = 0; phNewContext->dwUpper = 0;
phNewContext->dwLower = 0; phNewContext->dwLower = 0;
...@@ -838,6 +840,7 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW( ...@@ -838,6 +840,7 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW(
ret = SEC_E_BUFFER_TOO_SMALL; ret = SEC_E_BUFFER_TOO_SMALL;
if ((phContext == NULL) && (pInput == NULL)) if ((phContext == NULL) && (pInput == NULL))
{ {
HeapFree(GetProcessHeap(), 0, helper->session_key);
cleanup_helper(helper); cleanup_helper(helper);
phNewContext->dwUpper = 0; phNewContext->dwUpper = 0;
phNewContext->dwLower = 0; phNewContext->dwLower = 0;
...@@ -851,6 +854,7 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW( ...@@ -851,6 +854,7 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW(
ret = SEC_E_INTERNAL_ERROR; ret = SEC_E_INTERNAL_ERROR;
if ((phContext == NULL) && (pInput == NULL)) if ((phContext == NULL) && (pInput == NULL))
{ {
HeapFree(GetProcessHeap(), 0, helper->session_key);
cleanup_helper(helper); cleanup_helper(helper);
phNewContext->dwUpper = 0; phNewContext->dwUpper = 0;
phNewContext->dwLower = 0; phNewContext->dwLower = 0;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment