Changeset 1503 in vbox for trunk

Timestamp:

Mar 15, 2007 10:19:01 AM (18 years ago)

Author:

vboxsync

Message:

extra checks

File:

• trunk/src/VBox/VMM/VMMGC/SELMGC.cpp (modified) (1 diff)

trunk/src/VBox/VMM/VMMGC/SELMGC.cpp

@@ -289 +289 @@
 
             /** @todo not sure how the partial case is handled; probably not allowed */
-            if (offRedirBitmap + sizeof(pVM->selm.s.Tss.redirBitmap) <= pVM->selm.s.cbGuestTss)
+            if (   offRedirBitmap <= offRange 
+                && offRedirBitmap + sizeof(pVM->selm.s.Tss.redirBitmap) >= offRange + cb
+                && offRedirBitmap + sizeof(pVM->selm.s.Tss.redirBitmap) <= pVM->selm.s.cbGuestTss)
             {
-                /** @todo check if fault was in this range and, if so, only update the changed part. */
+                Log(("offIoBitmap=%x offRedirBitmap=%x cbTSS=%x\n", pGuestTSS->offIoBitmap, offRedirBitmap, pVM->selm.s.cbGuestTss));
+                /** @todo only update the changed part. */
                 for (uint32_t i=0;i<sizeof(pVM->selm.s.Tss.redirBitmap)/8;i++)
                 {
                     rc = MMGCRamRead(pVM, &pVM->selm.s.Tss.redirBitmap[i*8], (uint8_t *)pGuestTSS + offRedirBitmap + i*8, 8);
-                    AssertRC(rc);
+                    AssertMsg(rc == VINF_SUCCESS, ("MMGCRamRead %VGv failed with %Vrc\n", (uint8_t *)pGuestTSS + offRedirBitmap + i*8, rc));
                 }
                 STAM_COUNTER_INC(&pVM->selm.s.StatGCWriteGuestTSSRedir);