Also use PulseAudio by default when pipewire-pulse is running

[Aroun]

This patch allows to use PulseAudio by default when pipewire-pulse is running, which is the pipewire in-place replacement for pulseaudio process. Instead of falling-back to ALSA.

diff --git a/src/VBox/Main/xml/Settings.cpp b/src/VBox/Main/xml/Settings.cpp
index 046ae0d..0c99559 100644
--- a/src/VBox/Main/xml/Settings.cpp
+++ b/src/VBox/Main/xml/Settings.cpp
@@ -8933,7 +8933,8 @@ AudioDriverType_T MachineConfigFile::getHostDefaultAudioDriver()
     {
 # ifdef VBOX_WITH_AUDIO_PULSE
         /* Check for the pulse library & that the pulse audio daemon is running. */
-        if (RTProcIsRunningByName("pulseaudio") &&
+        if (RTProcIsRunningByName("pulseaudio") ||
+           RTProcIsRunningByName("pipewire-pulse") &&
             RTLdrIsLoadable("libpulse.so.0"))
             s_enmLinuxDriver = AudioDriverType_Pulse;
         else

[Aroun]

Hum, actually, if the left part of the OR is true, then I think the && will not be evaluated, so we should add parenthesis

diff --git a/src/VBox/Main/xml/Settings.cpp b/src/VBox/Main/xml/Settings.cpp
index 046ae0d..0c99559 100644
--- a/src/VBox/Main/xml/Settings.cpp
+++ b/src/VBox/Main/xml/Settings.cpp
@@ -8933,7 +8933,8 @@ AudioDriverType_T MachineConfigFile::getHostDefaultAudioDriver()
     {
 # ifdef VBOX_WITH_AUDIO_PULSE
         /* Check for the pulse library & that the pulse audio daemon is running. */
-        if (RTProcIsRunningByName("pulseaudio") &&
+        if ((RTProcIsRunningByName("pulseaudio") ||
+           RTProcIsRunningByName("pipewire-pulse")) &&
             RTLdrIsLoadable("libpulse.so.0"))
             s_enmLinuxDriver = AudioDriverType_Pulse;
         else

[pentagonik]

Thanks for the patch! This will be available in the next upcoming maintenance version.

[galitsyn]

Hello,

We just released VirtualBox 7.0.10. This issue should be fixed in this release. Culd you please give it a try? Packages are available on our downloads page.