Changeset 107724 in vbox

Timestamp:

Jan 13, 2025 8:20:36 PM (4 months ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

166826

Message:

Storage/VHD.cpp: Fix some harmless parfait warnings about possible integer overflows before assigning to a wider type, bugref:3409

File:

• trunk/src/VBox/Storage/VHD.cpp (modified) (3 diffs)

trunk/src/VBox/Storage/VHD.cpp

@@ -1598 +1598 @@
                     }
 
-                    cbToRead = cSectors * VHD_SECTOR_SIZE;
+                    cbToRead = (size_t)cSectors * VHD_SECTOR_SIZE;
 
                     LogFlowFunc(("uVhdOffset=%llu cbToRead=%u\n", uVhdOffset, cbToRead));
@@ -1624 +1624 @@
                     }
 
-                    cbToRead = cSectors * VHD_SECTOR_SIZE;
+                    cbToRead = (size_t)cSectors * VHD_SECTOR_SIZE;
                     LogFunc(("Sectors free: uVhdOffset=%llu cbToRead=%u\n", uVhdOffset, cbToRead));
                     rc = VERR_VD_BLOCK_FREE;
@@ -1691 +1691 @@
                 || (cbToWrite != pImage->cbDataBlock))
             {
-                *pcbPreRead = cBATEntryIndex * VHD_SECTOR_SIZE;
+                *pcbPreRead = (size_t)cBATEntryIndex * VHD_SECTOR_SIZE;
                 *pcbPostRead = pImage->cSectorsPerDataBlock * VHD_SECTOR_SIZE - cbToWrite - *pcbPreRead;