Changeset 98266 in vbox for trunk/src/VBox/Main/src-client

Timestamp:

Jan 24, 2023 9:46:42 AM (2 years ago)

Author:

vboxsync

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

155469

Message:

Main/ConsoleImpl.cpp: rc -> hrc/vrc. bugref:10223

File:

• trunk/src/VBox/Main/src-client/ConsoleImpl.cpp (modified) (92 diffs)

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

@@ -3787 +3787 @@
     AssertReturn(enmVMState == VMSTATE_SUSPENDED, VERR_INVALID_STATE);
 
-    int hrc = pThis->i_configMediumAttachment(pcszDevice,
+    int vrc = pThis->i_configMediumAttachment(pcszDevice,
                                               uInstance,
                                               enmBus,
@@ -3806 +3806 @@
                                               NULL /* paLedDevType */,
                                               NULL /* ppLunL0 */);
-    LogFlowFunc(("Returning %Rrc\n", hrc));
-    return hrc;
+    LogFlowFunc(("Returning %Rrc\n", vrc));
+    return vrc;
 }
 
@@ -3967 +3967 @@
     AssertReturn(enmVMState == VMSTATE_SUSPENDED, VERR_INVALID_STATE);
 
-    int hrc = pThis->i_configMediumAttachment(pcszDevice,
+    int vrc = pThis->i_configMediumAttachment(pcszDevice,
                                               uInstance,
                                               enmBus,
@@ -3986 +3986 @@
                                               NULL /* paLedDevType */,
                                               NULL);
-    LogFlowFunc(("Returning %Rrc\n", hrc));
-    return hrc;
+    LogFlowFunc(("Returning %Rrc\n", vrc));
+    return vrc;
 }
 
@@ -5260 +5260 @@
                 {
                     PPDMIBASE pBase;
-                    int rc2 = ptrVM.vtable()->pfnPDMR3QueryDriverOnLun(ptrVM.rawUVM(),
-                                                                       i_getAudioAdapterDeviceName(aAudioAdapter).c_str(),
-                                                                       0 /* iInstance */, ulLUN, "AUDIO", &pBase);
-                    if (RT_FAILURE(rc2))
+                    int vrc2 = ptrVM.vtable()->pfnPDMR3QueryDriverOnLun(ptrVM.rawUVM(),
+                                                                        i_getAudioAdapterDeviceName(aAudioAdapter).c_str(),
+                                                                        0 /* iInstance */, ulLUN, "AUDIO", &pBase);
+                    if (RT_FAILURE(vrc2))
                         continue;
 
@@ -6839 +6839 @@
     AutoReadLock alock(this COMMA_LOCKVAL_SRC_POS);
 
-    HRESULT rc = S_OK;
     for (size_t i = 0; i < aAttachments.size(); ++i)
     {
@@ -6879 +6878 @@
 
         BOOL fBuiltinIOCache;
-        rc = mMachine->COMGETTER(IOCacheEnabled)(&fBuiltinIOCache);
-        if (FAILED(rc))
+        hrc = mMachine->COMGETTER(IOCacheEnabled)(&fBuiltinIOCache);
+        if (FAILED(hrc))
             return hrc;
 
@@ -6900 +6899 @@
                                                       fUseHostIOCache, fBuiltinIOCache, fInsertDiskIntegrityDrv,
                                                       false /* fSetupMerge */, 0 /* uMergeSource */, 0 /* uMergeTarget */,
-                                                      pAttachment, mMachineState, &rc);
+                                                      pAttachment, mMachineState, &hrc);
         if (RT_FAILURE(vrc))
             throw setErrorBoth(E_FAIL, vrc, "%Rrc", vrc);
@@ -6914 +6913 @@
 HRESULT Console::i_onVMProcessPriorityChange(VMProcPriority_T priority)
 {
-    HRESULT rc = S_OK;
-
     AutoCaller autoCaller(this);
-    if (FAILED(autoCaller.hrc()))
-        return autoCaller.hrc();
+    HRESULT hrc = autoCaller.hrc();
+    if (FAILED(hrc))
+        return hrc;
 
     RTPROCPRIORITY enmProcPriority = RTPROCPRIORITY_DEFAULT;
@@ -6943 +6941 @@
     int vrc = RTProcSetPriority(enmProcPriority);
     if (RT_FAILURE(vrc))
-        rc = setErrorBoth(VBOX_E_VM_ERROR, vrc,
-                          tr("Could not set the priority of the process (%Rrc). Try to set it when VM is not started."), vrc);
-
-    return rc;
+        hrc = setErrorBoth(VBOX_E_VM_ERROR, vrc,
+                           tr("Could not set the priority of the process (%Rrc). Try to set it when VM is not started."), vrc);
+
+    return hrc;
 }
 
@@ -7293 +7291 @@
                     autoCaller.release();
 
-                    HRESULT rc = i_powerDown();
-                    AssertComRC(rc);
+                    HRESULT hrc2 = i_powerDown();
+                    AssertComRC(hrc2);
 
                     autoCaller.add();
@@ -8246 +8244 @@
 
     AutoCaller autoCaller(this);
-    HRESULT rc = autoCaller.hrc();
-    if (FAILED(rc)) return rc;
+    HRESULT hrc = autoCaller.hrc();
+    if (FAILED(hrc))
+        return hrc;
 
     AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
@@ -8259 +8258 @@
     /* Set up release logging as early as possible after the check if
      * there is already a running VM which we shouldn't disturb. */
-    rc = i_consoleInitReleaseLog(mMachine);
-    if (FAILED(rc))
-        return rc;
+    hrc = i_consoleInitReleaseLog(mMachine);
+    if (FAILED(hrc))
+        return hrc;
 
 #ifdef VBOX_OPENSSL_FIPS
@@ -8269 +8268 @@
     /* test and clear the TeleporterEnabled property  */
     BOOL fTeleporterEnabled;
-    rc = mMachine->COMGETTER(TeleporterEnabled)(&fTeleporterEnabled);
-    if (FAILED(rc))
-        return rc;
+    hrc = mMachine->COMGETTER(TeleporterEnabled)(&fTeleporterEnabled);
+    if (FAILED(hrc))
+        return hrc;
 
 #if 0 /** @todo we should save it afterwards, but that isn't necessarily a good idea. Find a better place for this (VBoxSVC).  */
     if (fTeleporterEnabled)
     {
-        rc = mMachine->COMSETTER(TeleporterEnabled)(FALSE);
-        if (FAILED(rc))
-            return rc;
+        hrc = mMachine->COMSETTER(TeleporterEnabled)(FALSE);
+        if (FAILED(hrc))
+            return hrc;
     }
 #endif
