Changeset 64711 in vbox for trunk

Timestamp:

Nov 18, 2016 11:45:00 AM (8 years ago)

Author:

vboxsync

Message:

VD: Check that the size is aligned to a 512 byte sector boundary

Location:

trunk/src/VBox/Storage

Files:

• VD.cpp (modified) (1 diff)
• VDI.cpp (modified) (1 diff)

trunk/src/VBox/Storage/VD.cpp

@@ -7013 +7013 @@
                            ("cbSize=%llu\n", cbSize),
                            rc = VERR_INVALID_PARAMETER);
+        AssertMsgBreakStmt(!(cbSize % 512),
+                           ("cbSize=%llu\n", cbSize),
+                           rc = VERR_VD_INVALID_SIZE);
         AssertMsgBreakStmt(   ((uImageFlags & ~VD_IMAGE_FLAGS_MASK) == 0)
                            || ((uImageFlags & (VD_IMAGE_FLAGS_FIXED | VD_IMAGE_FLAGS_DIFF)) != VD_IMAGE_FLAGS_FIXED),

trunk/src/VBox/Storage/VDI.cpp

@@ -1415 +1415 @@
     if (   !cbSize
         || cbSize >= _1P * 4 - _1M * 3
-        || cbSize < VDI_IMAGE_DEFAULT_BLOCK_SIZE)
+        || cbSize < VDI_IMAGE_DEFAULT_BLOCK_SIZE
+        || (cbSize % 512))
         return VERR_VD_INVALID_SIZE;