Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-10-01 23:14:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

NtAllocateVirtualMemory: Add check to make sure that BaseAddress + RegionSize is inside MemoryArea.

Browse source 
For AllocationType of MEM_RESET add FIXME's and return STATUS_SUCCESS, without modifying attributes of region. Fixes 2 virtual tests for kernel32_winetest.

svn path=/trunk/; revision=43049
This commit is contained in:
Michael Martin 2009-09-14 09:21:05 +00:00
parent 89975fc479
commit 398838e6c3
1 changed files with 30 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
reactos/ntoskrnl/mm/anonmem.c
View file

@ -729,6 +729,36 @@ NtAllocateVirtualMemory(IN HANDLE ProcessHandle,
{
MemoryAreaLength = (ULONG_PTR)MemoryArea->EndingAddress -
(ULONG_PTR)MemoryArea->StartingAddress;
if (((ULONG)BaseAddress + RegionSize) > (ULONG)MemoryArea->EndingAddress)
{
DPRINT("BaseAddress + RegionSize %x is larger than MemoryArea's EndingAddress %x\n",
(ULONG)BaseAddress + RegionSize, MemoryArea->EndingAddress);
MmUnlockAddressSpace(AddressSpace);
ObDereferenceObject(Process);
return STATUS_MEMORY_NOT_ALLOCATED;
}
if (AllocationType == MEM_RESET)
{
if (MmIsPagePresent(Process, BaseAddress))
{
/* FIXME: mark pages as not modified */
}
else
{
/* FIXME: if pages are in paging file discard them and bring in pages of zeros */
}
MmUnlockAddressSpace(AddressSpace);
ObDereferenceObject(Process);
/* MEM_RESET does not modify any attributes of region */
return STATUS_SUCCESS;
}
if (MemoryArea->Type == MEMORY_AREA_VIRTUAL_MEMORY &&
MemoryAreaLength >= RegionSize)
{