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

Timestamp:

Mar 24, 2010 5:21:51 PM (15 years ago)

Author:

vboxsync

Message:

Main/VirtualBox: new parameter for overriding VM file existence check on creation.

Location:

trunk/src/VBox/Main

Files:

• ApplianceImpl.cpp (modified) (2 diffs)
• MachineImpl.cpp (modified) (3 diffs)
• VirtualBoxImpl.cpp (modified) (6 diffs)
• idl/VirtualBox.xidl (modified) (2 diffs)
• include/MachineImpl.h (modified) (1 diff)
• include/VirtualBoxImpl.h (modified) (2 diffs)
• testcase/tstAPI.cpp (modified) (2 diffs)
• testcase/tstVBoxAPILinux.cpp (modified) (2 diffs)

trunk/src/VBox/Main/ApplianceImpl.cpp

@@ -6 +6 @@
 
 /*
- * Copyright (C) 2008-2009 Sun Microsystems, Inc.
+ * Copyright (C) 2008-2010 Sun Microsystems, Inc.
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -1233 +1233 @@
             const Utf8Str &strNameVBox = vsdeName.front()->strVbox;
             rc = mVirtualBox->CreateMachine(Bstr(strNameVBox), Bstr(strOsTypeVBox),
-                                                 Bstr(), Bstr(),
+                                                 Bstr(), Bstr(), FALSE,
                                                  pNewMachine.asOutParam());
             if (FAILED(rc)) throw rc;

trunk/src/VBox/Main/MachineImpl.cpp

@@ -266 +266 @@
  *                      (ignored otherwise)
  *  @param aOsType      OS Type of this machine
+ *  @param aOverride    |TRUE| to override VM config file existence checks.
+ *                      |FALSE| refuses to overwrite existing VM configs.
  *  @param aNameSync    |TRUE| to automatically sync settings dir and file
  *                      name with the machine name. |FALSE| is used for legacy
@@ -284 +286 @@
                       CBSTR aName /* = NULL */,
                       GuestOSType *aOsType /* = NULL */,
+                      BOOL aOverride /* = FALSE */,
                       BOOL aNameSync /* = TRUE */,
                       const Guid *aId /* = NULL */)
@@ -347 +350 @@
             RTFILE f = NIL_RTFILE;
             int vrc = RTFileOpen(&f, mData->m_strConfigFileFull.c_str(), RTFILE_O_READ | RTFILE_O_OPEN | RTFILE_O_DENY_NONE);
-            if (    RT_SUCCESS(vrc)
+            if (    (RT_SUCCESS(vrc) && !aOverride)
                  || vrc == VERR_SHARING_VIOLATION
                )

trunk/src/VBox/Main/VirtualBoxImpl.cpp

@@ -6 +6 @@
 
 /*
- * Copyright (C) 2006-2009 Sun Microsystems, Inc.
+ * Copyright (C) 2006-2010 Sun Microsystems, Inc.
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -1031 +1031 @@
                                        IN_BSTR aBaseFolder,
                                        IN_BSTR aId,
+                                       BOOL aOverride,
                                        IMachine **aMachine)
 {
@@ -1084 +1085 @@
                        aName,
                        osType,
+                       aOverride,
                        TRUE /* aNameSync */,
                        &id);
@@ -1139 +1141 @@
                        aName,
                        osType,
+                       FALSE /* aOverride */,
                        FALSE /* aNameSync */,
                        &id);
@@ -4309 +4312 @@
     AssertReturn(pvUser, VERR_INVALID_POINTER);
 
+    com::Initialize();
+
     // create an event queue for the current thread
     EventQueue *eventQ = new EventQueue();
@@ -4327 +4332 @@
 
     delete eventQ;
+
+    com::Shutdown();
 
     LogFlowFuncLeave();

trunk/src/VBox/Main/idl/VirtualBox.xidl

@@ -1690 +1690 @@
   <interface
     name="IVirtualBox" extends="$dispatched"
-    uuid="2158464a-f706-414b-a8c4-fb589dfc6b62"
+    uuid="3f36e024-7fed-4f20-a02c-9158a82b44e6"
     wsmap="managed"
   >
@@ -1919 +1919 @@
       <param name="id" type="uuid" mod="string" dir="in">
         <desc>Machine UUID (optional).</desc>
+      </param>
+      <param name="override" type="boolean" dir="in">
+        <desc>Create the VM even if there are conflicting files.</desc>
       </param>
       <param name="machine" type="IMachine" dir="return">

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

@@ -354 +354 @@
                  CBSTR aName = NULL,
                  GuestOSType *aOsType = NULL,
+                 BOOL aOverride = FALSE,
                  BOOL aNameSync = TRUE,
                  const Guid *aId = NULL);

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

@@ -7 +7 @@
 
 /*
- * Copyright (C) 2006-2009 Sun Microsystems, Inc.
+ * Copyright (C) 2006-2010 Sun Microsystems, Inc.
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -125 +125 @@
 
     STDMETHOD(CreateMachine) (IN_BSTR aName, IN_BSTR aOsTypeId, IN_BSTR aBaseFolder,
-                              IN_BSTR aId, IMachine **aMachine);
+                              IN_BSTR aId, BOOL aOverride, IMachine **aMachine);
     STDMETHOD(CreateLegacyMachine) (IN_BSTR aName, IN_BSTR aOsTypeId, IN_BSTR aSettingsFile,
                                     IN_BSTR aId, IMachine **aMachine);

trunk/src/VBox/Main/testcase/tstAPI.cpp

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2006-2009 Sun Microsystems, Inc.
+ * Copyright (C) 2006-2010 Sun Microsystems, Inc.
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -742 +742 @@
         RTPrintf ("Creating a new machine object (base dir '%ls', name '%ls')...\n",
                 baseDir.raw(), name.raw());
-        CHECK_ERROR_BREAK (virtualBox, CreateMachine (baseDir, name,
+        CHECK_ERROR_BREAK (virtualBox, CreateMachine (name, L"", baseDir, L"",
+                                                      false,
                                                       machine.asOutParam()));
 

trunk/src/VBox/Main/testcase/tstVBoxAPILinux.cpp

@@ -8 +8 @@
 
 /*
- * Copyright (C) 2006-2007 Sun Microsystems, Inc.
+ * Copyright (C) 2006-2010 Sun Microsystems, Inc.
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -195 +195 @@
     nsCOMPtr <IMachine> machine;
     rc = virtualBox->CreateMachine(NS_LITERAL_STRING("A brand new name").get(),
-                                   nsnull, nsnull, nsnull, getter_AddRefs(machine));
+                                   nsnull, nsnull, nsnull, false, getter_AddRefs(machine));
     if (NS_FAILED(rc))
     {