Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1eac4601 authored by Glenn Kasten's avatar Glenn Kasten Committed by Android (Google) Code Review
Browse files

Merge "Simplify range check for audio_mode_t"

parents d742fa3f 01aaf2c4

media/libmedia/AudioSystem.cpp

+1 −1
Original line number Diff line number Diff line
@@ -158,7 +158,7 @@ status_t AudioSystem::getStreamMute(int stream, bool* mute) 


status_t AudioSystem::setMode(int mode)

{

    if (mode >= AUDIO_MODE_CNT) return BAD_VALUE;

    if (uint32_t(mode) >= AUDIO_MODE_CNT) return BAD_VALUE;

    const sp<IAudioFlinger>& af = AudioSystem::get_audio_flinger();

    if (af == 0) return PERMISSION_DENIED;

    return af->setMode(mode);

services/audioflinger/AudioFlinger.cpp

+1 −1
Original line number Diff line number Diff line
@@ -574,7 +574,7 @@ status_t AudioFlinger::setMode(int mode) 
    if (!settingsAllowed()) {

        return PERMISSION_DENIED;

    }

    if ((mode < 0) || (mode >= AUDIO_MODE_CNT)) {

    if (uint32_t(mode) >= AUDIO_MODE_CNT) {

        ALOGW("Illegal value: setMode(%d)", mode);

        return BAD_VALUE;

    }

services/audioflinger/AudioPolicyService.cpp

+1 −1
Original line number Diff line number Diff line
@@ -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;

    }