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

[Alexandre Julliard]

Qian Hong  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?