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

Commit e336c7d7 authored by Alex Buynytskyy's avatar Alex Buynytskyy
Browse files

Stop marking apps as privileged if they are not signed properly. 

Fixes: 311374917
Test: atest android.content.pm.cts.PackageManagerTest
Change-Id: I5b5b81cf43b06837a22c8dfd170a112106dd64c1
Merged-In: I5b5b81cf43b06837a22c8dfd170a112106dd64c1
(cherry picked from commit 3ee5dfdc)
parent eee89b5a

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

+5 −2
Original line number Diff line number Diff line
@@ -4373,7 +4373,9 @@ final class InstallPackageHelper { 


    private void assertPackageWithSharedUserIdIsPrivileged(AndroidPackage pkg)

            throws PackageManagerException {

        if (!pkg.isPrivileged() && (pkg.getSharedUserId() != null)) {

        if (!pkg.isPrivileged()

                && (pkg.getSharedUserId() != null)

                && !pkg.isLeavingSharedUid()) {

            SharedUserSetting sharedUserSetting = null;

            try {

                sharedUserSetting = mPm.mSettings.getSharedUserLPw(pkg.getSharedUserId(),
@@ -4407,7 +4409,8 @@ final class InstallPackageHelper { 
        if (((scanFlags & SCAN_AS_PRIVILEGED) == 0)

                && !pkg.isPrivileged()

                && (pkg.getSharedUserId() != null)

                && !skipVendorPrivilegeScan) {

                && !skipVendorPrivilegeScan

                && !pkg.isLeavingSharedUid()) {

            SharedUserSetting sharedUserSetting = null;

            synchronized (mPm.mLock) {

                try {