diff options
| author | 2019-07-08 12:58:44 -0700 | |
|---|---|---|
| committer | 2019-07-08 12:58:44 -0700 | |
| commit | 66c02ffd6d954673c4412a7e7349b491595d82b5 (patch) | |
| tree | c5bfa6c723be5d2d4632a936d62e077cb3650055 | |
| parent | 2b9a187195756d78dfac7ecea05b4b00f242256f (diff) | |
| parent | 540104635699567e58e7337e5f0a31a6bc21a562 (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.java | 5 |
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; |