summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Philip P. Moltmann <moltmann@google.com> 2020-11-05 23:39:31 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2020-11-05 23:39:31 +0000
commit5fe0e5eda36d26e66dd30b0cb69f51195d4a9fb8 (patch)
tree145be1c9ca3cca18921ab1ff23a38dd88e6cae53
parent9259e1b2bea309e472b91bb299dc03c81ff15563 (diff)
parent6a6a15d5951c15e763664b567bfa56f088978f97 (diff)
Merge changes from topic "12933477" into rvc-dev am: 6a6a15d595
Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/12986657 Change-Id: I56efae68cce32e25bdab66d176759ea9c01b9f27
-rw-r--r--services/core/java/com/android/server/pm/permission/BasePermission.java4
1 files changed, 2 insertions, 2 deletions
diff --git a/services/core/java/com/android/server/pm/permission/BasePermission.java b/services/core/java/com/android/server/pm/permission/BasePermission.java
index 2d4b5012bf15..5e04171a3bca 100644
--- a/services/core/java/com/android/server/pm/permission/BasePermission.java
+++ b/services/core/java/com/android/server/pm/permission/BasePermission.java
@@ -367,7 +367,7 @@ public final class BasePermission {
if (bp == null) {
bp = new BasePermission(p.getName(), p.getPackageName(), TYPE_NORMAL);
}
- boolean wasNormal = bp.isNormal();
+ boolean wasNonRuntime = !bp.isRuntime();
StringBuilder r = null;
if (bp.perm == null) {
if (bp.sourcePackageName == null
@@ -411,7 +411,7 @@ public final class BasePermission {
&& Objects.equals(bp.perm.getName(), p.getName())) {
bp.protectionLevel = p.getProtectionLevel();
}
- if (bp.isRuntime() && (ownerChanged || wasNormal)) {
+ if (bp.isRuntime() && (ownerChanged || wasNonRuntime)) {
// If this is a runtime permission and the owner has changed, or this was a normal
// permission, then permission state should be cleaned up
bp.mPermissionDefinitionChanged = true;