diff options
author | Fraunhofer IIS FDK <audio-fdk@iis.fraunhofer.de> | 2020-07-06 16:37:38 +0200 |
---|---|---|
committer | Jean-Michel Trivi <jmtrivi@google.com> | 2020-08-10 19:57:03 +0000 |
commit | b5dfe8f92dd94e91f8391a9dc3d1fa7b0415ece2 (patch) | |
tree | 53705cee093f9ffd9699c4b1b8d07e546717d9ae /libMpegTPDec | |
parent | 792919ffb2928356b0877184833d1acda64c3518 (diff) | |
download | fdk-aac-b5dfe8f92dd94e91f8391a9dc3d1fa7b0415ece2.tar.gz fdk-aac-b5dfe8f92dd94e91f8391a9dc3d1fa7b0415ece2.tar.bz2 fdk-aac-b5dfe8f92dd94e91f8391a9dc3d1fa7b0415ece2.zip |
Fix heap buffer overflow in sbrDecoder_AssignQmfChannels2SbrChannels().
In the bug the SBR decoder has already set up 9 channels and tries to
allocate one more channel. The assignment of the QMF channels to SBR
channels fails since the QMF domain manages only 8+1 channels instead
of 10 channels as reqeusted by SBR.
Here we have added a check in sbrDecoder_InitElement() which will
return with a parse error in case additional SBR channels would exceed
the maximum number of SBR channels. This solves the potential heap
buffer overflow.
Bug: 158762825
Test: atest DecoderTestAacDrc DecoderTestAacFormat DecoderTestXheAac
Change-Id: I0150ac6d5a47ffce883010f531928656eebc619e
Diffstat (limited to 'libMpegTPDec')
0 files changed, 0 insertions, 0 deletions