[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 at 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.