Simplify range check for audio_mode_t (continued)

gkasten · gkasten · commit 8f397cdd4c53 · 2012-01-18T15:57:40.000-08:00
Missed one place in earlier CL of same name

Change-Id: I0dd25364d0b8d5d731c02d352f139a0c8d4df1a8
diff --git 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<IAudioPolicyService>& aps = AudioSystem::get_audio_policy_service();
     if (aps == 0) return PERMISSION_DENIED;
 

Original file line number	Diff line number	Diff line change
`@@ -538,6 +538,7 @@ audio_policy_dev_state_t AudioSystem::getDeviceConnectionState(audio_devices_t d`
`538`	`538`
`539`	`539`	`status_t AudioSystem::setPhoneState(audio_mode_t state)`
`540`	`540`	`{`
	`541`	`+ if (uint32_t(state) >= AUDIO_MODE_CNT) return BAD_VALUE;`
`541`	`542`	`const sp<IAudioPolicyService>& aps = AudioSystem::get_audio_policy_service();`
`542`	`543`	`if (aps == 0) return PERMISSION_DENIED;`
`543`	`544`