Changeset 13502 in vbox

Timestamp:

Oct 22, 2008 3:52:53 PM (16 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

38327

Message:

VMM: Prevent the debugger from crashing the VMM passing it guest addresses above 4GB when the guest is in 32-bit mode.

File:

• trunk/src/VBox/VMM/DBGFMem.cpp (modified) (3 diffs)

trunk/src/VBox/VMM/DBGFMem.cpp

@@ -75 +75 @@
     else
     {
+        if (    (   pAddress->FlatPtr >= _4G
+                 || pAddress->FlatPtr + cbRange > _4G)
+            &&  enmMode != PGMMODE_AMD64
+            &&  enmMode != PGMMODE_AMD64_NX)
+            return VERR_DBGF_MEM_NOT_FOUND;
         RTGCUINTPTR GCPtrHit;
         rc = PGMR3DbgScanVirtual(pVM, pAddress->FlatPtr, cbRange, pabNeedle, cbNeedle, &GCPtrHit);
@@ -147 +152 @@
         rc = PGMPhysSimpleReadGCPhys(pVM, pvBuf, pAddress->FlatPtr, cbRead);
     else
+    {
+        if (    (   pAddress->FlatPtr >= _4G
+                 || pAddress->FlatPtr + cbRead > _4G)
+            &&  enmMode != PGMMODE_AMD64
+            &&  enmMode != PGMMODE_AMD64_NX)
+            return VERR_PAGE_TABLE_NOT_PRESENT;
         rc = PGMPhysSimpleReadGCPtr(pVM, pvBuf, pAddress->FlatPtr, cbRead);
+    }
     return rc;
 }
@@ -205 +217 @@
         rc = PGMPhysSimpleReadGCPhys(pVM, pszBuf, pAddress->FlatPtr, cchBuf);
     else
+    {
+        if (    (   pAddress->FlatPtr >= _4G
+                 || pAddress->FlatPtr + cchBuf > _4G)
+            &&  enmMode != PGMMODE_AMD64
+            &&  enmMode != PGMMODE_AMD64_NX)
+            return VERR_PAGE_TABLE_NOT_PRESENT;
         rc = PGMPhysSimpleReadGCPtr(pVM, pszBuf, pAddress->FlatPtr, cchBuf);
+    }
 
     /*