@@ -8317 +8316 @@
         {
             Bstr bstrSavedStateFile;
-            rc = mMachine->COMGETTER(StateFilePath)(bstrSavedStateFile.asOutParam());
-            if (FAILED(rc))
-                throw rc;
+            hrc = mMachine->COMGETTER(StateFilePath)(bstrSavedStateFile.asOutParam());
+            if (FAILED(hrc))
+                throw hrc;
             strSavedStateFile = bstrSavedStateFile;
 
 #ifdef VBOX_WITH_FULL_VM_ENCRYPTION
-            rc = mMachine->COMGETTER(StateKeyId)(bstrStateKeyId.asOutParam());
-            if (FAILED(rc))
-                throw rc;
-            rc = mMachine->COMGETTER(StateKeyStore)(bstrStateKeyStore.asOutParam());
-            if (FAILED(rc))
-                throw rc;
+            hrc = mMachine->COMGETTER(StateKeyId)(bstrStateKeyId.asOutParam());
+            if (FAILED(hrc))
+                throw hrc;
+            hrc = mMachine->COMGETTER(StateKeyStore)(bstrStateKeyStore.asOutParam());
+            if (FAILED(hrc))
+                throw hrc;
 #endif
 
@@ -8367 +8366 @@
          * saved state file (if not yet done).
          */
-        rc = i_loadDataFromSavedState();
-        if (FAILED(rc))
-            throw rc;
+        hrc = i_loadDataFromSavedState();
+        if (FAILED(hrc))
+            throw hrc;
 
         /* Check all types of shared folders and compose a single list */
@@ -8411 +8410 @@
          * asynchronously */
         try { task = new VMPowerUpTask(this, pPowerupProgress); }
-        catch (std::bad_alloc &) { throw rc = E_OUTOFMEMORY; }
+        catch (std::bad_alloc &) { throw hrc = E_OUTOFMEMORY; }
         if (!task->isOk())
             throw task->hrc();
@@ -8420 +8419 @@
         if (mMachineState == MachineState_Saved || mMachineState == MachineState_AbortedSaved)
             try { task->mSavedStateFile = strSavedStateFile; }
-            catch (std::bad_alloc &) { throw rc = E_OUTOFMEMORY; }
+            catch (std::bad_alloc &) { throw hrc = E_OUTOFMEMORY; }
         task->mTeleporterEnabled = fTeleporterEnabled;
 
@@ -8432 +8431 @@
          */
         ComPtr<ISnapshot> pCurrentSnapshot;
-        rc = mMachine->COMGETTER(CurrentSnapshot)(pCurrentSnapshot.asOutParam());
-        if (FAILED(rc))
-            throw rc;
+        hrc = mMachine->COMGETTER(CurrentSnapshot)(pCurrentSnapshot.asOutParam());
+        if (FAILED(hrc))
+            throw hrc;
 
         BOOL fCurrentSnapshotIsOnline = false;
         if (pCurrentSnapshot)
         {
-            rc = pCurrentSnapshot->COMGETTER(Online)(&fCurrentSnapshotIsOnline);
-            if (FAILED(rc))
-                throw rc;
+            hrc = pCurrentSnapshot->COMGETTER(Online)(&fCurrentSnapshotIsOnline);
+            if (FAILED(hrc))
+                throw hrc;
         }
 
@@ -8449 +8448 @@
 
             com::SafeIfaceArray<IMediumAttachment> atts;
-            rc = mMachine->COMGETTER(MediumAttachments)(ComSafeArrayAsOutParam(atts));
-            if (FAILED(rc))
-                throw rc;
+            hrc = mMachine->COMGETTER(MediumAttachments)(ComSafeArrayAsOutParam(atts));
+            if (FAILED(hrc))
+                throw hrc;
 
             for (size_t i = 0;
@@ -8458 +8457 @@
             {
                 DeviceType_T devType;
-                rc = atts[i]->COMGETTER(Type)(&devType);
+                hrc = atts[i]->COMGETTER(Type)(&devType);
                 /** @todo later applies to floppies as well */
                 if (devType == DeviceType_HardDisk)
                 {
                     ComPtr<IMedium> pMedium;
-                    rc = atts[i]->COMGETTER(Medium)(pMedium.asOutParam());
-                    if (FAILED(rc))
-                        throw rc;
+                    hrc = atts[i]->COMGETTER(Medium)(pMedium.asOutParam());
+                    if (FAILED(hrc))
+                        throw hrc;
 
                     /* needs autoreset? */
                     BOOL autoReset = FALSE;
-                    rc = pMedium->COMGETTER(AutoReset)(&autoReset);
-                    if (FAILED(rc))
-                        throw rc;
+                    hrc = pMedium->COMGETTER(AutoReset)(&autoReset);
+                    if (FAILED(hrc))
+                        throw hrc;
 
                     if (autoReset)
                     {
                         ComPtr<IProgress> pResetProgress;
-                        rc = pMedium->Reset(pResetProgress.asOutParam());
-                        if (FAILED(rc))
-                            throw rc;
+                        hrc = pMedium->Reset(pResetProgress.asOutParam());
+                        if (FAILED(hrc))
+                            throw hrc;
 
                         /* save for later use on the powerup thread */
@@ -8552 +8551 @@
                 ulTotalOperationsWeight += 1;
             }
-            rc = pPowerupProgress->init(static_cast<IConsole *>(this),
-                                        progressDesc.raw(),
-                                        TRUE, // Cancelable
-                                        cOperations,
-                                        ulTotalOperationsWeight,
-                                        tr("Starting Hard Disk operations"),
-                                        1);
-            AssertComRCReturnRC(rc);
+            hrc = pPowerupProgress->init(static_cast<IConsole *>(this),
+                                         progressDesc.raw(),
+                                         TRUE, // Cancelable
+                                         cOperations,
+                                         ulTotalOperationsWeight,
+                                         tr("Starting Hard Disk operations"),
+                                         1);
+            AssertComRCReturnRC(hrc);
         }
         else if (   mMachineState == MachineState_Saved
                  || mMachineState == MachineState_AbortedSaved
                  || !fTeleporterEnabled)
-            rc = pPowerupProgress->init(static_cast<IConsole *>(this),
-                                        progressDesc.raw(),
-                                        FALSE /* aCancelable */);
+            hrc = pPowerupProgress->init(static_cast<IConsole *>(this),
+                                         progressDesc.raw(),
+                                         FALSE /* aCancelable */);
         else if (fTeleporterEnabled)
-            rc = pPowerupProgress->init(static_cast<IConsole *>(this),
-                                        progressDesc.raw(),
-                                        TRUE /* aCancelable */,
-                                        3    /* cOperations */,
-                                        10   /* ulTotalOperationsWeight */,
-                                        tr("Teleporting virtual machine"),
-                                        1    /* ulFirstOperationWeight */);
-
-        if (FAILED(rc))
-            throw rc;
+            hrc = pPowerupProgress->init(static_cast<IConsole *>(this),
+                                         progressDesc.raw(),
+                                         TRUE /* aCancelable */,
+                                         3    /* cOperations */,
+                                         10   /* ulTotalOperationsWeight */,
+                                         tr("Teleporting virtual machine"),
+                                         1    /* ulFirstOperationWeight */);
+
+        if (FAILED(hrc))
+            throw hrc;
 
         /* Tell VBoxSVC and Machine about the progress object so they can
            combine/proxy it to any openRemoteSession caller. */
         LogFlowThisFunc(("Calling BeginPowerUp...\n"));
-        rc = mControl->BeginPowerUp(pPowerupProgress);
-        if (FAILED(rc))
+        hrc = mControl->BeginPowerUp(pPowerupProgress);
+        if (FAILED(hrc))
         {
             LogFlowThisFunc(("BeginPowerUp failed\n"));
-            throw rc;
+            throw hrc;
         }
         fBeganPoweringUp = true;
@@ -8592 +8591 @@
         LogFlowThisFunc(("Checking if canceled...\n"));
         BOOL fCanceled;
-        rc = pPowerupProgress->COMGETTER(Canceled)(&fCanceled);
-        if (FAILED(rc))
-            throw rc;
+        hrc = pPowerupProgress->COMGETTER(Canceled)(&fCanceled);
+        if (FAILED(hrc))
+            throw hrc;
 
         if (fCanceled)
@@ -8658 +8657 @@
         if (aProgress)
         {
-            rc = pPowerupProgress.queryInterfaceTo(aProgress);
-            AssertComRCReturnRC(rc);
+            hrc = pPowerupProgress.queryInterfaceTo(aProgress);
+            AssertComRCReturnRC(hrc);
         }
 
@@ -8668 +8667 @@
 #endif
 
-        rc = task->createThread();
+        hrc = task->createThread();
         task = NULL;
-        if (FAILED(rc))
-            throw rc;
+        if (FAILED(hrc))
+            throw hrc;
 
         /* finally, set the state: no right to fail in this method afterwards
@@ -8685 +8684 @@
     catch (HRESULT aRC)
     {
-        rc = aRC;
-    }
-
-    if (FAILED(rc) && fBeganPoweringUp)
+        hrc = aRC;
+    }
+
+    if (FAILED(hrc) && fBeganPoweringUp)
     {
 
         /* The progress object will fetch the current error info */
         if (!pPowerupProgress.isNull())
-            pPowerupProgress->i_notifyComplete(rc);
+            pPowerupProgress->i_notifyComplete(hrc);
 
         /* Save the error info across the IPC below. Can't be done before the
@@ -8702 +8701 @@
 
         /* signal end of operation */
-        mControl->EndPowerUp(rc);
+        mControl->EndPowerUp(hrc);
     }
 
@@ -8711 +8710 @@
     }
 
-    LogFlowThisFunc(("mMachineState=%d, rc=%Rhrc\n", mMachineState, rc));
+    LogFlowThisFunc(("mMachineState=%d, hrc=%Rhrc\n", mMachineState, hrc));
     LogFlowThisFuncLeave();
-    return rc;
+    return hrc;
 }
 
