Re: [PATCH 0/2] MR4328: ntdll: Don't allocate vprot entries until commit.

[Alex Henrie (＠alexhenrie)]

Alex Henrie (@alexhenrie) commented about dlls/ntdll/tests/virtual.c:
>  NtClose( file );
>
> }
>
> +static void test_massive_memory_reservation(void)
> +{
> - /* No ok() usage here, failing the test == getting killed by the OOM killer */
> - SIZE_T i;
> - for (i = page_size; i; i <<= 1)
> - {
> -
>    void* ptr;
> -
>    ptr = VirtualAlloc(NULL, i, MEM_RESERVE, PAGE_NOACCESS);
> -
>    if (ptr)

Does the call to VirtualAlloc always succeed on Windows Vista and later? If so, I think it would be better to have `ok(ptr, ...)` instead of `if (ptr)`.