diff options
| author | jiabin <jiabin@google.com> | 2022-07-11 23:28:48 +0000 |
|---|---|---|
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2022-07-11 23:28:48 +0000 |
| commit | c9c8e96cfde05bc2fb44a5221d5169c155d0bd95 (patch) | |
| tree | 888d2e03164b74a4b2066c5b7141925551472029 /audio/common/all-versions/default/7.0/HidlUtils.cpp | |
| parent | 5b71f85740a99e53854276a222f194c9e6c4e204 (diff) | |
| parent | 3074154353d66c91e2cb87d24368f8365f300f9c (diff) | |
[automerge] Fix array out of bound in audioTransportToHal. 2p: f16c6d3a57 2p: 0d43585645 am: 3074154353
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/19214732
Change-Id: I816384174e4a35d262f5406bf5100a54d3f6e976
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
Diffstat (limited to 'audio/common/all-versions/default/7.0/HidlUtils.cpp')
| -rw-r--r-- | audio/common/all-versions/default/7.0/HidlUtils.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/audio/common/all-versions/default/7.0/HidlUtils.cpp b/audio/common/all-versions/default/7.0/HidlUtils.cpp index 0fd2947963..f89c898da8 100644 --- a/audio/common/all-versions/default/7.0/HidlUtils.cpp +++ b/audio/common/all-versions/default/7.0/HidlUtils.cpp @@ -898,7 +898,7 @@ status_t HidlUtils::audioTransportsToHal(const hidl_vec<AudioTransport>& transpo for (const auto& transport : transports) { switch (transport.audioCapability.getDiscriminator()) { case AudioTransport::AudioCapability::hidl_discriminator::profile: - if (halPort->num_audio_profiles > AUDIO_PORT_MAX_AUDIO_PROFILES) { + if (halPort->num_audio_profiles >= AUDIO_PORT_MAX_AUDIO_PROFILES) { ALOGE("%s, too many audio profiles", __func__); result = BAD_VALUE; break; @@ -914,7 +914,8 @@ status_t HidlUtils::audioTransportsToHal(const hidl_vec<AudioTransport>& transpo result); break; case AudioTransport::AudioCapability::hidl_discriminator::edid: - if (halPort->num_extra_audio_descriptors > AUDIO_PORT_MAX_EXTRA_AUDIO_DESCRIPTORS) { + if (halPort->num_extra_audio_descriptors >= + AUDIO_PORT_MAX_EXTRA_AUDIO_DESCRIPTORS) { ALOGE("%s, too many extra audio descriptors", __func__); result = BAD_VALUE; break; |