Changeset 7652 in vbox for trunk/src/VBox/Devices/Storage/VDIHDDCore.cpp

Timestamp:

Mar 31, 2008 12:05:47 PM (17 years ago)

Author:

vboxsync

Message:

Tiny validity check and comment cleanup in new VDI implementation.

File:

• trunk/src/VBox/Devices/Storage/VDIHDDCore.cpp (modified) (5 diffs)

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

@@ -366 +366 @@
     vdiSetupImageDesc(pImage);
 
-    /* create file */
+    /* Create image file. */
     rc = RTFileOpen(&File, pImage->pszFilename,
                     RTFILE_O_READWRITE | RTFILE_O_CREATE | RTFILE_O_DENY_ALL);
@@ -381 +381 @@
     if (enmType == VD_IMAGE_TYPE_FIXED)
     {
-        /* check the free space on the disk and leave early if there is not
-         * sufficient space available */
+        /* Check the free space on the disk and leave early if there is not
+         * sufficient space available. */
         RTFOFF cbFree = 0;
         rc = RTFsQuerySizes(pImage->pszFilename, NULL, &cbFree, NULL, NULL);
@@ -460 +460 @@
         cbFill = (uint64_t)getImageBlocks(&pImage->Header) * pImage->cbTotalBlockData;
         uOff = 0;
-        /* do loop to fill all image. */
+        /* Write data to all image blocks. */
         while (uOff < cbFill)
         {
@@ -711 +711 @@
     PVDIIMAGEDESC pImage;
 
-    if (   !pszFilename
+    if (   !VALID_PTR(pszFilename)
         || !*pszFilename)
     {
@@ -801 +801 @@
 
     /* Check remaining arguments. */
-    if (   !pszFilename
+    if (   !VALID_PTR(pszFilename)
         || !*pszFilename
         || (enmType != VD_IMAGE_TYPE_NORMAL && enmType != VD_IMAGE_TYPE_FIXED)