Changeset 4675 in vbox for trunk/src/VBox/Additions/WINNT/VBoxGuest/VBoxGuest.cpp

Timestamp:

Sep 10, 2007 3:54:41 PM (17 years ago)

Author:

vboxsync

Message:

Fixed wraparound

File:

• trunk/src/VBox/Additions/WINNT/VBoxGuest/VBoxGuest.cpp (modified) (3 diffs)

trunk/src/VBox/Additions/WINNT/VBoxGuest/VBoxGuest.cpp

@@ -550 +550 @@
     {
         /* deflate */
-        for (uint32_t i=pDevExt->MemBalloon.cBalloons-1;i>=u32BalloonSize;i--)
-        {
-            PMDL  pMdl = pDevExt->MemBalloon.paMdlMemBalloon[i];
+        for (uint32_t _i=pDevExt->MemBalloon.cBalloons;_i>u32BalloonSize;_i--)
+        {
+            uint32_t index = _i - 1;
+            PMDL  pMdl = pDevExt->MemBalloon.paMdlMemBalloon[index];
 
             Assert(pMdl);
             if (pMdl)
             {
+#ifdef TARGET_NT4
                 PVOID pvBalloon = MmGetMdlVirtualAddress(pMdl);
+#endif
 
                 PPFN_NUMBER pPageDesc = MmGetMdlPfnArray(pMdl);
@@ -576 +579 @@
                 }
 
-                dprintf(("VBoxGuest::VBoxGuestSetBalloonSize %d MB free chunk at %x\n", i, pvBalloon));
-
                 /* Free the ballooned memory */
 #ifndef TARGET_NT4
+                dprintf(("VBoxGuest::VBoxGuestSetBalloonSize %d MB free chunk at %x\n", index, pMdl));
                 MmFreePagesFromMdl(pMdl);
                 ExFreePool(pMdl);
 #else
+                dprintf(("VBoxGuest::VBoxGuestSetBalloonSize %d MB free chunk at %x\n", index, pvBalloon));
                 MmUnlockPages (pMdl);
                 IoFreeMdl (pMdl);
@@ -588 +591 @@
 #endif
 
-                pDevExt->MemBalloon.paMdlMemBalloon[i] = NULL;
+                pDevExt->MemBalloon.paMdlMemBalloon[index] = NULL;
                 pDevExt->MemBalloon.cBalloons--;
             }