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

Unverified Commit 06f9bcdc authored by Luca Stefani's avatar Luca Stefani Committed by Michael Bestas
Browse files

Make default permissions updates tied to version incremental 

Otherwise adding new rules won't have any effect for us.

Change-Id: Ib81082a184956ea550bdccdbdf53b2b4cc48f487
parent 28816d01

services/core/java/com/android/server/pm/PackageManagerService.java

+6 −4
Original line number Diff line number Diff line
@@ -4347,7 +4347,7 @@ public class PackageManagerService implements PackageSender, TestUtilityService 
            final int userId = livingUsers.get(i).id;

            final boolean isPermissionUpgradeNeeded = !Objects.equals(

                    mPermissionManager.getDefaultPermissionGrantFingerprint(userId),

                    Build.FINGERPRINT);

                    Build.VERSION.INCREMENTAL);

            if (isPermissionUpgradeNeeded) {

                grantPermissionsUserIds = ArrayUtils.appendInt(

                        grantPermissionsUserIds, userId);
@@ -4356,7 +4356,8 @@ public class PackageManagerService implements PackageSender, TestUtilityService 
        // If we upgraded grant all default permissions before kicking off.

        for (int userId : grantPermissionsUserIds) {

            mLegacyPermissionManager.grantDefaultPermissions(userId);

            mPermissionManager.setDefaultPermissionGrantFingerprint(Build.FINGERPRINT, userId);

            mPermissionManager.setDefaultPermissionGrantFingerprint(

                    Build.VERSION.INCREMENTAL, userId);

        }

        if (grantPermissionsUserIds == EMPTY_INT_ARRAY) {

            // If we did not grant default permissions, we preload from this the
@@ -4521,7 +4522,8 @@ public class PackageManagerService implements PackageSender, TestUtilityService 
        if (!convertedFromPreCreated || !readPermissionStateForUser(userId)) {

            mPermissionManager.onUserCreated(userId);

            mLegacyPermissionManager.grantDefaultPermissions(userId);

            mPermissionManager.setDefaultPermissionGrantFingerprint(Build.FINGERPRINT, userId);

            mPermissionManager.setDefaultPermissionGrantFingerprint(

                    Build.VERSION.INCREMENTAL, userId);

            mDomainVerificationManager.clearUser(userId);

        }

    }
@@ -4533,7 +4535,7 @@ public class PackageManagerService implements PackageSender, TestUtilityService 
            mPermissionManager.readLegacyPermissionStateTEMP();

            final boolean isPermissionUpgradeNeeded = !Objects.equals(

                    mPermissionManager.getDefaultPermissionGrantFingerprint(userId),

                    Build.FINGERPRINT);

                    Build.VERSION.INCREMENTAL);

            return isPermissionUpgradeNeeded;

        }

    }