aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJean-Michel Trivi <jmtrivi@google.com>2016-04-05 12:38:32 -0700
committerMarco Nelissen <marcone@google.com>2016-04-15 08:14:50 -0700
commit97a1b8140d410ed3942006aa22b40ccb322f747b (patch)
tree8c421956dd65ce2292ab691dd87ca42cb0b21b75
parenta06d1c2b9af1621037b48557aac42b5ecbdb03b3 (diff)
downloadfdk-aac-97a1b8140d410ed3942006aa22b40ccb322f747b.tar.gz
fdk-aac-97a1b8140d410ed3942006aa22b40ccb322f747b.tar.bz2
fdk-aac-97a1b8140d410ed3942006aa22b40ccb322f747b.zip
Fix aacDecoder_drcExtractAndMap()
Parse DVB DRC data only when numThreads is below MAX_DRC_THREADS. The post-increment is necessary as it is used in fill element DRC data section. This solution parses as many DRC payloads as allowed by MAX_DRC_THREADS and skips all remaining DRC elements in the stream. Bug 27792766 Bug 26751339 Change-Id: Ie1641888bac1757c4d1491119f977fc5d436eaea
-rw-r--r--libAACdec/src/aacdec_drc.cpp10
1 files changed, 1 insertions, 9 deletions
diff --git a/libAACdec/src/aacdec_drc.cpp b/libAACdec/src/aacdec_drc.cpp
index f939a1a..b836746 100644
--- a/libAACdec/src/aacdec_drc.cpp
+++ b/libAACdec/src/aacdec_drc.cpp
@@ -680,11 +680,7 @@ static int aacDecoder_drcExtractAndMap (
}
self->numPayloads = 0;
- if (self->numThreads >= MAX_DRC_THREADS) {
- self->numThreads = MAX_DRC_THREADS - 1;
- }
-
- if (self->dvbAncDataAvailable)
+ if (self->dvbAncDataAvailable && self->numThreads < MAX_DRC_THREADS)
{ /* Append a DVB heavy compression payload thread if available. */
int bitsParsed;
@@ -710,10 +706,6 @@ static int aacDecoder_drcExtractAndMap (
/* coupling channels not supported */
- if (self->numThreads >= MAX_DRC_THREADS) {
- self->numThreads = MAX_DRC_THREADS - 1;
- }
-
/* check for valid threads */
for (thread = 0; thread < self->numThreads; thread++) {
CDrcPayload *pThreadBs = &threadBs[thread];