Changeset 98454 in vbox

Timestamp:

Feb 2, 2023 7:38:19 PM (2 years ago)

Author:

vboxsync

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

155704

Message:

Audio/DrvAudio: Fixed wrong assumption in drvAudioStreamGetBackendState() when a stream gets initialized asynchronously by the backend (restores old behaviour). Follow-up fix for r155203. bugref:9558

File:

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

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

@@ -612 +612 @@
     if (pThis->pHostDrvAudio)
     {
-        /* Don't call if the backend wasn't created for this stream (disabled). */
-        if (pStreamEx->fStatus & PDMAUDIOSTREAM_STS_BACKEND_CREATED)
-        {
-            AssertPtrReturn(pThis->pHostDrvAudio->pfnStreamGetState, PDMHOSTAUDIOSTREAMSTATE_NOT_WORKING);
-            PDMHOSTAUDIOSTREAMSTATE enmState = pThis->pHostDrvAudio->pfnStreamGetState(pThis->pHostDrvAudio, pStreamEx->pBackend);
-            Log9Func(("%s: %s\n", pStreamEx->Core.Cfg.szName, PDMHostAudioStreamStateGetName(enmState) ));
-            Assert(   enmState > PDMHOSTAUDIOSTREAMSTATE_INVALID
-                   && enmState < PDMHOSTAUDIOSTREAMSTATE_END
-                   && (enmState != PDMHOSTAUDIOSTREAMSTATE_DRAINING || pStreamEx->Core.Cfg.enmDir == PDMAUDIODIR_OUT));
-            return enmState;
-        }
+        AssertPtrReturn(pThis->pHostDrvAudio->pfnStreamGetState, PDMHOSTAUDIOSTREAMSTATE_NOT_WORKING);
+        PDMHOSTAUDIOSTREAMSTATE enmState = pThis->pHostDrvAudio->pfnStreamGetState(pThis->pHostDrvAudio, pStreamEx->pBackend);
+        Log9Func(("%s: %s\n", pStreamEx->Core.Cfg.szName, PDMHostAudioStreamStateGetName(enmState) ));
+        Assert(   enmState > PDMHOSTAUDIOSTREAMSTATE_INVALID
+               && enmState < PDMHOSTAUDIOSTREAMSTATE_END
+               && (enmState != PDMHOSTAUDIOSTREAMSTATE_DRAINING || pStreamEx->Core.Cfg.enmDir == PDMAUDIODIR_OUT));
+        return enmState;
     }
     Log9Func(("%s: not-working\n", pStreamEx->Core.Cfg.szName));