DBG+GUI: fix many application verifier issues

src/dbg/_dbgfunctions.cpp

@@ -103,7 +103,8 @@ static bool _patchrestore(duint addr)
 
 static void _getcallstack(DBGCALLSTACK* callstack)
 {
-    stackgetcallstack(GetContextDataEx(hActiveThread, UE_CSP), (CALLSTACK*)callstack);
+    if(hActiveThread)
+        stackgetcallstack(GetContextDataEx(hActiveThread, UE_CSP), (CALLSTACK*)callstack);
 }
 
 static void _getsehchain(DBGSEHCHAIN* sehchain)

src/dbg/_exports.cpp

@@ -90,7 +90,7 @@ extern "C" DLL_EXPORT bool _dbg_valfromstring(const char* string, duint* value)
 
 extern "C" DLL_EXPORT bool _dbg_isdebugging()
 {
-    return IsFileBeingDebugged();
+    return hDebugLoopThread && IsFileBeingDebugged();
 }
 
 extern "C" DLL_EXPORT bool _dbg_isjumpgoingtoexecute(duint addr)

src/dbg/loop.cpp

@@ -2,6 +2,7 @@
 #include "memory.h"
 #include "threading.h"
 #include "module.h"
+#include "debugger.h"
 
 std::map<DepthModuleRange, LOOPSINFO, DepthModuleRangeCompare> loops;
 

src/dbg/memory.cpp

@@ -36,6 +36,9 @@ void MemUpdateMap()
 
         do
         {
+            if(!DbgIsDebugging())
+                return;
+
             // Query memory attributes
             MEMORY_BASIC_INFORMATION mbi;
             memset(&mbi, 0, sizeof(mbi));
@@ -106,6 +109,9 @@ void MemUpdateMap()
     char curMod[MAX_MODULE_SIZE] = "";
     for(int i = pagecount - 1; i > -1; i--)
     {
+        if(!DbgIsDebugging())
+            return;
+
         auto & currentPage = pageVector.at(i);
         if(!currentPage.info[0] || (scmp(curMod, currentPage.info) && !bListAllPages)) //there is a module
             continue; //skip non-modules
@@ -331,7 +337,7 @@ bool MemoryReadSafePage(HANDLE hProcess, LPVOID lpBaseAddress, LPVOID lpBuffer,
 
 bool MemRead(duint BaseAddress, void* Buffer, duint Size, duint* NumberOfBytesRead, bool cache)
 {
-    if(!MemIsCanonicalAddress(BaseAddress))
+    if(!MemIsCanonicalAddress(BaseAddress) || !DbgIsDebugging())
         return false;
 
     if(cache && !MemIsValidReadPtr(BaseAddress, true))
@@ -383,7 +389,7 @@ bool MemReadUnsafePage(HANDLE hProcess, LPVOID lpBaseAddress, LPVOID lpBuffer, S
 
 bool MemReadUnsafe(duint BaseAddress, void* Buffer, duint Size, duint* NumberOfBytesRead)
 {
-    if(!MemIsCanonicalAddress(BaseAddress) || BaseAddress < PAGE_SIZE)
+    if(!MemIsCanonicalAddress(BaseAddress) || BaseAddress < PAGE_SIZE || !DbgIsDebugging())
         return false;
 
     if(!Buffer || !Size)

src/dbg/stackinfo.cpp

@@ -358,5 +358,6 @@ void stackupdatesettings()
 {
     ShowSuspectedCallStack = settingboolget("Engine", "ShowSuspectedCallStack");
     std::vector<CALLSTACKENTRY> dummy;
-    stackgetcallstack(GetContextDataEx(hActiveThread, UE_CSP), dummy, false);
+    if(hActiveThread)
+        stackgetcallstack(GetContextDataEx(hActiveThread, UE_CSP), dummy, false);
 }

src/dbg_assert.h

@@ -26,7 +26,7 @@
 #define __DBG_MACRO_CONCAT(a, b) a##b
 
 // Trigger an exception if the debugger is not currently active
-#define ASSERT_DEBUGGING(message) ASSERT_TRUE(DbgIsDebugging(), message)
+#define ASSERT_DEBUGGING(message) ASSERT_TRUE(IsFileBeingDebugged(), message)
 
 // Trigger an exception if expression is false
 #define ASSERT_TRUE(...) \

src/gui/Src/BasicView/Disassembly.cpp

@@ -14,6 +14,7 @@ Disassembly::Disassembly(QWidget* parent) : AbstractTableView(parent), mDisassem
     mMemPage = new MemoryPage(0, 0);
 
     mInstBuffer.clear();
+    setDrawDebugOnly(true);
 
     historyClear();
 

src/gui/Src/BasicView/HexDump.cpp

@@ -11,6 +11,8 @@ HexDump::HexDump(QWidget* parent)
     memset(&data, 0, sizeof(SelectionData_t));
     mSelection = data;
 
+    setDrawDebugOnly(true);
+
     mGuiState = HexDump::NoState;
 
     setRowCount(0);