Changeset 70903 in vbox for trunk/src/VBox/Devices/Audio/DevSB16.cpp

Timestamp:

Feb 8, 2018 10:07:07 AM (7 years ago)

Author:

vboxsync

Message:

Audio/DevSB16.cpp: Use sb16DestroyDrvStream() in sb16PowerOff() to avoid code duplication and disable the stream before destroying it.

File:

• trunk/src/VBox/Devices/Audio/DevSB16.cpp (modified) (2 diffs)

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

@@ -2356 +2356 @@
         pDrv->pConnector->pfnStreamRelease(pDrv->pConnector, pDrv->Out.pStream);
 
-        int rc2 = pDrv->pConnector->pfnStreamDestroy(pDrv->pConnector, pDrv->Out.pStream);
-        if (RT_SUCCESS(rc2))
-            pDrv->Out.pStream = NULL;
+        int rc2 = pDrv->pConnector->pfnStreamControl(pDrv->pConnector, pDrv->Out.pStream, PDMAUDIOSTREAMCMD_DISABLE);
+        AssertRC(rc2);
+
+        rc2 = pDrv->pConnector->pfnStreamDestroy(pDrv->pConnector, pDrv->Out.pStream);
+        AssertRC(rc2);
+
+        pDrv->Out.pStream = NULL;
     }
 }
@@ -2462 +2466 @@
     PSB16DRIVER pDrv;
     RTListForEach(&pThis->lstDrv, pDrv, SB16DRIVER, Node)
-    {
-        if (pDrv->Out.pStream)
-        {
-            pDrv->pConnector->pfnStreamRelease(pDrv->pConnector, pDrv->Out.pStream);
-
-            int rc2 = pDrv->pConnector->pfnStreamDestroy(pDrv->pConnector, pDrv->Out.pStream);
-            if (RT_SUCCESS(rc2))
-                pDrv->Out.pStream = NULL;
-        }
-    }
+        sb16DestroyDrvStream(pThis, pDrv);
 }