@@ -8761 +8760 @@
     ULONG step = 0;
 
-    HRESULT rc = S_OK;
+    HRESULT hrc = S_OK;
     int vrc = VINF_SUCCESS;
 
@@ -8992 +8991 @@
             mpUVM = pUVM;
             pUVM = NULL;
-            rc = setErrorBoth(VBOX_E_VM_ERROR, vrc, tr("Could not destroy the machine. (Error: %Rrc)"), vrc);
+            hrc = setErrorBoth(VBOX_E_VM_ERROR, vrc, tr("Could not destroy the machine. (Error: %Rrc)"), vrc);
         }
 
@@ -9008 +9007 @@
     }
     else
-        rc = setErrorBoth(VBOX_E_VM_ERROR, vrc, tr("Could not power off the machine. (Error: %Rrc)"), vrc);
+        hrc = setErrorBoth(VBOX_E_VM_ERROR, vrc, tr("Could not power off the machine. (Error: %Rrc)"), vrc);
 
     /*
@@ -9024 +9023 @@
 
     LogFlowThisFuncLeave();
-    return rc;
+    return hrc;
 }
 
@@ -9037 +9036 @@
     AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
 
-    HRESULT rc = S_OK;
+    HRESULT hrc = S_OK;
 
     if (mMachineState != aMachineState)
@@ -9073 +9072 @@
              */
             LogFlowThisFunc(("Doing mControl->UpdateState()...\n"));
-            rc = mControl->UpdateState(aMachineState);
-            LogFlowThisFunc(("mControl->UpdateState()=%Rhrc\n", rc));
-        }
-    }
-
-    return rc;
+            hrc = mControl->UpdateState(aMachineState);
+            LogFlowThisFunc(("mControl->UpdateState()=%Rhrc\n", hrc));
+        }
+    }
+
+    return hrc;
 }
 
@@ -9133 +9132 @@
                      && m_pVMMDev->isShFlActive();
 
-    HRESULT rc = S_OK;
+    HRESULT hrc = S_OK;
 
     try
@@ -9150 +9149 @@
 
             SafeIfaceArray<ISharedFolder> folders;
-            rc = mMachine->COMGETTER(SharedFolders)(ComSafeArrayAsOutParam(folders));
-            if (FAILED(rc)) throw rc;
+            hrc = mMachine->COMGETTER(SharedFolders)(ComSafeArrayAsOutParam(folders));
+            if (FAILED(hrc)) throw hrc;
 
             for (size_t i = 0; i < folders.size(); ++i)
