Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-06-06 18:00:41 +00:00
Code Issues Projects Releases Packages Wiki Activity

[ADVAPI32] Add parameter checks to LogonUserExW and initialize the token handle.

Browse source
This commit is contained in:
Eric Kohl 2018-08-05 16:16:53 +02:00
parent b4040ea740
commit 5ebcc6cdc1
1 changed files with 18 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
dll/win32/advapi32/misc/logon.c
View file

@ -469,7 +469,21 @@ LogonUserExW(
NTSTATUS SubStatus = STATUS_SUCCESS; NTSTATUS SubStatus = STATUS_SUCCESS;
NTSTATUS Status; NTSTATUS Status;
*phToken = NULL; if ((ppProfileBuffer != NULL && pdwProfileLength == NULL) ||
(ppProfileBuffer == NULL && pdwProfileLength != NULL))
{
SetLastError(ERROR_INVALID_PARAMETER);
return FALSE;
}
if (ppProfileBuffer != NULL && pdwProfileLength != NULL)
{
*ppProfileBuffer = NULL;
*pdwProfileLength = 0;
}
if (phToken != NULL)
*phToken = NULL;
switch (dwLogonType) switch (dwLogonType)
{ {
@ -654,9 +668,10 @@ LogonUserExW(
TRACE("TokenHandle: %p\n", TokenHandle); TRACE("TokenHandle: %p\n", TokenHandle);
} }
*phToken = TokenHandle; if (phToken != NULL)
*phToken = TokenHandle;
/* FIXME: return ppLogonSid, ppProfileBuffer, pdwProfileLength and pQuotaLimits */ /* FIXME: return ppLogonSid and pQuotaLimits */
done: done:
if (ProfileBuffer != NULL) if (ProfileBuffer != NULL)