mirror of
https://github.com/morgan9e/FreeRDP
synced 2026-04-15 00:44:19 +09:00
[winpr,sspi] fix missing return check
This commit is contained in:
@@ -391,8 +391,12 @@ static SECURITY_STATUS SEC_ENTRY ntlm_AcquireCredentialsHandleW(
|
|||||||
{
|
{
|
||||||
UINT32 identityFlags = sspi_GetAuthIdentityFlags(pAuthData);
|
UINT32 identityFlags = sspi_GetAuthIdentityFlags(pAuthData);
|
||||||
|
|
||||||
sspi_CopyAuthIdentity(&(credentials->identity),
|
if (sspi_CopyAuthIdentity(&(credentials->identity),
|
||||||
(const SEC_WINNT_AUTH_IDENTITY_INFO*)pAuthData);
|
(const SEC_WINNT_AUTH_IDENTITY_INFO*)pAuthData) < 0)
|
||||||
|
{
|
||||||
|
sspi_CredentialsFree(credentials);
|
||||||
|
return SEC_E_INVALID_PARAMETER;
|
||||||
|
}
|
||||||
|
|
||||||
if (identityFlags & SEC_WINNT_AUTH_IDENTITY_EXTENDED)
|
if (identityFlags & SEC_WINNT_AUTH_IDENTITY_EXTENDED)
|
||||||
settings = (((SEC_WINNT_AUTH_IDENTITY_WINPR*)pAuthData)->ntlmSettings);
|
settings = (((SEC_WINNT_AUTH_IDENTITY_WINPR*)pAuthData)->ntlmSettings);
|
||||||
@@ -1233,7 +1237,14 @@ static SECURITY_STATUS SEC_ENTRY ntlm_DecryptMessage(PCtxtHandle phContext, PSec
|
|||||||
/* Decrypt message using with RC4, result overwrites original buffer */
|
/* Decrypt message using with RC4, result overwrites original buffer */
|
||||||
|
|
||||||
if (context->confidentiality)
|
if (context->confidentiality)
|
||||||
winpr_RC4_Update(context->RecvRc4Seal, length, (BYTE*)data, (BYTE*)data_buffer->pvBuffer);
|
{
|
||||||
|
if (!winpr_RC4_Update(context->RecvRc4Seal, length, (BYTE*)data,
|
||||||
|
(BYTE*)data_buffer->pvBuffer))
|
||||||
|
{
|
||||||
|
free(data);
|
||||||
|
return SEC_E_INSUFFICIENT_MEMORY;
|
||||||
|
}
|
||||||
|
}
|
||||||
else
|
else
|
||||||
CopyMemory(data_buffer->pvBuffer, data, length);
|
CopyMemory(data_buffer->pvBuffer, data, length);
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user