vbox的更動 54060 路徑 trunk/src/VBox/Main/src-client/ConsoleImpl.cpp

時間撮記:

2015-2-2 下午09:17:31 (10 年 以前)

作者:

vboxsync

訊息:

Main: Fix i_configMediumAttachment inconsistency for USB storage devices

檔案:

• trunk/src/VBox/Main/src-client/ConsoleImpl.cpp (修改) (6 筆差異)

trunk/src/VBox/Main/src-client/ConsoleImpl.cpp

@@ -3675 +3675 @@
     AssertReturn(enmVMState == VMSTATE_SUSPENDED, VERR_INVALID_STATE);
 
-    /* Determine the base path for the device instance. */
-    PCFGMNODE pCtlInst;
-    if (strcmp(pcszDevice, "Msd"))
-        pCtlInst = CFGMR3GetChildF(CFGMR3GetRootU(pUVM), "Devices/%s/%u/", pcszDevice, uInstance);
-    else
-        pCtlInst = CFGMR3GetChildF(CFGMR3GetRootU(pUVM), "USB/%s/", pcszDevice, uInstance);
-    AssertReturn(pCtlInst, VERR_INTERNAL_ERROR);
-
-    PCFGMNODE pLunL0 = NULL;
-    int rc = pThis->i_configMediumAttachment(pCtlInst,
-                                             pcszDevice,
+    int rc = pThis->i_configMediumAttachment(pcszDevice,
                                              uInstance,
                                              enmBus,
@@ -3701 +3691 @@
                                              pUVM,
                                              NULL /* paLedDevType */,
-                                             &pLunL0);
-    /* Dump the changed LUN if possible, dump the complete device otherwise */
-    CFGMR3Dump(pLunL0 ? pLunL0 : pCtlInst);
-
+                                             NULL /* ppLunL0 */);
     LogFlowFunc(("Returning %Rrc\n", rc));
     return rc;
@@ -3863 +3850 @@
     AssertReturn(enmVMState == VMSTATE_SUSPENDED, VERR_INVALID_STATE);
 
-    /*
-     * Determine the base path for the device instance. USB Msd devices are handled different
-     * because the PDM USB API requires a differnet CFGM tree when attaching a new USB device.
-     */
-    PCFGMNODE pCtlInst;
-
-    if (enmBus == StorageBus_USB)
-        pCtlInst = CFGMR3CreateTree(pUVM);
-    else
-        pCtlInst = CFGMR3GetChildF(CFGMR3GetRootU(pUVM), "Devices/%s/%u/", pcszDevice, uInstance);
-
-    AssertReturn(pCtlInst, VERR_INTERNAL_ERROR);
-
-    PCFGMNODE pLunL0 = NULL;
-    int rc = pThis->i_configMediumAttachment(pCtlInst,
-                                             pcszDevice,
+    int rc = pThis->i_configMediumAttachment(pcszDevice,
                                              uInstance,
                                              enmBus,
@@ -3894 +3866 @@
                                              pUVM,
                                              NULL /* paLedDevType */,
-                                             &pLunL0);
-    /* Dump the changed LUN if possible, dump the complete device otherwise */
-    if (enmBus != StorageBus_USB)
-        CFGMR3Dump(pLunL0 ? pLunL0 : pCtlInst);
-
+                                             NULL);
     LogFlowFunc(("Returning %Rrc\n", rc));
     return rc;
@@ -9702 +9670 @@
         return VINF_SUCCESS;
 
-    /* Determine the base path for the device instance. */
-    PCFGMNODE pCtlInst;
-
-    if (enmBus == StorageBus_USB)
-        pCtlInst = CFGMR3GetChildF(CFGMR3GetRootU(pUVM), "USB/%s/", pcszDevice);
-    else
-        pCtlInst = CFGMR3GetChildF(CFGMR3GetRootU(pUVM), "Devices/%s/%u/", pcszDevice, uInstance);
-
-    AssertReturn(pCtlInst, VERR_INTERNAL_ERROR);
-
     /* Update the device instance configuration. */
-    PCFGMNODE pLunL0 = NULL;
-    int rc = pThis->i_configMediumAttachment(pCtlInst,
-                                             pcszDevice,
+    int rc = pThis->i_configMediumAttachment(pcszDevice,
                                              uInstance,
                                              enmBus,
@@ -9731 +9687 @@
                                              pUVM,
                                              NULL /* paLedDevType */,
-                                             &pLunL0);
-    /* Dump the changed LUN if possible, dump the complete device otherwise */
-    CFGMR3Dump(pLunL0 ? pLunL0 : pCtlInst);
+                                             NULL /* ppLunL0)*/);
     if (RT_FAILURE(rc))
     {