ReceiveTempSMS
/
Sweden-Number
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

winedbg: Native dbghelp can modify the passed debug context in StackWalk, so guard against it.

This commit is contained in:
Eric Pouech 2006-08-13 20:46:54 +02:00 committed by Alexandre Julliard
parent 588589b8de
commit 84650ebbb2
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
programs/winedbg/stack.c
View File

@ -147,6 +147,10 @@ unsigned stack_fetch_frames(void)
{ {
STACKFRAME64 sf; STACKFRAME64 sf;
unsigned nf = 0; unsigned nf = 0;
/* as native stackwalk can modify the context passed to it, simply copy
* it to avoid any damage
*/
CONTEXT ctx = dbg_context;
HeapFree(GetProcessHeap(), 0, dbg_curr_thread->frames); HeapFree(GetProcessHeap(), 0, dbg_curr_thread->frames);
dbg_curr_thread->frames = NULL; dbg_curr_thread->frames = NULL;
@ -163,7 +167,7 @@ unsigned stack_fetch_frames(void)
} }
while (StackWalk64(IMAGE_FILE_MACHINE_I386, dbg_curr_process->handle, while (StackWalk64(IMAGE_FILE_MACHINE_I386, dbg_curr_process->handle,
dbg_curr_thread->handle, &sf, &dbg_context, stack_read_mem, dbg_curr_thread->handle, &sf, &ctx, stack_read_mem,
SymFunctionTableAccess64, SymGetModuleBase64, NULL)) SymFunctionTableAccess64, SymGetModuleBase64, NULL))
{ {
dbg_curr_thread->frames = dbg_heap_realloc(dbg_curr_thread->frames, dbg_curr_thread->frames = dbg_heap_realloc(dbg_curr_thread->frames,