Changeset 90178 in vbox

Timestamp:

Jul 14, 2021 9:55:20 AM (4 years ago)

Author:

vboxsync

Message:

DevIchAc97: Don't assert on valid config in ichac97R3StreamSetUp as we do sometimes get here with Hz=0. bugref:9890

File:

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

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

@@ -2011 +2011 @@
      * Don't continue if the frequency is out of range (the rest of the
      * properties should be okay).
+     * Note! Don't assert on this as we may easily end up here with Hz=0.
      */
     char szTmp[PDMAUDIOSTRMCFGTOSTRING_MAX];
-    ASSERT_GUEST_MSG_RETURN(AudioHlpStreamCfgIsValid(&Cfg),
-                            ("Invalid stream #%u rate: %s\n", pStreamCC->u8SD,
-                             PDMAudioStrmCfgToString(&Cfg, szTmp, sizeof(szTmp)) ),
-                            VERR_OUT_OF_RANGE);
+    if (AudioHlpStreamCfgIsValid(&Cfg))
+    { }
+    else
+    {
+        LogFunc(("Invalid stream #%u rate: %s\n", pStreamCC->u8SD, PDMAudioStrmCfgToString(&Cfg, szTmp, sizeof(szTmp)) ));
+        return VERR_OUT_OF_RANGE;
+    }
 
     /*