Changeset 21981 for trunk/src/kernel32/thread.cpp

Timestamp:

Mar 18, 2012, 12:41:27 AM (13 years ago)

Author:

dmik

Message:

kernel32: Fix stack size interpretation in CreateThread().

It now uses the stack size parameter as a number of bytes to
pre-commit, not as a total stack size, unless
STACK_SIZE_PARAM_IS_A_RESERVATION is set.

Closes #77.

File:

• trunk/src/kernel32/thread.cpp (modified) (3 diffs)

trunk/src/kernel32/thread.cpp

@@ -674 +674 @@
 //******************************************************************************
 //******************************************************************************
-Win32Thread::Win32Thread(LPTHREAD_START_ROUTINE pUserCallback, LPVOID lpData, DWORD dwFlags, HANDLE hThread)
+Win32Thread::Win32Thread(LPTHREAD_START_ROUTINE pUserCallback, LPVOID lpData, DWORD dwFlags, HANDLE hThread, DWORD cbCommitStack)
 {
     lpUserData = lpData;
@@ -680 +680 @@
     this->dwFlags = dwFlags;
     this->hThread = hThread;
+    this->cbCommitStack = cbCommitStack;
 
     teb = CreateTEB(hThread, 0xFFFFFFFF);
@@ -721 +722 @@
 
     dprintf(("Stack top 0x%x, stack end 0x%x", winteb->stack_top, winteb->stack_low));
+
+    if (me->cbCommitStack) {
+        // pre-commit part of the stack
+        PBYTE stack = ((PBYTE) (winteb->stack_top)) - 1;
+        for (int i = 0; i < (me->cbCommitStack + 0xFFF) / 0x1000; i++) {
+            BYTE unused = *stack;
+            unused = unused;
+            stack -= 0x1000;
+        }
+    }
 
     if( IsDBCSEnv())