Changeset 67498 in vbox for trunk/src

Timestamp:

Jun 20, 2017 11:26:45 AM (8 years ago)

Author:

vboxsync

Message:

Audio/DrvAudio.cpp: Do not attempt to write more than available in drvAudioStreamWrite().

File:

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

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

@@ -940 +940 @@
                    pGstStream->szName, pszGstSts, pszHstSts));
 #endif
-        pGstStream->Out.tsLastWriteMS = RTTimeMilliTS();
-
-        if (!AudioMixBufFreeBytes(&pGstStream->MixBuf))
-        {
-            LogRel2(("Audio: Guest stream '%s' full, expect stuttering audio output\n", pGstStream->szName));
-
-#ifdef DEBUG_andy
-            AssertMsgFailed(("[%s] Failed because guest stream full (guest status %s, host status %s): cbBuf=%RU32\n",
-                             pGstStream->szName, pszGstSts, pszHstSts, cbBuf));
-#endif
-            break;
-        }
+        const uint32_t cbFree = AudioMixBufFreeBytes(&pGstStream->MixBuf);
+        if (!cbFree) /* No free guest side buffer space, bail out. */
+            break;
+
+        /* Do not attempt to write more than the guest side currently can handle. */
+        if (cbBuf > cbFree)
+            cbBuf = cbFree;
 
         uint32_t csWritten = 0;