diff options
author | 2025-03-21 16:28:39 -0700 | |
---|---|---|
committer | 2025-03-21 16:28:39 -0700 | |
commit | da47f1d52ddaea9d78afe9ea0b33afd7e45c8931 (patch) | |
tree | 3670af1ca6281223e007abff93711a3368e86b4b | |
parent | 5e83086a3e8188378a253aa0383f782a931b94ae (diff) | |
parent | a7a47eb55bc83df88df27aa73d09a9d68822433c (diff) |
Merge "Revert "[permission] Debugging access state concurrency"" into main
-rw-r--r-- | services/permission/java/com/android/server/permission/access/permission/AppIdPermissionPolicy.kt | 7 |
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 |