@@ -9158 +9157 @@
 
                 Bstr bstr;
-                rc = pSharedFolder->COMGETTER(Name)(bstr.asOutParam());
-                if (FAILED(rc)) throw rc;
+                hrc = pSharedFolder->COMGETTER(Name)(bstr.asOutParam());
+                if (FAILED(hrc)) throw hrc;
                 Utf8Str strName(bstr);
 
-                rc = pSharedFolder->COMGETTER(HostPath)(bstr.asOutParam());
-                if (FAILED(rc)) throw rc;
+                hrc = pSharedFolder->COMGETTER(HostPath)(bstr.asOutParam());
+                if (FAILED(hrc)) throw hrc;
                 Utf8Str strHostPath(bstr);
 
                 BOOL writable;
-                rc = pSharedFolder->COMGETTER(Writable)(&writable);
-                if (FAILED(rc)) throw rc;
+                hrc = pSharedFolder->COMGETTER(Writable)(&writable);
+                if (FAILED(hrc)) throw hrc;
 
                 BOOL autoMount;
-                rc = pSharedFolder->COMGETTER(AutoMount)(&autoMount);
-                if (FAILED(rc)) throw rc;
-
-                rc = pSharedFolder->COMGETTER(AutoMountPoint)(bstr.asOutParam());
-                if (FAILED(rc)) throw rc;
+                hrc = pSharedFolder->COMGETTER(AutoMount)(&autoMount);
+                if (FAILED(hrc)) throw hrc;
+
+                hrc = pSharedFolder->COMGETTER(AutoMountPoint)(bstr.asOutParam());
+                if (FAILED(hrc)) throw hrc;
                 Utf8Str strAutoMountPoint(bstr);
 
@@ -9201 +9200 @@
                                )
                             {
-                                rc = i_removeSharedFolder(strName);
-                                if (FAILED(rc)) throw rc;
+                                hrc = i_removeSharedFolder(strName);
+                                if (FAILED(hrc)) throw hrc;
                             }
 
                             /* create the new machine folder */
-                            rc = i_createSharedFolder(strName,
-                                                      SharedFolderData(strHostPath, !!writable, !!autoMount, strAutoMountPoint));
-                            if (FAILED(rc)) throw rc;
+                            hrc = i_createSharedFolder(strName,
+                                                       SharedFolderData(strHostPath, !!writable, !!autoMount, strAutoMountPoint));
+                            if (FAILED(hrc)) throw hrc;
                         }
                     }
@@ -9228 +9227 @@
                     {
                         /* remove the outdated machine folder */
-                        rc = i_removeSharedFolder(it->first);
-                        if (FAILED(rc)) throw rc;
+                        hrc = i_removeSharedFolder(it->first);
+                        if (FAILED(hrc)) throw hrc;
 
                         /* create the global folder if there is any */
@@ -9236 +9235 @@
                         if (git != m_mapGlobalSharedFolders.end())
                         {
-                            rc = i_createSharedFolder(git->first, git->second);
-                            if (FAILED(rc)) throw rc;
+                            hrc = i_createSharedFolder(git->first, git->second);
+                            if (FAILED(hrc)) throw hrc;
                         }
                     }
@@ -9244 +9243 @@
         }
     }
-    catch (HRESULT rc2)
-    {
-        rc = rc2;
+    catch (HRESULT hrc2)
+    {
+        hrc = hrc2;
         if (online)
             i_atVMRuntimeErrorCallbackF(0, "BrokenSharedFolder", N_("Broken shared folder!"));
@@ -9253 +9252 @@
     LogFlowThisFunc(("Leaving\n"));
 
-    return rc;
+    return hrc;
 }
 
@@ -9644 +9643 @@
                  */
                 ComPtr<IProgress> pProgress;
-                HRESULT rc = that->mControl->BeginPoweringDown(pProgress.asOutParam());
-                AssertComRC(rc);
+                HRESULT hrc = that->mControl->BeginPoweringDown(pProgress.asOutParam());
+                AssertComRC(hrc);
 
                 /* sync the state with the server */
@@ -9664 +9663 @@
                 {
                     LogRelFunc(("E_OUTOFMEMORY creating VMPowerDownTask"));
-                    rc = E_OUTOFMEMORY;
+                    hrc = E_OUTOFMEMORY;
                     break;
                 }
@@ -9677 +9676 @@
                 if (pTask->isOk())
                 {
-                    rc = pTask->createThread();
+                    hrc = pTask->createThread();
                     pTask = NULL;
-                    if (FAILED(rc))
+                    if (FAILED(hrc))
                         LogRelFunc(("Problem with creating thread for VMPowerDownTask.\n"));
                 }
@@ -9687 +9686 @@
                     delete pTask;
                     pTask = NULL;
-                    rc = E_FAIL;
+                    hrc = E_FAIL;
                 }
             }
@@ -9716 +9715 @@
             {
                 ComPtr<IVirtualBox> pVirtualBox;
-                HRESULT rc = that->mMachine->COMGETTER(Parent)(pVirtualBox.asOutParam());
-                AssertComRC(rc);
-                if (SUCCEEDED(rc) && !pVirtualBox.isNull())
+                HRESULT hrc = that->mMachine->COMGETTER(Parent)(pVirtualBox.asOutParam());
+                AssertComRC(hrc);
+                if (SUCCEEDED(hrc) && !pVirtualBox.isNull())
                     stopCloudGateway(pVirtualBox, that->mGateway);
             }
@@ -10044 +10043 @@
     }
 
-    int rc = pVMMDev->hgcmHostCall("VBoxDragAndDropSvc", DragAndDropSvc::HOST_DND_FN_SET_MODE, 1 /* cParms */, &parm);
-    if (RT_FAILURE(rc))
-        LogRel(("Error changing drag and drop mode: %Rrc\n", rc));
-
-    return rc;
+    int vrc = pVMMDev->hgcmHostCall("VBoxDragAndDropSvc", DragAndDropSvc::HOST_DND_FN_SET_MODE, 1 /* cParms */, &parm);
+    if (RT_FAILURE(vrc))
+        LogRel(("Error changing drag and drop mode: %Rrc\n", vrc));
+
+    return vrc;
 }
 
@@ -10328 +10327 @@
 # endif /* VBOX_STRICT */
 
-    HRESULT rc = S_OK;
-
     ULONG slot = 0;
