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

Commit 67651f97 authored by Eric Laurent's avatar Eric Laurent
Browse files

Revert "audio flinger: move record permission checks to audio policy service" 

This reverts commit c7b71a1b.

Bug: 72628781
Test: Capture several videos from camera app
Change-Id: If633e677d87320335e368a8340599c9e4a398201
parent 5f07c65a

services/audioflinger/AudioFlinger.cpp

+6 −0
Original line number Diff line number Diff line
@@ -1622,6 +1622,12 @@ sp<media::IAudioRecord> AudioFlinger::createRecord(const CreateRecordInput& inpu 
        clientPid = callingPid;

    }



    // check calling permissions

    if (!recordingAllowed(input.opPackageName, input.clientInfo.clientTid, clientUid)) {

        ALOGE("createRecord() permission denied: recording not allowed");

        lStatus = PERMISSION_DENIED;

        goto Exit;

    }

    // we don't yet support anything other than linear PCM

    if (!audio_is_valid_format(input.config.format) || !audio_is_linear_pcm(input.config.format)) {

        ALOGE("createRecord() invalid format %#x", input.config.format);

services/audioflinger/Threads.cpp

+4 −0
Original line number Diff line number Diff line
@@ -7801,6 +7801,10 @@ status_t AudioFlinger::MmapThread::start(const AudioClient& client, 
        return NO_ERROR;

    }



    if (!isOutput() && !recordingAllowed(client.packageName, client.clientPid, client.clientUid)) {

        return PERMISSION_DENIED;

    }



    audio_port_handle_t portId = AUDIO_PORT_HANDLE_NONE;



    audio_io_handle_t io = mId;

services/audiopolicy/service/AudioPolicyInterfaceImpl.cpp

+5 −23
Original line number Diff line number Diff line
@@ -296,7 +296,6 @@ status_t AudioPolicyService::getInputForAttr(const audio_attributes_t *attr, 
    if (mAudioPolicyManager == NULL) {

        return NO_INIT;

    }



    // already checked by client, but double-check in case the client wrapper is bypassed

    if (attr->source < AUDIO_SOURCE_DEFAULT && attr->source >= AUDIO_SOURCE_CNT &&

            attr->source != AUDIO_SOURCE_HOTWORD && attr->source != AUDIO_SOURCE_FM_TUNER) {
@@ -320,13 +319,6 @@ status_t AudioPolicyService::getInputForAttr(const audio_attributes_t *attr, 
        pid = callingPid;

    }



    // check calling permissions

    if (!recordingAllowed(opPackageName, pid, uid)) {

        ALOGE("%s permission denied: recording not allowed for uid %d pid %d",

                __func__, uid, pid);

        return PERMISSION_DENIED;

    }



    if ((attr->source == AUDIO_SOURCE_HOTWORD) && !captureHotwordAllowed(pid, uid)) {

        return BAD_VALUE;

    }
@@ -400,28 +392,18 @@ status_t AudioPolicyService::startInput(audio_port_handle_t portId, bool *silenc 
    if (mAudioPolicyManager == NULL) {

        return NO_INIT;

    }

    sp<AudioRecordClient> client;

    {



    Mutex::Autolock _l(mLock);



    ssize_t index = mAudioRecordClients.indexOfKey(portId);

    if (index < 0) {

        return INVALID_OPERATION;

    }

        client = mAudioRecordClients.valueAt(index);

    }



    // check calling permissions

    if (!recordingAllowed(client->opPackageName, client->pid, client->uid)) {

        ALOGE("%s permission denied: recording not allowed for uid %d pid %d",

                __func__, client->uid, client->pid);

        return PERMISSION_DENIED;

    }

    sp<AudioRecordClient> client = mAudioRecordClients.valueAt(index);



    // If UID inactive it records silence until becoming active

    *silenced = !mUidPolicy->isUidActive(client->uid) && !client->isVirtualDevice;



    Mutex::Autolock _l(mLock);

    AudioPolicyInterface::concurrency_type__mask_t concurrency =

            AudioPolicyInterface::API_INPUT_CONCURRENCY_NONE;