summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2020-03-06 09:57:35 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2020-03-06 09:57:35 +0000
commit7156be3a4cc226a781b8f6f82942cd5f71768f65 (patch)
treeccd9ab0b54b225896f4dfc383c0d0997cad048f5
parent3d9475fef4e52315a91f533cdf432fe4662f6c76 (diff)
parentc2dbfdc430f82a0d914cf95bd02a098a06d85f78 (diff)
Merge "libbinder: cleanup appOpNote globals" into rvc-dev am: c2dbfdc430
Change-Id: Id291a51744a461ee5717872268616ed143115e41
-rw-r--r--libs/binder/AppOpsManager.cpp9
1 files changed, 3 insertions, 6 deletions
diff --git a/libs/binder/AppOpsManager.cpp b/libs/binder/AppOpsManager.cpp
index c21f3ef226..e732829ada 100644
--- a/libs/binder/AppOpsManager.cpp
+++ b/libs/binder/AppOpsManager.cpp
@@ -42,12 +42,6 @@ static const sp<IBinder>& getClientId() {
return gClientId;
}
-thread_local uint64_t notedAppOpsInThisBinderTransaction[2];
-thread_local int32_t uidOfThisBinderTransaction = -1;
-
-// Whether an appop should be collected: 0 == not initialized, 1 == don't note, 2 == note
-uint8_t appOpsToNote[AppOpsManager::_NUM_OP] = {0};
-
AppOpsManager::AppOpsManager()
{
}
@@ -174,6 +168,9 @@ void AppOpsManager::setCameraAudioRestriction(int32_t mode) {
// check it the appops needs to be collected and cache result
bool AppOpsManager::shouldCollectNotes(int32_t opcode) {
+ // Whether an appop should be collected: 0 == not initialized, 1 == don't note, 2 == note
+ static uint8_t appOpsToNote[AppOpsManager::_NUM_OP] = {0};
+
if (appOpsToNote[opcode] == 0) {
if (getService()->shouldCollectNotes(opcode)) {
appOpsToNote[opcode] = 2;