From 8f397cdd4c53e1a2a96bc4bdaeded59a007c85b8 Mon Sep 17 00:00:00 2001 From: Glenn Kasten Date: Wed, 18 Jan 2012 14:58:32 -0800 Subject: [PATCH] Simplify range check for audio_mode_t (continued) Missed one place in earlier CL of same name Change-Id: I0dd25364d0b8d5d731c02d352f139a0c8d4df1a8 --- media/libmedia/AudioSystem.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/media/libmedia/AudioSystem.cpp b/media/libmedia/AudioSystem.cpp index 28892df24f4c7..bbe4d8d1e8a6f 100644 --- a/media/libmedia/AudioSystem.cpp +++ b/media/libmedia/AudioSystem.cpp @@ -538,6 +538,7 @@ audio_policy_dev_state_t AudioSystem::getDeviceConnectionState(audio_devices_t d status_t AudioSystem::setPhoneState(audio_mode_t state) { + if (uint32_t(state) >= AUDIO_MODE_CNT) return BAD_VALUE; const sp& aps = AudioSystem::get_audio_policy_service(); if (aps == 0) return PERMISSION_DENIED;