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)`.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4328#note_51351