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

Commit 55f290fc authored by Atneya Nair's avatar Atneya Nair
Browse files

[audio] Flag off hardening enforcement 

Keep the implementation of hardening, but exempt all apps for now unless
a flag is in place to avoid regression.

Test: atest CtsMediaAudioPermissionTestCases
Flag: com.android.media.audio.hardening_strict
Bug: 376481063
Change-Id: Idadcd80447566e867a43727d3d838c9591ad4a70
parent b46b40a6

services/audioflinger/Tracks.cpp

+12 −5
Original line number Diff line number Diff line
@@ -75,6 +75,7 @@ namespace android { 


using ::android::aidl_utils::binderStatusFromStatusT;

using ::com::android::media::audio::hardening_impl;

using ::com::android::media::audio::hardening_strict;

using binder::Status;

using com::android::media::audio::audioserver_permissions;

using com::android::media::permission::PermissionEnum::CAPTURE_AUDIO_HOTWORD;
@@ -82,12 +83,18 @@ using content::AttributionSourceState; 
using media::VolumeShaper;



static bool shouldExemptFromOpControl(audio_usage_t usage) {

    // TODO(b/389136997) this should be swapped to another flag when it is added, but use this flag

    // for now since it is already in teamfood

    if (hardening_strict()) {

        switch (usage) {

            case AUDIO_USAGE_VIRTUAL_SOURCE:

                return true;

            default:

                return media::permission::isSystemUsage(usage);

        }

    } else {

        return true;

    }

}



// ----------------------------------------------------------------------------