Changeset 2279 in vbox

Timestamp:

Apr 20, 2007 2:31:17 PM (18 years ago)

Author:

vboxsync

Message:

Don't recursively call PGMVerifyAccess as we might run out of stack.

File:

• trunk/src/VBox/VMM/VMMAll/PGMAll.cpp (modified) (1 diff)

trunk/src/VBox/VMM/VMMAll/PGMAll.cpp

@@ -504 +504 @@
         &&  (   PAGE_ADDRESS(Addr) != PAGE_ADDRESS(Addr + cbSize - 1)
              || Addr + cbSize < Addr))
-        return PGMVerifyAccess(pVM, Addr + PAGE_SIZE, cbSize > PAGE_SIZE ? cbSize - PAGE_SIZE : 1, fAccess);
+    {
+        Addr += PAGE_SIZE;
+        if (cbSize > PAGE_SIZE)
+            cbSize =- PAGE_SIZE;
+        else
+            cbSize = 0;
+
+        /* Don't recursively call PGMVerifyAccess as we might run out of stack. */
+        for(;;)
+        {
+            rc = PGMVerifyAccess(pVM, Addr, 1, fAccess);
+            if (rc != VINF_SUCCESS)
+                break;
+
+            Addr += PAGE_SIZE;
+            if (cbSize > PAGE_SIZE)
+                cbSize =- PAGE_SIZE;
+            else
+                break;
+        }
+    }
     return rc;
 }