-    rc = networkAdapter->COMGETTER(Slot)(&slot);
-    AssertComRC(rc);
+    HRESULT hrc = networkAdapter->COMGETTER(Slot)(&slot);
+    AssertComRCReturnRC(hrc);
 
 # ifdef RT_OS_LINUX
@@ -10349 +10346 @@
         /* The name of the TAP interface we are using */
         Bstr tapDeviceName;
-        rc = networkAdapter->COMGETTER(BridgedInterface)(tapDeviceName.asOutParam());
-        if (FAILED(rc))
+        hrc = networkAdapter->COMGETTER(BridgedInterface)(tapDeviceName.asOutParam());
+        if (FAILED(hrc))
             tapDeviceName.setNull(); /* Is this necessary? */
         if (tapDeviceName.isEmpty())
         {
             LogRel(("No TAP device name was supplied.\n"));
-            rc = setError(E_FAIL, tr("No TAP device name was supplied for the host networking interface"));
-        }
-
-        if (SUCCEEDED(rc))
+            hrc = setError(E_FAIL, tr("No TAP device name was supplied for the host networking interface"));
+        }
+
+        if (SUCCEEDED(hrc))
         {
             /* If we are using a static TAP device then try to open it. */
@@ -10368 +10365 @@
             {
                 LogRel(("Failed to open the host network interface %ls\n", tapDeviceName.raw()));
-                rc = setErrorBoth(E_FAIL, vrc, tr("Failed to open the host network interface %ls"), tapDeviceName.raw());
+                hrc = setErrorBoth(E_FAIL, vrc, tr("Failed to open the host network interface %ls"), tapDeviceName.raw());
             }
         }
-        if (SUCCEEDED(rc))
+        if (SUCCEEDED(hrc))
         {
             /*
@@ -10393 +10390 @@
                 LogRel(("Configuration error: Failed to configure /dev/net/tun non blocking. Error: %s\n", strerror(iErr)));
                 vrc = VERR_HOSTIF_BLOCKING;
-                rc = setErrorBoth(E_FAIL, vrc, tr("could not set up the host networking device for non blocking access: %s"),
-                                  strerror(errno));
+                hrc = setErrorBoth(E_FAIL, vrc, tr("could not set up the host networking device for non blocking access: %s"),
+                                   strerror(errno));
             }
         }
@@ -10400 +10397 @@
     else
     {
-        LogRel(("Configuration error: Failed to open /dev/net/tun rc=%Rrc\n", vrc));
+        LogRel(("Configuration error: Failed to open /dev/net/tun vrc=%Rrc\n", vrc));
         switch (vrc)
         {
             case VERR_ACCESS_DENIED:
                 /* will be handled by our caller */
-                rc = E_ACCESSDENIED;
+                hrc = E_ACCESSDENIED;
                 break;
             default:
-                rc = setErrorBoth(E_FAIL, vrc, tr("Could not set up the host networking device: %Rrc"), vrc);
+                hrc = setErrorBoth(E_FAIL, vrc, tr("Could not set up the host networking device: %Rrc"), vrc);
                 break;
         }
@@ -10419 +10416 @@
     /* The name of the TAP interface we are using */
     Bstr tapDeviceName;
-    rc = networkAdapter->COMGETTER(BridgedInterface)(tapDeviceName.asOutParam());
-    if (FAILED(rc))
+    hrc = networkAdapter->COMGETTER(BridgedInterface)(tapDeviceName.asOutParam());
+    if (FAILED(hrc))
         tapDeviceName.setNull(); /* Is this necessary? */
     if (tapDeviceName.isEmpty())
     {
         LogRel(("No TAP device name was supplied.\n"));
-        rc = setError(E_FAIL, tr("No TAP device name was supplied for the host networking interface"));
+        hrc = setError(E_FAIL, tr("No TAP device name was supplied for the host networking interface"));
     }
     char szTapdev[1024] = "/dev/";
@@ -10446 +10443 @@
             case VERR_ACCESS_DENIED:
                 /* will be handled by our caller */
-                rc = E_ACCESSDENIED;
+                hrc = E_ACCESSDENIED;
                 break;
             default:
-                rc = setErrorBoth(E_FAIL, vrc, tr("Failed to open the host network interface %ls"), tapDeviceName.raw());
+                hrc = setErrorBoth(E_FAIL, vrc, tr("Failed to open the host network interface %ls"), tapDeviceName.raw());
                 break;
         }
@@ -10457 +10454 @@
 # endif
     /* in case of failure, cleanup. */
-    if (RT_FAILURE(vrc) && SUCCEEDED(rc))
+    if (RT_FAILURE(vrc) && SUCCEEDED(hrc))
     {
         LogRel(("General failure attaching to host interface\n"));
-        rc = setErrorBoth(E_FAIL, vrc, tr("General failure attaching to host interface"));
-    }
-    LogFlowThisFunc(("rc=%Rhrc\n", rc));
-    return rc;
+        hrc = setErrorBoth(E_FAIL, vrc, tr("General failure attaching to host interface"));
+    }
+    LogFlowThisFunc(("hrc=%Rhrc\n", hrc));
+    return hrc;
 }
 
@@ -10483 +10480 @@
     AssertReturn(isWriteLockOnCurrentThread(), E_FAIL);
 
-    HRESULT rc = S_OK;
 # ifdef VBOX_STRICT
     /* paranoia */
@@ -10492 +10488 @@
 
     ULONG slot = 0;
-    rc = networkAdapter->COMGETTER(Slot)(&slot);
-    AssertComRC(rc);
+    HRESULT hrc = networkAdapter->COMGETTER(Slot)(&slot);
+    AssertComRCReturnRC(hrc);
 
     /* is there an open TAP device? */
@@ -10503 +10499 @@
         Bstr tapDeviceName, tapTerminateApplication;
         bool isStatic = true;
-        rc = networkAdapter->COMGETTER(BridgedInterface)(tapDeviceName.asOutParam());
-        if (FAILED(rc) || tapDeviceName.isEmpty())
+        hrc = networkAdapter->COMGETTER(BridgedInterface)(tapDeviceName.asOutParam());
+        if (FAILED(hrc) || tapDeviceName.isEmpty())
         {
             /* If the name is empty, this is a dynamic TAP device, so close it now,
@@ -10510 +10506 @@
                need the FD to pass to the termination script. */
             isStatic = false;
-            int rcVBox = RTFileClose(maTapFD[slot]);
-            AssertRC(rcVBox);
+            int vrc = RTFileClose(maTapFD[slot]);
+            AssertRC(vrc);
             maTapFD[slot] = NIL_RTFILE;
         }
