[PATCH 2/3] ntdll: Improve invalid parameter handling in NtAccessCheck. (try 2)

[Alexandre Julliard]

Qian Hong <qhong at codeweavers.com> writes:

> @@ -1586,7 +1586,16 @@ NtAccessCheck(
>          SecurityDescriptor, ClientToken, DesiredAccess, GenericMapping,
>          PrivilegeSet, ReturnLength, GrantedAccess, AccessStatus);
>  
> -    if (!PrivilegeSet || !ReturnLength)
> +    if (!ReturnLength)
> +        return STATUS_ACCESS_VIOLATION;
> +
> +    if (*ReturnLength == 0)
> +    {
> +        *ReturnLength = sizeof(PRIVILEGE_SET);

Shouldn't this depend on the number of privileges to be returned?

-- 
Alexandre Julliard
julliard at winehq.org