Re: [ros-dev] [ros-diffs] [tkreuzer] 63388: [NTOSKRNL] Fix ending address calculation for the commit path in NtAllocateVirtualMemory like done for the reserve path in r63356. Add a comment about a Windows kernel bug, which ...

[Thomas Faber]

On 2014-05-20 23:11, tkreuzer(a)svn.reactos.org wrote:
>   --- trunk/reactos/ntoskrnl/mm/ARM3/virtual.c
> [iso-8859-1] (original)
>  +++ trunk/reactos/ntoskrnl/mm/ARM3/virtual.c   [iso-8859-1] Tue May 20 21:11:43 2014
>  @@ -4593,8 +4593,8 @@
>       // on the user input, and then compute the actual region size once all the
>       // alignments have been done.
>       //
>  +    EndingAddress = (((ULONG_PTR)PBaseAddress + PRegionSize - 1) | (PAGE_SIZE - 1));
>       StartingAddress = (ULONG_PTR)PAGE_ALIGN(PBaseAddress);
>  -    EndingAddress = (((ULONG_PTR)PBaseAddress + PRegionSize - 1) | (PAGE_SIZE - 1));
>       PRegionSize = EndingAddress - StartingAddress + 1;
>       // 
Can you explain this a bit? I don't see any logic change.