aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Limit smoothedNoise values to avoid integer overflows in adjustTimeSlotHQ() ↵Fraunhofer IIS FDK2021-03-190-0/+0
|\ | | | | | | | | | | | | | | and adjustTimeSlotHQ_GainAndNoise(). am: ba54a5c040 am: d40cec83ed Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/13893570 Change-Id: I293c2bcb4de6aaf609ba067688f8574f732be6e8
| * Limit smoothedNoise values to avoid integer overflows in adjustTimeSlotHQ() ↵Fraunhofer IIS FDK2021-03-190-0/+0
| |\ | | | | | | | | | | | | | | | | | | | | | and adjustTimeSlotHQ_GainAndNoise(). am: ba54a5c040 Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/13893570 Change-Id: I84f177df0aaa55929682b56a7bf2902c4f5fb9e7
| | * Limit smoothedNoise values to avoid integer overflows in adjustTimeSlotHQ() ↵Fraunhofer IIS FDK2021-02-141-1/+18
| | | | | | | | | | | | | | | | | | | | | | | | and adjustTimeSlotHQ_GainAndNoise(). Bug: 176246647 Test: atest DecoderTestXheAac DecoderTestAacDrc Change-Id: Ib630d56a626ddd59a9155df38cda2011c3165346
* | | Avoid integer overflow in dct_II(). am: 19c8cc495e am: 7f707f8057Fraunhofer IIS FDK2021-03-190-0/+0
|\| | | | | | | | | | | | | | | | | Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/13893569 Change-Id: I9b2e017443215c646b67471a5c5babb4886b2a49
| * | Avoid integer overflow in dct_II(). am: 19c8cc495eFraunhofer IIS FDK2021-03-190-0/+0
| |\| | | | | | | | | | | | | | | | Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/13893569 Change-Id: I806a23071a940dbdb74d0e921c2b7312ae7c856b
| | * Avoid integer overflow in dct_II().Fraunhofer IIS FDK2021-02-141-15/+13
| | | | | | | | | | | | | | | | | | Bug: 176246647 Test: atest DecoderTestXheAac DecoderTestAacDrc Change-Id: I6c30c4dec3f85410c2748eb42d38f5eb72521ec5
* | | Prevent integer overflows in dualChannelFiltering() and ↵Fraunhofer IIS FDK2021-03-190-0/+0
|\| | | | | | | | | | | | | | | | | | | | | | | eightChannelFiltering(). am: 614b9f005a am: a948ff1cd4 Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/13893568 Change-Id: Ie1b7ec7af6e41e095d93253c8ecc4eb63d50d295
| * | Prevent integer overflows in dualChannelFiltering() and ↵Fraunhofer IIS FDK2021-03-190-0/+0
| |\| | | | | | | | | | | | | | | | | | | | | | eightChannelFiltering(). am: 614b9f005a Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/13893568 Change-Id: Ibcb7694e41a6836f5c46901cce534ea3d8813ab2
| | * Prevent integer overflows in dualChannelFiltering() and eightChannelFiltering().Fraunhofer IIS FDK2021-02-141-13/+13
| | | | | | | | | | | | | | | | | | Bug: 176246647 Test: atest DecoderTestXheAac DecoderTestAacDrc Change-Id: Ic9217bbb3980807036ae6ae121e6ddb7cc1bce35
* | | Adjust VBR mode depending on given peak bitrate and fix crash recovery ↵Fraunhofer IIS FDK2021-03-190-0/+0
|\| | | | | | | | | | | | | | | | | | | | | | | usage. am: f89d1d0a4e am: f04901b42a Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/13893567 Change-Id: Icbb5dab11813dda07c2bf945781ef2b3e720e002
| * | Adjust VBR mode depending on given peak bitrate and fix crash recovery ↵Fraunhofer IIS FDK2021-03-190-0/+0
| |\| | | | | | | | | | | | | | | | | | | | | | usage. am: f89d1d0a4e Original change: https://googleplex-android-review.googlesource.com/c/platform/external/aac/+/13893567 Change-Id: I45a95e5562b8e7b8fef1e27d5e1aa28df1f1325f
| | * Adjust VBR mode depending on given peak bitrate and fix crash recovery usage.Fraunhofer IIS FDK2021-02-144-36/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Operating the FDK encoder in A2DP with variable bitrate mode configuration resulted in unexpected encoder return value AACENC_ENCODE_ERROR. Due to peak bitrate restriction the encoder quite often runs into requantization. In case the bitrate constraint is not fulfilled, the so-called crash recovery is used as final emergency step. The crash recovery reduces the overall bit consumption considering the given number of bits to be saved. The bit difference is extracted from audio element structures. In VBR mode the element wise bit consumption state was not updated since there is typically no bitrate limitation required. The patch solves the choppy audio problems and increases audio quality for AAC VBR encoding. The changes in FDKaacEnc_QCMain() ensure that audio element bit info is always updated. This is achieved by always calling FDKaacEnc_BitResRedistribution() and FDKaacEnc_prepareBitDistribution() with maxBitsPerFrame variable as total bits parameter. Furthermore, VBR assumes a certain target bitrate which is used for internal configuration and limitations. In case the peak bitrate parameter is less than the VBR mode target bitrate the maximum of both bitrate configurations was used. The function FDKaacEnc_AdjustVBRBitrateMode() is added to adjust the encoder internal VBR mode to a desired target bitrate less than given peak bitrate. It is possible that the peak bitrate is very close to the desired target bitrate. The virtual available bitreservoir is quite low and the encoder would run quite often into requantization with needless audio quality reduction. In such a configuration, it is a better choice to use the CBR targeted threshold adaption to avoid audio quality reduction. In FDKaacEnc_Initialize(), there was already a bitResMode selection depending on available bitreservoir for CBR. This selection will now also be used for VBR. In case the bitResMode is AACENC_BR_MODE_FULL and VBR mode is selected, the VBR threshold adaption is used. Otherwise, CBR threshold adaption strategy is used and therefore, no unnecessary fill bits are written. Bug: 161400526 Test: see bug Change-Id: I1865f817180150da6add2623a64f1a102622784a
* | | [automerger skipped] Mark ab/7061308 as merged in stage. am: 6d10f91a77 -s ↵Xin Li2021-02-200-0/+0
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ours am: 10a3289cdd -s ours am: c726622078 -s ours am skip reason: Change-Id Id04d2590c249cad13da8dd8975d0a12ce96a9597 with SHA-1 a5f0e4af6d is in history Original change: undetermined MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I53e6e4b91f7c54dc9ee702d7461fe36212c27aa4
| * | [automerger skipped] Mark ab/7061308 as merged in stage. am: 6d10f91a77 -s ↵Xin Li2021-02-200-0/+0
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ours am: 10a3289cdd -s ours am skip reason: Change-Id Id04d2590c249cad13da8dd8975d0a12ce96a9597 with SHA-1 a5f0e4af6d is in history Original change: undetermined MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I351222fa379905cfb22d3d4a5c9022dda52a07cd
| | * \ [automerger skipped] Mark ab/7061308 as merged in stage. am: 6d10f91a77 -s oursXin Li2021-02-200-0/+0
| | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am skip reason: Change-Id Id04d2590c249cad13da8dd8975d0a12ce96a9597 with SHA-1 a5f0e4af6d is in history Original change: undetermined MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: Ia658697341d335415b30e7c47015c78cbc308c34
| | | * \ Mark ab/7061308 as merged in stage.Xin Li2021-02-200-0/+0
| | | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 180401296 Merged-In: Id04d2590c249cad13da8dd8975d0a12ce96a9597 Change-Id: I4da99b78b01817308d8572d3c0d5b224b7d36a73
* | | | | | Merge "[LSC] Add LOCAL_LICENSE_KINDS to external/aac" am: 9f1be9d06f am: ↵Bob Badour2021-02-172-0/+42
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7ad87e81d7 am: 2020366ce7 am: 24a33f559c Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1588770 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: Id1fd6df26e6dab8ef19de7b1a889bc3ec1544867
| * | | | | Merge "[LSC] Add LOCAL_LICENSE_KINDS to external/aac" am: 9f1be9d06f am: ↵Bob Badour2021-02-172-0/+42
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7ad87e81d7 am: 2020366ce7 Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1588770 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I889815ab215531c3f2cd8a166c257ab9d00dbe8f
| | * | | | Merge "[LSC] Add LOCAL_LICENSE_KINDS to external/aac" am: 9f1be9d06f am: ↵Bob Badour2021-02-172-0/+42
| | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7ad87e81d7 Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1588770 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I12c397ed7120bed1014941d0b1413d2bf9c29bb5
| | | * | | Merge "[LSC] Add LOCAL_LICENSE_KINDS to external/aac" am: 9f1be9d06fBob Badour2021-02-172-0/+42
| | | |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1588770 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I37a9304f439a0ed700126de649a37378d03ce6bc
| | | | * \ \ Merge "[LSC] Add LOCAL_LICENSE_KINDS to external/aac"Bob Badour2021-02-172-0/+42
| | | | |\ \ \
| | | | | * | | [LSC] Add LOCAL_LICENSE_KINDS to external/aacBob Badour2021-02-142-0/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added SPDX-license-identifier-Apache-2.0 to: fuzzer/Android.bp Added SPDX-license-identifier-Apache-2.0 legacy_by_exception_only to: Android.bp Bug: 68860345 Bug: 151177513 Bug: 151953481 Test: m all Exempt-From-Owner-Approval: janitorial work Change-Id: I518bcad1cb91b80b2bcce1726f53aadf9afc749e
* | | | | | | | Merge "Fix improper assignment in aac_enc_fuzzer" am: dde78303e8 am: ↵Treehugger Robot2021-02-171-1/+1
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b3ae2fd27f am: 80ccc5df59 am: 80ec04f39f Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1592151 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I959fcfd422c98426ddd36e355abe73e8c48d442d
| * | | | | | | Merge "Fix improper assignment in aac_enc_fuzzer" am: dde78303e8 am: ↵Treehugger Robot2021-02-171-1/+1
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b3ae2fd27f am: 80ccc5df59 Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1592151 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I95a53a55339caf67184bb3c8fc8c562c634bb159
| | * | | | | | Merge "Fix improper assignment in aac_enc_fuzzer" am: dde78303e8 am: b3ae2fd27fTreehugger Robot2021-02-171-1/+1
| | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1592151 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I5758ea2841ee0323b20459e0a2dabacd509c14ca
| | | * | | | | Merge "Fix improper assignment in aac_enc_fuzzer" am: dde78303e8Treehugger Robot2021-02-171-1/+1
| | | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1592151 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: Ib28a2522296895dec1d6a5cc465a7427e19ab93a
| | | | * | | | Merge "Fix improper assignment in aac_enc_fuzzer"Treehugger Robot2021-02-171-1/+1
| | | | |\ \ \ \ | | | | | |/ / / | | | | |/| | |
| | | | | * | | Fix improper assignment in aac_enc_fuzzerAnuj Joshi2021-02-171-1/+1
| | | | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit updates the fuzzer to use correct value of kSignalingModesSize and prevent OOB access on kSignalingModes Test: ./aac_enc_fuzzer Bug: 180220144 Change-Id: Ic9d01c5c166f2291d8d8ffacaa7813fb1560322b
* | | | | | | Merge "aac_enc_fuzzer: Improve code coverage" am: fbed528d62 am: 118aad7689 ↵Treehugger Robot2021-02-143-52/+174
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 2ee2fa993b am: 8105418c36 Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1566866 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: Ie80743b93a236920ac9f758676ca5f2a63db3169
| * | | | | | Merge "aac_enc_fuzzer: Improve code coverage" am: fbed528d62 am: 118aad7689 ↵Treehugger Robot2021-02-143-52/+174
|/| | | | | | | |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 2ee2fa993b Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1566866 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I8283ef52ab3e1bfe2d15bb2181ff91d55f8887c7
| * | | | | Merge "aac_enc_fuzzer: Improve code coverage" am: fbed528d62 am: 118aad7689Treehugger Robot2021-02-143-52/+174
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1566866 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I4a552820e12c26f42486d2c5be088b46c03e47b0
| | * | | | Merge "aac_enc_fuzzer: Improve code coverage" am: fbed528d62Treehugger Robot2021-02-143-52/+174
| | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1566866 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: Ic9e8b3426ba251349474e101fa5038172de4ad70
| | | * | | Merge "aac_enc_fuzzer: Improve code coverage"Treehugger Robot2021-02-143-52/+174
| | | |\ \ \
| | | | * | | aac_enc_fuzzer: Improve code coverageAnuj Joshi2021-02-013-52/+174
| | | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test: ./aac_enc_fuzzer Bug: 172429950 Change-Id: Id8c5c6351627ba09d6b0f8eeb81ad81003eba0c5
* | | | | | Merge "aac remove diag" am: f13c7a4054 am: 2b3a48d3b7 am: 8aa9b563bdCindy Zhou2021-01-211-7/+0
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1549588 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I3a49851a5dbdb4487083d20661c7825378a11424
| * | | | | Merge "aac remove diag" am: f13c7a4054 am: 2b3a48d3b7Cindy Zhou2021-01-201-7/+0
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1549588 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I6b976c7c29a345e632d25f2e6ac1c2e5ea631b9f
| | * | | | Merge "aac remove diag" am: f13c7a4054Cindy Zhou2021-01-201-7/+0
| | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Original change: https://android-review.googlesource.com/c/platform/external/aac/+/1549588 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I228ca5c02b5284a17e96c2f09adc27848d364703
| | | * | | Merge "aac remove diag"Cindy Zhou2021-01-201-7/+0
| | | |\ \ \
| | | | * | | aac remove diagCindy Zhou2021-01-131-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removing diag mode since it's only required for debugging purposes. Bug: 158010610 Test: n/a Change-Id: I597880ef5b63d12d2971237d02406faceda7e623
* | | | | | | Merge "Solve issue regarding config changes between Loudness-only vs. ↵Jean-Michel Trivi2021-01-192-4/+58
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Loudness+DRC."
| * | | | | | | Solve issue regarding config changes between Loudness-only vs. Loudness+DRC.Fraunhofer IIS FDK2021-01-142-4/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix config changes from Loudness+DRC to Loudness-only configurations by clearing the DRC configuration Bug: 176246647 Test: atest android.media.cts.DecoderTestAacFormat android.media.cts.DecoderTestXheAac android.media.cts.DecoderTestAacDrc Change-Id: I7afef848308478c29a82b13f24ba8c2a9760fd45
* | | | | | | | Merge "Overcome SBR encoder coupling quality issue in ELD."TreeHugger Robot2021-01-182-10/+17
|\ \ \ \ \ \ \ \
| * | | | | | | | Overcome SBR encoder coupling quality issue in ELD.Fraunhofer IIS FDK2021-01-142-10/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug 177604812 Test: see SBR/PS encoder Change-Id: Ie7b9f8d1dec3708557d1e1189d45bc0e3bf5444c
* | | | | | | | | Merge "Fix input buffer flushing for PS encoder."TreeHugger Robot2021-01-181-5/+6
|\| | | | | | | |
| * | | | | | | | Fix input buffer flushing for PS encoder.Fraunhofer IIS FDK2021-01-141-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug 177604812 Test: see PS/SBR encoder Change-Id: I3152bd7db5e039ecc54885e5b58c1cf6cb598b28
* | | | | | | | | Merge "Fix wrong input buffer feed for PS encoder at less input samples than ↵TreeHugger Robot2021-01-181-2/+2
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | framesize."
| * | | | | | | | Fix wrong input buffer feed for PS encoder at less input samples than framesize.Fraunhofer IIS FDK2021-01-141-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug 177604812 Test: see PS/SBR encoder Change-Id: I08a20209bcb83784fda5c14584b6e99c65672545
* | | | | | | | | Merge "Read uniDrcGainExtension element only if all DRC gain sequences are ↵TreeHugger Robot2021-01-151-5/+8
|\ \ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | | | | | | | | | | | parsed correctly."
| * | | | | | | | Read uniDrcGainExtension element only if all DRC gain sequences are parsed ↵Fraunhofer IIS FDK2021-01-141-5/+8
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | correctly. Prevent error return during parsing of incomplete uniDrcGain() element. This fixes unnecessary concealment of frames for AAC in case a uniDrcGain() is provided, but the uniDrcConfig() element is not available. Bug: 176246647 Test: atest android.media.cts.DecoderTestAacFormat android.media.cts.DecoderTestXheAac android.media.cts.DecoderTestAacDrc Change-Id: I5ae343eaea481774c53240dbce4da0f4ff06cd1d
* | | | | | | | Merge "Fix USAC time domain limiter latency at config change."TreeHugger Robot2021-01-155-141/+156
|\ \ \ \ \ \ \ \