Changeset 99559 in vbox for trunk/src/VBox/Devices/Audio

Timestamp:

Apr 28, 2023 2:30:50 PM (2 years ago)

Author:

vboxsync

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

157073

Message:

Audio: Improved range checks for audio channels (PDMAUDIO_MAX_CHANNELS).

File:

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

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

@@ -3276 +3276 @@
                                   uBuf.Wave.u.FmtExt.Data.fChannelMask,
                                   audioTestWaveCountBits(uBuf.Wave.u.FmtExt.Data.fChannelMask), uBuf.Wave.u.Fmt.Data.cChannels);
+                else if (uBuf.Wave.u.Fmt.Data.cChannels > PDMAUDIO_MAX_CHANNELS)
+                    RTErrInfoSetF(pErrInfo, rc, "More than %u channels are not supported (%u given)",
+                                  PDMAUDIO_MAX_CHANNELS, uBuf.Wave.u.Fmt.Data.cChannels);
                 else if (   uBuf.Wave.u.Fmt.Data.uFormatTag == RTRIFFWAVEFMT_TAG_EXTENSIBLE
                          && RTUuidCompareStr(&uBuf.Wave.u.FmtExt.Data.SubFormat, RTRIFFWAVEFMTEXT_SUBTYPE_PCM) != 0)
@@ -3297 +3300 @@
                                                           - (unsigned)PDMAUDIOCHANNELID_FIRST_STANDARD;
                         unsigned                iCh       = 0;
-                        for (unsigned idCh = 0; idCh < 32 && iCh < uBuf.Wave.u.Fmt.Data.cChannels; idCh++)
+                        for (unsigned idCh = 0; idCh < PDMAUDIO_MAX_CHANNELS && iCh < uBuf.Wave.u.Fmt.Data.cChannels; idCh++)
                             if (uBuf.Wave.u.FmtExt.Data.fChannelMask & RT_BIT_32(idCh))
                             {