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

Timestamp:

Dec 14, 2016 1:39:25 PM (8 years ago)

Author:

vboxsync

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

112268

Message:

Assertion / logging fixes.

File:

• trunk/src/VBox/Devices/Audio/DrvAudio.cpp (modified) (4 diffs)

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

@@ -768 +768 @@
     AssertPtrReturn(pStream,    VERR_INVALID_POINTER);
     AssertPtrReturn(pvBuf,      VERR_INVALID_POINTER);
-    AssertReturn(cbBuf,         VERR_INVALID_POINTER);
+    AssertReturn(cbBuf,         VERR_INVALID_PARAMETER);
     /* pcbWritten is optional. */
 
@@ -805 +805 @@
 
         PPDMAUDIOSTREAM pGstStream = pHstStream->pPair;
+        AssertPtr(pGstStream);
 
         AssertMsg(pGstStream->fStatus & PDMAUDIOSTRMSTS_FLAG_ENABLED,
@@ -815 +816 @@
             LogRel2(("Audio: Guest stream '%s' full, expect stuttering audio output\n", pGstStream->szName));
 #ifdef DEBUG_andy
-            AssertMsgFailed(("%s: Guest stream full\n", pGstStream->szName));
+            AssertMsgFailed(("%s: Guest stream full: cbBuf=%RU32\n", pGstStream->szName, cbBuf));
 #endif
             break;
@@ -848 +849 @@
             {
                 LogRel2(("Audio: Lost audio samples (%RU32) due to full host stream '%s', expect stuttering audio output\n",
-                         csWritten - csMixed, pGstStream->szName));
+                         csWritten - csMixed, pHstStream->szName));
             }