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

Commit 20eff34d authored by Hai Zhang's avatar Hai Zhang
Browse files

Don't sync app op mode for root and system uid. 

Root and system server can pass all permission checks, so don't touch
their app op mode to keep compatibility.

Fixes: 155838450
Bug: 152592358
Test: b/155838450#comment32
Change-Id: Ia14542d132563aa01235dc9a9be1fd18851fd29e
parent f07d289b

services/core/java/com/android/server/policy/PermissionPolicyService.java

+9 −1
Original line number Diff line number Diff line
@@ -825,7 +825,15 @@ public final class PermissionPolicyService extends SystemService { 
                return;

            }



            if (pkgInfo == null || pkg == null || pkgInfo.requestedPermissions == null) {

            if (pkgInfo == null || pkg == null || pkgInfo.applicationInfo == null

                    || pkgInfo.requestedPermissions == null) {

                return;

            }



            final int uid = pkgInfo.applicationInfo.uid;

            if (uid == Process.ROOT_UID || uid == Process.SYSTEM_UID) {

                // Root and system server always pass permission checks, so don't touch their app

                // ops to keep compatibility.

                return;

            }