DBG: possibility to lock the stack (set bFreezeStack to true to freeze on automatic stack updates as CSP changes)

Author: mrexodia

src/dbg/debugger.cpp

@@ -33,6 +33,7 @@ static bool isDetachedByUser = false;
 static bool bIsAttached = false;
 static bool bSkipExceptions = false;
 static bool bBreakOnNextDll = false;
+static bool bFreezeStack = false;
 static int ecount = 0;
 static std::vector<ExceptionRange> ignoredExceptionRange;
 static HANDLE hEvent = 0;
@@ -214,7 +215,7 @@ void DebugUpdateGui(duint disasm_addr, bool stack)
     }
     duint csp = GetContextDataEx(hActiveThread, UE_CSP);
     if(stack)
-        GuiStackDumpAt(csp, csp);
+        DebugUpdateStack(csp, csp);
     static duint cacheCsp = 0;
     if(csp != cacheCsp)
     {
@@ -233,6 +234,17 @@ void DebugUpdateGui(duint disasm_addr, bool stack)
     GuiUpdateAllViews();
 }
 
+void DebugUpdateStack(duint dumpAddr, duint csp, bool forceDump)
+{
+    if (!forceDump && bFreezeStack)
+    {
+        SELECTIONDATA selection;
+        if (GuiSelectionGet(GUI_STACK, &selection))
+            dumpAddr = selection.start;
+    }
+    GuiStackDumpAt(dumpAddr, csp);
+}
+
 void cbUserBreakpoint()
 {
     hActiveThread = ThreadGetHandle(((DEBUG_EVENT*)GetDebugData())->dwThreadId);

src/dbg/debugger.h

@@ -69,6 +69,7 @@ bool dbgisrunning();
 bool dbgisdll();
 void dbgsetattachevent(HANDLE handle);
 void DebugUpdateGui(duint disasm_addr, bool stack);
+void DebugUpdateStack(duint dumpAddr, duint csp, bool forceDump = false);
 void dbgsetskipexceptions(bool skip);
 void dbgsetstepping(bool stepping);
 void dbgsetispausedbyuser(bool b);

src/dbg/debugger_commands.cpp

@@ -1045,7 +1045,7 @@ CMDRESULT cbDebugStackDump(int argc, char* argv[])
     duint size = 0;
     duint base = MemFindBaseAddr(csp, &size);
     if(base && addr >= base && addr < (base + size))
-        GuiStackDumpAt(addr, csp);
+        DebugUpdateStack(addr, csp, true);
     else
         dputs("Invalid stack address!");
     return STATUS_CONTINUE;

src/dbg/instruction.cpp

@@ -767,7 +767,7 @@ CMDRESULT cbInstrPush(int argc, char* argv[])
     }
     Script::Stack::Push(value);
     duint csp = GetContextDataEx(hActiveThread, UE_CSP);
-    GuiStackDumpAt(csp, csp);
+    DebugUpdateStack(csp, csp);
     GuiUpdateRegisterView();
     return STATUS_CONTINUE;
 }
@@ -776,7 +776,7 @@ CMDRESULT cbInstrPop(int argc, char* argv[])
 {
     duint value = Script::Stack::Pop();
     duint csp = GetContextDataEx(hActiveThread, UE_CSP);
-    GuiStackDumpAt(csp, csp);
+    DebugUpdateStack(csp, csp);
     GuiUpdateRegisterView();
     if (argc > 1)
     {

src/dbg/value.cpp

@@ -2180,7 +2180,7 @@ bool valtostring(const char* string, duint value, bool silent)
         else if(strstr(regName(), "sp")) //update stack
         {
             duint csp = GetContextDataEx(hActiveThread, UE_CSP);
-            GuiStackDumpAt(csp, csp);
+            DebugUpdateStack(csp, csp);
             GuiUpdateRegisterView();
         }
         else