Changeset 89413 in vbox for trunk/src/VBox/ValidationKit/utils/audio/vkat.cpp

Timestamp:

May 31, 2021 8:16:30 PM (4 years ago)

Author:

vboxsync

Message:

ValKit/AudioTest: Make sure audioTestPlayOne reads a frame aligned number of bytes from the file or we'll get into trouble. bugref:10008

File:

• trunk/src/VBox/ValidationKit/utils/audio/vkat.cpp (modified) (1 diff)

trunk/src/VBox/ValidationKit/utils/audio/vkat.cpp

@@ -1604 +1604 @@
                 if (RT_SUCCESS(rc))
                 {
-                    uint64_t const nsStarted = RTTimeNanoTS();
+                    uint64_t const  nsStarted = RTTimeNanoTS();
 
                     /*
                      * Transfer data as quickly as we're allowed.
                      */
+                    uint8_t         abSamples[16384];
+                    uint32_t const  cbSamplesAligned = PDMAudioPropsFloorBytesToFrame(&pStream->Props, sizeof(abSamples));
                     for (;;)
                     {
                         /* Read a chunk from the wave file. */
-                        uint8_t  abSamples[16384];
                         size_t   cbSamples = 0;
-                        rc = AudioTestWaveFileRead(&WaveFile, abSamples, sizeof(abSamples), &cbSamples);
+                        rc = AudioTestWaveFileRead(&WaveFile, abSamples, cbSamplesAligned, &cbSamples);
                         if (RT_SUCCESS(rc) && cbSamples > 0)
                         {