@@ -10518 +10514 @@
             /* If we are using a static TAP device, we close it now, after having called the
                termination script. */
-            int rcVBox = RTFileClose(maTapFD[slot]);
-            AssertRC(rcVBox);
+            int vrc = RTFileClose(maTapFD[slot]);
+            AssertRC(vrc);
         }
         /* the TAP device name and handle are no longer valid */
@@ -10525 +10521 @@
         maTAPDeviceName[slot] = "";
     }
-    LogFlowThisFunc(("returning %d\n", rc));
-    return rc;
+    LogFlowThisFunc(("returning %Rhrc\n", hrc));
+    return hrc;
 }
 
@@ -10546 +10542 @@
      * host interface termination handling
      */
-    HRESULT rc = S_OK;
     ComPtr<IVirtualBox> pVirtualBox;
     mMachine->COMGETTER(Parent)(pVirtualBox.asOutParam());
@@ -10558 +10553 @@
         pSystemProperties->GetMaxNetworkAdapters(chipsetType, &maxNetworkAdapters);
 
+    HRESULT hrc = S_OK;
     for (ULONG slot = 0; slot < maxNetworkAdapters; slot++)
     {
         ComPtr<INetworkAdapter> pNetworkAdapter;
-        rc = mMachine->GetNetworkAdapter(slot, pNetworkAdapter.asOutParam());
-        if (FAILED(rc)) break;
+        hrc = mMachine->GetNetworkAdapter(slot, pNetworkAdapter.asOutParam());
+        if (FAILED(hrc)) break;
 
         BOOL enabled = FALSE;
@@ -10574 +10570 @@
         {
 #if ((defined(RT_OS_LINUX) || defined(RT_OS_FREEBSD)) && !defined(VBOX_WITH_NETFLT))
-            HRESULT rc2 = i_detachFromTapInterface(pNetworkAdapter);
-            if (FAILED(rc2) && SUCCEEDED(rc))
-                rc = rc2;
+            HRESULT hrc2 = i_detachFromTapInterface(pNetworkAdapter);
+            if (FAILED(hrc2) && SUCCEEDED(hrc))
+                hrc = hrc2;
 #endif /* (RT_OS_LINUX || RT_OS_FREEBSD) && !VBOX_WITH_NETFLT */
         }
     }
 
-    return rc;
+    return hrc;
 }
 
@@ -10618 +10614 @@
  */
 /*static*/ DECLCALLBACK(void)
-Console::i_genericVMSetErrorCallback(PUVM pUVM, void *pvUser, int rc, RT_SRC_POS_DECL, const char *pszFormat, va_list args)
+Console::i_genericVMSetErrorCallback(PUVM pUVM, void *pvUser, int vrc, RT_SRC_POS_DECL, const char *pszFormat, va_list args)
 {
     RT_SRC_POS_NOREF();
@@ -10632 +10628 @@
     {
         if (pErrorText->length())
-            pErrorText->appendPrintf(".\n%N (%Rrc)", pszFormat, &va2, rc, rc);
+            pErrorText->appendPrintf(".\n%N (%Rrc)", pszFormat, &va2, vrc, vrc);
         else
-            pErrorText->printf("%N (%Rrc)", pszFormat, &va2, rc, rc);
+            pErrorText->printf("%N (%Rrc)", pszFormat, &va2, vrc, vrc);
     }
     catch (std::bad_alloc &)
@@ -10925 +10921 @@
     VirtualBoxBase::initializeComForThread();
 
-    HRESULT rc = S_OK;
+    HRESULT hrc = S_OK;
     int vrc = VINF_SUCCESS;
 
@@ -10962 +10958 @@
              it != pTask->hardDiskProgresses.end(); ++it)
         {
-            HRESULT rc2 = (*it)->WaitForCompletion(-1);
-            AssertComRC(rc2);
-
-            rc = pTask->mProgress->SetNextOperation(BstrFmt(tr("Disk Image Reset Operation - Immutable Image")).raw(), 1);
-            AssertComRCReturnVoid(rc);
+            HRESULT hrc2 = (*it)->WaitForCompletion(-1);
+            AssertComRC(hrc2);
+
+            hrc = pTask->mProgress->SetNextOperation(BstrFmt(tr("Disk Image Reset Operation - Immutable Image")).raw(), 1);
+            AssertComRCReturnVoid(hrc);
         }
 
@@ -10979 +10975 @@
         if (!pTask->mTeleporterEnabled)
         {
-            rc = pConsole->mControl->LockMedia();
-            if (FAILED(rc)) throw rc;
+            hrc = pConsole->mControl->LockMedia();
+            if (FAILED(hrc)) throw hrc;
         }
 
@@ -11073 +11069 @@
                     {
                         const SharedFolderData &d = it->second;
-                        rc = pConsole->i_createSharedFolder(it->first, d);
-                        if (FAILED(rc))
+                        hrc = pConsole->i_createSharedFolder(it->first, d);
+                        if (FAILED(hrc))
                         {
                             ErrorInfoKeeper eik;
@@ -11084 +11080 @@
                         }
                     }
-                    if (FAILED(rc))
-                        rc = S_OK;          // do not fail with broken shared folders
+                    if (FAILED(hrc))
+                        hrc = S_OK;          // do not fail with broken shared folders
 
                     /* acquire the lock again */
