summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Treehugger Robot <android-test-infra-autosubmit@system.gserviceaccount.com> 2025-03-21 16:28:39 -0700
committer Android (Google) Code Review <android-gerrit@google.com> 2025-03-21 16:28:39 -0700
commitda47f1d52ddaea9d78afe9ea0b33afd7e45c8931 (patch)
tree3670af1ca6281223e007abff93711a3368e86b4b
parent5e83086a3e8188378a253aa0383f782a931b94ae (diff)
parenta7a47eb55bc83df88df27aa73d09a9d68822433c (diff)
Merge "Revert "[permission] Debugging access state concurrency"" into main
-rw-r--r--services/permission/java/com/android/server/permission/access/permission/AppIdPermissionPolicy.kt7
1 files changed, 0 insertions, 7 deletions
diff --git a/services/permission/java/com/android/server/permission/access/permission/AppIdPermissionPolicy.kt b/services/permission/java/com/android/server/permission/access/permission/AppIdPermissionPolicy.kt
index 662e0c06f261..cfaf31743a78 100644
--- a/services/permission/java/com/android/server/permission/access/permission/AppIdPermissionPolicy.kt
+++ b/services/permission/java/com/android/server/permission/access/permission/AppIdPermissionPolicy.kt
@@ -1751,13 +1751,6 @@ class AppIdPermissionPolicy : SchemePolicy() {
}
val appIdPermissionFlags = newState.mutateUserState(userId)!!.mutateAppIdPermissionFlags()
val permissionFlags = appIdPermissionFlags.mutateOrPut(appId) { MutableIndexedMap() }
- // for debugging possible races TODO(b/401768134)
- oldState.userStates[userId]?.appIdPermissionFlags[appId]?.map?.let {
- if (permissionFlags.map === it) {
- throw IllegalStateException("Unexpected sharing between old/new state")
- }
- }
-
permissionFlags.putWithDefault(permissionName, newFlags, 0)
if (permissionFlags.isEmpty()) {
appIdPermissionFlags -= appId