diff options
| author | Glenn Kasten <gkasten@google.com> | 2012-01-06 16:47:31 -0800 |
|---|---|---|
| committer | Glenn Kasten <gkasten@google.com> | 2012-01-10 15:42:32 -0800 |
| commit | 930f4caa1e311ef7ff538c421a324396157eb24f (patch) | |
| tree | ac544b75a8964ac7759355fd0659d246070bcc57 /services/audioflinger/AudioPolicyService.cpp | |
| parent | c1dc1cb1d1eaf84e88669f1a5f22579a0d9237c2 (diff) | |
| download | frameworks_av-930f4caa1e311ef7ff538c421a324396157eb24f.zip frameworks_av-930f4caa1e311ef7ff538c421a324396157eb24f.tar.gz frameworks_av-930f4caa1e311ef7ff538c421a324396157eb24f.tar.bz2 | |
Simplify range check for audio_mode_t
AudioSystem::setMode previously allowed negative modes, but these were
then rejected by AudioFlinger.
Now negative modes (including AUDIO_MODE_INVALID and AUDIO_MODE_CURRENT)
are explicitly disallowed.
Change-Id: I0bac8fea737c8eb1f5b6afbb893e48739f88d745
Diffstat (limited to 'services/audioflinger/AudioPolicyService.cpp')
| -rw-r--r-- | services/audioflinger/AudioPolicyService.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/services/audioflinger/AudioPolicyService.cpp b/services/audioflinger/AudioPolicyService.cpp index f572fce..3f86d58 100644 --- a/services/audioflinger/AudioPolicyService.cpp +++ b/services/audioflinger/AudioPolicyService.cpp @@ -193,7 +193,7 @@ status_t AudioPolicyService::setPhoneState(int state) if (!checkPermission()) { return PERMISSION_DENIED; } - if (state < 0 || state >= AUDIO_MODE_CNT) { + if (uint32_t(state) >= AUDIO_MODE_CNT) { return BAD_VALUE; } |