Changeset 31570 in vbox for trunk/src/VBox/Main

Timestamp:

Aug 11, 2010 1:42:06 PM (15 years ago)

Author:

vboxsync

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

64674

Message:

Main: fix for missing UUIDs for dvd and floppy media

File:

• trunk/src/VBox/Main/MediumImpl.cpp (modified) (3 diffs)

trunk/src/VBox/Main/MediumImpl.cpp

@@ -789 +789 @@
          * Created state here and also add it to the registry */
         m->state = MediumState_Created;
+        // create new UUID
         unconst(m->id).create();
 
@@ -807 +808 @@
  * read/write or read-only.
  *
- * This gets called by VirtualBox::OpenHardDisk(), OpenDVDImage and OpenFloppyImage();
- * this also gets called by Machine::AttachDevice() and createImplicitDiffs()
- * when new diff images are created.
- *
- * Note that the UUID, format and the parent of this medium will be
- * determined when reading the medium storage unit, unless new values are
- * specified by the parameters. If the detected or set parent is
- * not known to VirtualBox, then this method will fail.
+ * This gets called by VirtualBox::OpenMedium() and also by
+ * Machine::AttachDevice() and createImplicitDiffs() when new diff
+ * images are created.
+ *
+ * For hard disks, the UUID, format and the parent of this medium will be
+ * determined when reading the medium storage unit. For DVD and floppy images,
+ * which have no UUIDs in their storage units, new UUIDs are created.
+ * If the detected or set parent is not known to VirtualBox, then this method
+ * will fail.
  *
  * @param aVirtualBox   VirtualBox object.
@@ -855 +857 @@
         rc = setLocation(aLocation, "RAW");
     if (FAILED(rc)) return rc;
+
+    if (    aDeviceType == DeviceType_DVD
+         || aDeviceType == DeviceType_Floppy)
+        // create new UUID
+        unconst(m->id).create();
 
     /* get all the information about the medium from the storage unit */