Changeset 88559 in vbox for trunk/src/VBox/Devices

Timestamp:

Apr 16, 2021 10:26:45 AM (4 years ago)

Author:

vboxsync

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

143854

Message:

DrvAudio: Need to return VERR_AUDIO_BACKEND_INIT_FAILED rather than VERR_FILE_NOT_FOUND/VERR_MODULE_NOT_FOUND when the lower driver fails to initalize due to missing libpulse.so.0 or similar. bugref:9890

File:

• trunk/src/VBox/Devices/Audio/DrvAudio.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Audio/DrvAudio.cpp

@@ -3786 +3786 @@
                                   N_("The host audio driver does not implement PDMIHOSTAUDIO!"));
         }
+    }
+
+    /*
+     * Do some status code simplification for beningn host driver init failures.
+     * The device above us will then replace it will the Null driver.
+     */
+    /** @todo Do the Null driver replacment here, then we don't have to duplicate
+     *        it in 3+ devices! */
+    if (   rc == VERR_MODULE_NOT_FOUND
+        || rc == VERR_SYMBOL_NOT_FOUND
+        || rc == VERR_FILE_NOT_FOUND
+        || rc == VERR_PATH_NOT_FOUND)
+    {
+        LogRel(("Audio: %Rrc -> VERR_AUDIO_BACKEND_INIT_FAILED\n", rc));
+        rc = VERR_AUDIO_BACKEND_INIT_FAILED;
     }