Changeset 103566 in vbox

Timestamp:

Feb 26, 2024 12:53:01 PM (11 months ago)

Author:

vboxsync

Message:

Audio/VKAT: Bugfixes for AudioTestBeaconAddConsecutive().

File:

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

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

@@ -2577 +2577 @@
             && pauBuf[i + 3] == byBeacon)
         {
+            offBeacon = i + cbStep; /* Point to data right *after* the beacon. */
+
             /* Make sure to handle overflows and let beacon start from scratch. */
             pBeacon->cbUsed = (pBeacon->cbUsed + cbStep) % pBeacon->cbSize;
@@ -2582 +2584 @@
             {
                 pBeacon->cbUsed = pBeacon->cbSize;
-                offBeacon       = i + cbStep; /* Point to data right *after* the beacon. */
+                break;
             }
         }
         else
         {
-            /* If beacon is not complete yet, we detected a gap here. Start all over then. */
-            if (RT_LIKELY(pBeacon->cbUsed != pBeacon->cbSize))
-                pBeacon->cbUsed = 0;
+            /* If beacon is complete, just skip, otherwise we detected a gap here. Start all over then. */
+            if (RT_UNLIKELY(pBeacon->cbUsed == pBeacon->cbSize))
+                break;
+
+            pBeacon->cbUsed = 0;
+            offBeacon       = UINT64_MAX;
         }
     }