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

Commit a5cc7cce authored by Eric Laurent's avatar Eric Laurent
Browse files

Issue 5081351: isWiredHeadsetOn() permission. 

AudioManager.isWiredHeadsetOn() should not require permission MODIFY_AUDIO_SETTINGS.
Remove permission checks on all getters in audio policy manager as permission enforcement
is really usefull for setters.
Also deprecate AudioManager.isWiredHeadsetOn() which name and implementation are deceptive.

Change-Id: I38f8df7c26c0d417bf0e2b74e4c11c2d143f2ecd
parent 0270b188

services/audioflinger/AudioPolicyService.cpp

+0 −9
Original line number Diff line number Diff line
@@ -182,9 +182,6 @@ audio_policy_dev_state_t AudioPolicyService::getDeviceConnectionState( 
    if (mpAudioPolicy == NULL) {

        return AUDIO_POLICY_DEVICE_STATE_UNAVAILABLE;

    }

    if (!checkPermission()) {

        return AUDIO_POLICY_DEVICE_STATE_UNAVAILABLE;

    }

    return mpAudioPolicy->get_device_connection_state(mpAudioPolicy, device,

                                                      device_address);

}
@@ -250,9 +247,6 @@ audio_policy_forced_cfg_t AudioPolicyService::getForceUse(audio_policy_force_use 
    if (mpAudioPolicy == NULL) {

        return AUDIO_POLICY_FORCE_NONE;

    }

    if (!checkPermission()) {

        return AUDIO_POLICY_FORCE_NONE;

    }

    if (usage < 0 || usage >= AUDIO_POLICY_FORCE_USE_CNT) {

        return AUDIO_POLICY_FORCE_NONE;

    }
@@ -434,9 +428,6 @@ status_t AudioPolicyService::getStreamVolumeIndex(audio_stream_type_t stream, in 
    if (mpAudioPolicy == NULL) {

        return NO_INIT;

    }

    if (!checkPermission()) {

        return PERMISSION_DENIED;

    }

    if (stream < 0 || stream >= AUDIO_STREAM_CNT) {

        return BAD_VALUE;

    }