Changeset 65219 in vbox

Timestamp:

Jan 10, 2017 11:00:36 AM (8 years ago)

Author:

vboxsync

Message:

Devices/Storage/IOBufMgmt: Update the amount of free memory immediately after the segment was allocated to prevent resetting bins if multiple segments are allocated. Fixes obscure data corruption in the data buffers because of segments pointing to the same address.

File:

• trunk/src/VBox/Devices/Storage/IOBufMgmt.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/Storage/IOBufMgmt.cpp

@@ -275 +275 @@
         cbAlloc = pSeg->cbSeg;
         AssertPtr(pSeg->pvSeg);
+
+        pThis->cbFree -= cbAlloc;
     }
 
@@ -406 +408 @@
                || RT_FAILURE(rc));
 
-        pThis->cbFree -= cbIoBufAlloc;
-
         RTCritSectLeave(&pThis->CritSectAlloc);
     }