Re: [PATCH v10 0/8] MR1628: ntdll: Implement Low Fragmentation Heap.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/ntdll/heap.c:
>      err = "invalid block size";
>  else if (block->tail_size > block_get_size( block ) - sizeof(*block))
>      err = "invalid block unused size";
> - else if ((next = next_block( subheap, block )) && (block_get_flags( next ) & BLOCK_FLAG_PREV_FREE))
> - else if ((next = next_block( subheap, block )) && (block_get_flags( next ) & BLOCK_FLAG_PREV_FREE) &&
> -
>         /* LFH blocks do not use BLOCK_FLAG_PREV_FREE or back pointer */
> -
>         !(block_get_flags( block ) & BLOCK_FLAG_LFH))
>    err = "invalid next block flags";
> else if (block_get_flags( block ) & BLOCK_FLAG_PREV_FREE)

```suggestion:-4+0
    else
    {
        /* LFH blocks do not use BLOCK_FLAG_PREV_FREE or back pointer */
        next = (block_get_flags( block ) & BLOCK_FLAG_LFH) ? NULL : next_block( subheap, block );
    }

if (!err && next && (block_get_flags( next ) & BLOCK_FLAG_PREV_FREE))
        err = "invalid next block flags";

if (!err && (block_get_flags( block ) & BLOCK_FLAG_PREV_FREE))
```