Re: [PATCH 0/1] MR8852: ntdll: Zero aligned size in initialize_block().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/kernel32/tests/heap.c:
> - HANDLE heap = GetProcessHeap();
> - size_t size, size_aligned;
> - ULONG_PTR v, expected;
> - char *p1, *p2;
> -
> - if (heap_flags & HEAP_PAGE_ALLOCS)
> - {
> -
>    /* This behaves differently, no support yet. */
> -
>    skip( "Skipping test with HEAP_PAGE_ALLOCS.\n" );
> -
>    return;
> - }
> -
> - for (size = 1; size <= 1048576 * 2; size *= 2)
> - {
> -
>    winetest_push_context( "heap_flags %#lx, size %Iu", heap_flags, size );
> -
>    p1 = HeapAlloc( heap, 0, size + 1 );

You should probably use `pHeapAlloc` here to avoid possible bounds checks by the compiler (although it didn't seem to complain for some reason).