Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit dca00a43 authored by Hai Zhang's avatar Hai Zhang
Browse files

Reorder check in resetRuntimePermissions(). 

This change is a no-op in behavior, but conceptually it makes more sense to emphasize the check on permission being runtime first.

Bug: 288465674
Change-Id: Ic468a3b04a41f2ae8ec98a5af30de0e416c0d715
Test: presubmit
parent b06fae25

services/permission/java/com/android/server/permission/access/permission/AppIdPermissionPolicy.kt

+1 −1
Original line number Diff line number Diff line
@@ -245,7 +245,7 @@ class AppIdPermissionPolicy : SchemePolicy() { 
        androidPackage.requestedPermissions.forEach { permissionName ->

            val permission = newState.systemState.permissions[permissionName]

                ?: return@forEach

            if (permission.isRemoved || !permission.isRuntime) {

            if (!permission.isRuntime || permission.isRemoved) {

                return@forEach

            }

            val isRequestedByOtherPackages = anyPackageInAppId(appId) {