Changeset 54948 in vbox for trunk/src/VBox/Main/include/MediumImpl.h

Timestamp:

Mar 25, 2015 4:56:48 PM (10 years ago)

Author:

vboxsync

Message:

Main/Medium+Snapshot: make all code recursing over trees (objects containing lists of child objects) use as little stack as possible, and establish safe depth limits to avoid crashes, plus related cleanups in related code areas

File:

• trunk/src/VBox/Main/include/MediumImpl.h (modified) (5 diffs)

trunk/src/VBox/Main/include/MediumImpl.h

@@ -1 +1 @@
 /* $Id$ */
-
 /** @file
- *
  * VirtualBox COM class implementation
  */
@@ -72 +70 @@
 
     // initializer used when loading settings
+    HRESULT initOne(Medium *aParent,
+                    DeviceType_T aDeviceType,
+                    const Guid &uuidMachineRegistry,
+                    const settings::Medium &data,
+                    const Utf8Str &strMachineFolder);
     HRESULT init(VirtualBox *aVirtualBox,
                  Medium *aParent,
@@ -77 +80 @@
                  const Guid &uuidMachineRegistry,
                  const settings::Medium &data,
-                 const Utf8Str &strMachineFolder);
+                 const Utf8Str &strMachineFolder,
+                 AutoWriteLock &mediaTreeLock);
 
     // initializer for host floppy/DVD
@@ -109 +113 @@
 
     /* handles caller/locking itself */
-    bool i_addRegistry(const Guid& id, bool fRecurse);
+    bool i_addRegistry(const Guid &id);
+    /* handles caller/locking itself, caller is responsible for tree lock */
+    bool i_addRegistryRecursive(const Guid &id);
     /* handles caller/locking itself */
-    bool i_removeRegistry(const Guid& id, bool fRecurse);
+    bool i_removeRegistry(const Guid& id);
+    /* handles caller/locking itself, caller is responsible for tree lock */
+    bool i_removeRegistryRecursive(const Guid& id);
     bool i_isInRegistry(const Guid& id);
     bool i_getFirstRegistryMachineId(Guid &uuid) const;
@@ -135 +143 @@
     HRESULT i_updatePath(const Utf8Str &strOldPath, const Utf8Str &strNewPath);
 
+    /* handles caller/locking itself */
     ComObjPtr<Medium> i_getBase(uint32_t *aLevel = NULL);
+    /* handles caller/locking itself */
+    uint32_t i_getDepth();
 
     bool i_isReadOnly();
     void i_updateId(const Guid &id);
 
+    void i_saveSettingsOne(settings::Medium &data,
+                           const Utf8Str &strHardDiskFolder);
     HRESULT i_saveSettings(settings::Medium &data,
                            const Utf8Str &strHardDiskFolder);