# HG changeset patch
# User jkeil
# Date 996401873 0
# Node ID db849cee5777e1b906fcae8a744d48fdff9bb0b3
# Parent  eda16e490ae70a0c5da099ac98e03a2b700d4343
Pre-allocate some stack space to work around a problem with DLL alloca() code
on a linux kernel.

diff -r eda16e490ae7 -r db849cee5777 loader/pe_image.c
--- a/loader/pe_image.c	Sun Jul 29 02:08:27 2001 +0000
+++ b/loader/pe_image.c	Sun Jul 29 10:17:53 2001 +0000
@@ -884,19 +884,23 @@
  * due to the PROCESS_Create stuff.
  */
 
-#if	0
 /*
- * so this is a dirty hack.
- * Why do we need it?
- *
- * Disable it for now, let's see if it breaks something
+ * This is a dirty hack.
+ * The win32 DLLs contain an alloca routine, that first probes the soon
+ * to be allocated new memory *below* the current stack pointer in 4KByte
+ * increments.  After the mem probing below the current %esp,  the stack
+ * pointer is finally decremented to make room for the "alloca"ed memory.
+ * Maybe the probing code is intended to extend the stack on a windows box.
+ * Anyway, the linux kernel does *not* extend the stack by simply accessing
+ * memory below %esp;  it segfaults.
+ * The extend_stack_for_dll_alloca() routine just preallocates a big chunk
+ * of memory on the stack, for use by the DLLs alloca routine.
  */
-static void This_Is_Dirty_Hack(void)
+static void extend_stack_for_dll_alloca(void)
 {
     void* mem=alloca(0x20000);
     *(int*)mem=0x1234;
 }
-#endif
 
 
 /* Called if the library is loaded or freed.
@@ -939,9 +943,7 @@
 		break;
 	}	
 	TRACE("for %s\n", wm->filename);
-#if 0
-	This_Is_Dirty_Hack();
-#endif
+	extend_stack_for_dll_alloca();
         retv = entry( wm->module, type, lpReserved );
     }