@@ -11098 +11094 @@
                 {
                     BOOL fVRDEEnabled = FALSE;
-                    rc = pConsole->i_getVRDEServer()->COMGETTER(Enabled)(&fVRDEEnabled);
-                    AssertComRCBreak(rc, RT_NOTHING);
+                    hrc = pConsole->i_getVRDEServer()->COMGETTER(Enabled)(&fVRDEEnabled);
+                    AssertComRCBreak(hrc, RT_NOTHING);
 
                     if (   fVRDEEnabled
@@ -11119 +11115 @@
                 {
                     ComPtr<IRecordingSettings> ptrRecordingSettings;
-                    rc = pConsole->mMachine->COMGETTER(RecordingSettings)(ptrRecordingSettings.asOutParam());
-                    AssertComRCBreak(rc, RT_NOTHING);
-
-                    rc = ptrRecordingSettings->COMGETTER(Enabled)(&fRecordingEnabled);
-                    AssertComRCBreak(rc, RT_NOTHING);
+                    hrc = pConsole->mMachine->COMGETTER(RecordingSettings)(ptrRecordingSettings.asOutParam());
+                    AssertComRCBreak(hrc, RT_NOTHING);
+
+                    hrc = ptrRecordingSettings->COMGETTER(Enabled)(&fRecordingEnabled);
+                    AssertComRCBreak(hrc, RT_NOTHING);
                 }
                 if (fRecordingEnabled)
@@ -11144 +11140 @@
                  * Capture USB devices.
                  */
-                rc = pConsole->i_captureUSBDevices(pConsole->mpUVM);
-                if (FAILED(rc))
+                hrc = pConsole->i_captureUSBDevices(pConsole->mpUVM);
+                if (FAILED(hrc))
                 {
                     alock.acquire();
@@ -11211 +11207 @@
                     /* -> ConsoleImplTeleporter.cpp */
                     bool fPowerOffOnFailure;
-                    rc = pConsole->i_teleporterTrg(pConsole->mpUVM, pConsole->mpVMM, pMachine, &pTask->mErrorMsg,
-                                                   pTask->mStartPaused, pTask->mProgress, &fPowerOffOnFailure);
-                    if (FAILED(rc) && fPowerOffOnFailure)
+                    hrc = pConsole->i_teleporterTrg(pConsole->mpUVM, pConsole->mpVMM, pMachine, &pTask->mErrorMsg,
+                                                    pTask->mStartPaused, pTask->mProgress, &fPowerOffOnFailure);
+                    if (FAILED(hrc) && fPowerOffOnFailure)
                     {
                         ErrorInfoKeeper eik;
@@ -11242 +11238 @@
 
             /* On failure, destroy the VM */
-            if (FAILED(rc) || RT_FAILURE(vrc))
+            if (FAILED(hrc) || RT_FAILURE(vrc))
             {
                 /* preserve existing error info */
@@ -11250 +11246 @@
                  * cleanup (VRDP, USB devices) */
                 alock.release();
-                HRESULT rc2 = pConsole->i_powerDown();
+                HRESULT hrc2 = pConsole->i_powerDown();
                 alock.acquire();
-                AssertComRC(rc2);
+                AssertComRC(hrc2);
             }
             else
@@ -11282 +11278 @@
         }
 
-        if (SUCCEEDED(rc) && RT_FAILURE(vrc))
+        if (SUCCEEDED(hrc) && RT_FAILURE(vrc))
         {
             /* If VMR3Create() or one of the other calls in this function fail,
              * an appropriate error message has been set in pTask->mErrorMsg.
-             * However since that happens via a callback, the rc status code in
+             * However since that happens via a callback, the hrc status code in
              * this function is not updated.
              */
@@ -11305 +11301 @@
         }
     }
-    catch (HRESULT aRC) { rc = aRC; }
+    catch (HRESULT hrcXcpt) { hrc = hrcXcpt; }
 
     if (   pConsole->mMachineState == MachineState_Starting
@@ -11337 +11333 @@
     alock.release();
 
-    if (SUCCEEDED(rc))
+    if (SUCCEEDED(hrc))
     {
         /* Notify the progress object of the success */
@@ -11345 +11341 @@
     {
         /* The progress object will fetch the current error info */
-        pTask->mProgress->i_notifyComplete(rc);
-        LogRel(("Power up failed (vrc=%Rrc, rc=%Rhrc (%#08X))\n", vrc, rc, rc));
+        pTask->mProgress->i_notifyComplete(hrc);
+        LogRel(("Power up failed (vrc=%Rrc, hrc=%Rhrc (%#08X))\n", vrc, hrc, hrc));
     }
 
     /* Notify VBoxSVC and any waiting openRemoteSession progress object. */
-    pConsole->mControl->EndPowerUp(rc);
+    pConsole->mControl->EndPowerUp(hrc);
 
 #if defined(RT_OS_WINDOWS)
@@ -11414 +11410 @@
 
     /* Update the device instance configuration. */
-    int rc = pThis->i_configMediumAttachment(pcszDevice,
-                                             uInstance,
-                                             enmBus,
-                                             fUseHostIOCache,
-                                             fBuiltinIOCache,
-                                             fInsertDiskIntegrityDrv,
-                                             fSetupMerge,
-                                             uMergeSource,
-                                             uMergeTarget,
-                                             aMediumAtt,
-                                             aMachineState,
-                                             phrc,
-                                             true /* fAttachDetach */,
-                                             false /* fForceUnmount */,
-                                             false /* fHotplug */,
-                                             pUVM,
-                                             pVMM,
-                                             NULL /* paLedDevType */,
-                                             NULL /* ppLunL0)*/);
-    if (RT_FAILURE(rc))
-    {
-        AssertMsgFailed(("rc=%Rrc\n", rc));
-        return rc;
+    int vrc = pThis->i_configMediumAttachment(pcszDevice,
+                                              uInstance,
+                                              enmBus,
+                                              fUseHostIOCache,
+                                              fBuiltinIOCache,
+                                              fInsertDiskIntegrityDrv,
+                                              fSetupMerge,
+                                              uMergeSource,
+                                              uMergeTarget,
+                                              aMediumAtt,
+                                              aMachineState,
+                                              phrc,
+                                              true /* fAttachDetach */,
+                                              false /* fForceUnmount */,
+                                              false /* fHotplug */,
+                                              pUVM,
+                                              pVMM,
+                                              NULL /* paLedDevType */,
+                                              NULL /* ppLunL0)*/);
+    if (RT_FAILURE(vrc))
+    {
+        AssertMsgFailed(("vrc=%Rrc\n", vrc));
+        return vrc;
     }
 
@@ -11455 +11451 @@
 void Console::i_powerDownThreadTask(VMPowerDownTask *pTask)
 {
-    int rc = VINF_SUCCESS; /* only used in assertion */
+    int vrc = VINF_SUCCESS; /* only used in assertion */
     LogFlowFuncEnter();
     try
     {
         if (pTask->isOk() == false)
-            rc = VERR_GENERAL_FAILURE;
+            vrc = VERR_GENERAL_FAILURE;
 
         const ComObjPtr<Console> &that = pTask->mConsole;
@@ -11483 +11479 @@
     catch (const std::exception &e)
     {
-        AssertMsgFailed(("Exception %s was caught, rc=%Rrc\n", e.what(), rc));
-        NOREF(e); NOREF(rc);
+        AssertMsgFailed(("Exception %s was caught, vrc=%Rrc\n", e.what(), vrc));
+        NOREF(e); NOREF(vrc);
     }
 
@@ -11654 +11650 @@
     SecretKey *pKey = NULL;
 
-    int rc = pConsole->m_pKeyStore->retainSecretKey(Utf8Str(pszId), &pKey);
-    if (RT_SUCCESS(rc))
+    int vrc = pConsole->m_pKeyStore->retainSecretKey(Utf8Str(pszId), &pKey);
+    if (RT_SUCCESS(vrc))
     {
         *ppbKey = (const uint8_t *)pKey->getKeyBuffer();
@@ -11661 +11657 @@
     }
 
-    return rc;
+    return vrc;
 }
 
@@ -11687 +11683 @@
     SecretKey *pKey = NULL;
 
-    int rc = pConsole->m_pKeyStore->retainSecretKey(Utf8Str(pszId), &pKey);
-    if (RT_SUCCESS(rc))
+    int vrc = pConsole->m_pKeyStore->retainSecretKey(Utf8Str(pszId), &pKey);
+    if (RT_SUCCESS(vrc))
         *ppszPassword = (const char *)pKey->getKeyBuffer();
 
-    return rc;
+    return vrc;
 }
 
@@ -11775 +11771 @@
          */
         PPDMLED pLed;
-        int rc = pThis->pLedPorts->pfnQueryStatusLed(pThis->pLedPorts, iLUN, &pLed);
-        if (RT_FAILURE(rc))
+        int vrc = pThis->pLedPorts->pfnQueryStatusLed(pThis->pLedPorts, iLUN, &pLed);
+        if (RT_FAILURE(vrc))
             pLed = NULL;
 
@@ -11824 +11820 @@
         {
             IMedium *pMedium = NULL;
-            HRESULT rc = pMediumAtt->COMGETTER(Medium)(&pMedium);
-            AssertComRC(rc);
-            if (SUCCEEDED(rc) && pMedium)
+            HRESULT hrc = pMediumAtt->COMGETTER(Medium)(&pMedium);
+            AssertComRC(hrc);
+            if (SUCCEEDED(hrc) && pMedium)
             {
                 BOOL fHostDrive = FALSE;
-                rc = pMedium->COMGETTER(HostDrive)(&fHostDrive);
-                AssertComRC(rc);
+                hrc = pMedium->COMGETTER(HostDrive)(&fHostDrive);
+                AssertComRC(hrc);
                 if (!fHostDrive)
                 {
@@ -11836 +11832 @@
 
                     ComPtr<IMediumAttachment> pNewMediumAtt;
-                    rc = pThis->pConsole->mControl->EjectMedium(pMediumAtt, pNewMediumAtt.asOutParam());
-                    if (SUCCEEDED(rc))
+                    hrc = pThis->pConsole->mControl->EjectMedium(pMediumAtt, pNewMediumAtt.asOutParam());
+                    if (SUCCEEDED(hrc))
                     {
                         pThis->pConsole->mMachine->SaveSettings();
@@ -11942 +11938 @@
 
     uint32_t iLedSet;
-    int rc = pHlp->pfnCFGMQueryU32(pCfg, "iLedSet", &iLedSet);
-    AssertLogRelMsgRCReturn(rc, ("Configuration error: Failed to query the \"iLedSet\" value! rc=%Rrc\n", rc), rc);
+    int vrc = pHlp->pfnCFGMQueryU32(pCfg, "iLedSet", &iLedSet);
+    AssertLogRelMsgRCReturn(vrc, ("Configuration error: Failed to query the \"iLedSet\" value! vrc=%Rrc\n", vrc), vrc);
     pThis->papLeds = pConsole->i_getLedSet(iLedSet);
 
-    rc = pHlp->pfnCFGMQueryBoolDef(pCfg, "HasMediumAttachments", &pThis->fHasMediumAttachments, false);
-    AssertLogRelMsgRCReturn(rc, ("Configuration error: Failed to query the \"HasMediumAttachments\" value! rc=%Rrc\n", rc), rc);
+    vrc = pHlp->pfnCFGMQueryBoolDef(pCfg, "HasMediumAttachments", &pThis->fHasMediumAttachments, false);
+    AssertLogRelMsgRCReturn(vrc, ("Configuration error: Failed to query the \"HasMediumAttachments\" value! vrc=%Rrc\n", vrc), vrc);
 
     if (pThis->fHasMediumAttachments)
     {
-        rc = pHlp->pfnCFGMQueryStringAlloc(pCfg, "DeviceInstance", &pThis->pszDeviceInstance);
-        AssertLogRelMsgRCReturn(rc, ("Configuration error: Failed to query the \"DeviceInstance\" value! rc=%Rrc\n", rc), rc);
-    }
-
-    rc = pHlp->pfnCFGMQueryU32Def(pCfg, "First", &pThis->iFirstLUN, 0);
-    AssertLogRelMsgRCReturn(rc, ("Configuration error: Failed to query the \"First\" value! rc=%Rrc\n", rc), rc);
-
-    rc = pHlp->pfnCFGMQueryU32Def(pCfg, "Last", &pThis->iLastLUN, 0);
-    AssertLogRelMsgRCReturn(rc, ("Configuration error: Failed to query the \"Last\" value! rc=%Rrc\n", rc), rc);
+        vrc = pHlp->pfnCFGMQueryStringAlloc(pCfg, "DeviceInstance", &pThis->pszDeviceInstance);
+        AssertLogRelMsgRCReturn(vrc, ("Configuration error: Failed to query the \"DeviceInstance\" value! vrc=%Rrc\n", vrc), vrc);
+    }
+
+    vrc = pHlp->pfnCFGMQueryU32Def(pCfg, "First", &pThis->iFirstLUN, 0);
+    AssertLogRelMsgRCReturn(vrc, ("Configuration error: Failed to query the \"First\" value! vrc=%Rrc\n", vrc), vrc);
+
+    vrc = pHlp->pfnCFGMQueryU32Def(pCfg, "Last", &pThis->iLastLUN, 0);
+    AssertLogRelMsgRCReturn(vrc, ("Configuration error: Failed to query the \"Last\" value! vrc=%Rrc\n", vrc), vrc);
 
     AssertLogRelMsgReturn(pThis->iFirstLUN <= pThis->iLastLUN,