Changeset 51918 in vbox for trunk/src/VBox/HostDrivers

Timestamp:

Jul 8, 2014 3:15:24 AM (11 years ago)

Author:

vboxsync

Message:

SUPDrv-win.cpp: Memory leak. Allocation adjustments.

File:

• trunk/src/VBox/HostDrivers/Support/win/SUPDrv-win.cpp (modified) (4 diffs)

trunk/src/VBox/HostDrivers/Support/win/SUPDrv-win.cpp

@@ -2227 +2227 @@
      * Note! Attempts at using SystemSessionProcessInformation failed with
      *       STATUS_ACCESS_VIOLATION.
+     * Note! The 32 bytes on the size of to counteract the allocation header
+     *       that rtR0MemAllocEx slaps on everything.
      */
-    ULONG       cbNeeded = _64K;
+    ULONG       cbNeeded = _64K - 32;
     uint32_t    cbBuf;
     uint8_t    *pbBuf = NULL;
     do
     {
-        cbBuf = RT_ALIGN(cbNeeded + _4K, _64K);
+        cbBuf = RT_ALIGN(cbNeeded + _4K, _64K) - 32;
         pbBuf = (uint8_t *)RTMemAlloc(cbBuf);
         if (!pbBuf)
@@ -3296 +3298 @@
     /*
      * Take a snapshot of all the handles in the system.
+     * Note! The 32 bytes on the size of to counteract the allocation header
+     *       that rtR0MemAllocEx slaps on everything.
      */
-    uint32_t    cbBuf    = _256K;
+    uint32_t    cbBuf    = _256K - 32;
     uint8_t    *pbBuf    = (uint8_t *)RTMemAlloc(cbBuf);
     ULONG       cbNeeded = cbBuf;
@@ -3307 +3311 @@
                && cbBuf <= 32U*_1M)
         {
-            cbBuf = RT_ALIGN_32(cbNeeded + _4K, _64K);
+            cbBuf = RT_ALIGN_32(cbNeeded + _4K, _64K) - 32;
             RTMemFree(pbBuf);
             pbBuf = (uint8_t *)RTMemAlloc(cbBuf);
@@ -3414 +3418 @@
     }
 
+    RTMemFree(pbBuf);
     return rc;
 }