summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Philip P. Moltmann <moltmann@google.com> 2019-07-08 12:58:44 -0700
committer android-build-merger <android-build-merger@google.com> 2019-07-08 12:58:44 -0700
commit66c02ffd6d954673c4412a7e7349b491595d82b5 (patch)
treec5bfa6c723be5d2d4632a936d62e077cb3650055
parent2b9a187195756d78dfac7ecea05b4b00f242256f (diff)
parent540104635699567e58e7337e5f0a31a6bc21a562 (diff)
Merge "Do not trigger revoke-perm callbacks if no change" into qt-dev
am: 5401046356 Change-Id: I6b55377cd97583ecf61875b27dc17a91cb344e67
-rw-r--r--services/core/java/com/android/server/pm/permission/PermissionManagerService.java5
1 files changed, 5 insertions, 0 deletions
diff --git a/services/core/java/com/android/server/pm/permission/PermissionManagerService.java b/services/core/java/com/android/server/pm/permission/PermissionManagerService.java
index a7da3ec950f4..d3e5df5d62d1 100644
--- a/services/core/java/com/android/server/pm/permission/PermissionManagerService.java
+++ b/services/core/java/com/android/server/pm/permission/PermissionManagerService.java
@@ -2269,6 +2269,11 @@ public class PermissionManagerService {
return;
}
+ // Permission is already revoked, no need to do anything.
+ if (!permissionsState.hasRuntimePermission(permName, userId)) {
+ return;
+ }
+
if (permissionsState.revokeRuntimePermission(bp, userId) ==
PERMISSION_OPERATION_FAILURE) {
return;