Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-07-03 03:04:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

- Remove a header structure from memory regions allocated with EngAllocUserMem. Modern versions of Windows (at least XP and higher) don't do this and instead save this information elsewhere. Confirmed by Alexandre Julliard's test case for calling WriteFile with the DIB section bits as buffer (5 less failures now) and by arguing with Evgeniy Boltik.

Browse source 
svn path=/trunk/; revision=40269
This commit is contained in:
Aleksey Bragin 2009-03-28 17:06:17 +00:00
parent b5ceb7733a
commit 02258e7e39
1 changed files with 7 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
reactos/subsystems/win32/win32k/eng/mem.c
View file

@ -32,13 +32,6 @@
#define NDEBUG
#include <debug.h>
typedef struct _USERMEMHEADER
{
ULONG Tag;
ULONG MemSize;
}
USERMEMHEADER, *PUSERMEMHEADER;
/*
* @implemented
*/
@ -76,8 +69,7 @@ EngAllocUserMem(SIZE_T cj, ULONG Tag)
{
PVOID NewMem = NULL;
NTSTATUS Status;
SIZE_T MemSize = sizeof(USERMEMHEADER) + cj;
PUSERMEMHEADER Header;
SIZE_T MemSize = cj;
Status = ZwAllocateVirtualMemory(NtCurrentProcess(), &NewMem, 0, &MemSize, MEM_COMMIT | MEM_RESERVE, PAGE_READWRITE);
@ -86,11 +78,9 @@ EngAllocUserMem(SIZE_T cj, ULONG Tag)
return NULL;
}
Header = (PUSERMEMHEADER) NewMem;
Header->Tag = Tag;
Header->MemSize = cj;
/* TODO: Add allocation info to AVL tree (stored inside W32PROCESS structure) */
return (PVOID)(Header + 1);
return NewMem;
}
/*
@ -99,10 +89,12 @@ EngAllocUserMem(SIZE_T cj, ULONG Tag)
VOID APIENTRY
EngFreeUserMem(PVOID pv)
{
PUSERMEMHEADER Header = ((PUSERMEMHEADER) pv) - 1;
PVOID BaseAddress = pv;
SIZE_T MemSize = 0;
ZwFreeVirtualMemory(NtCurrentProcess(), (PVOID *) &Header, &MemSize, MEM_RELEASE);
ZwFreeVirtualMemory(NtCurrentProcess(), &BaseAddress, &MemSize, MEM_RELEASE);
/* TODO: Remove allocation info from AVL tree */
}