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

Commit 3ee5dfdc 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
parent 7f7167e8

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

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


    private void assertPackageWithSharedUserIdIsPrivileged(AndroidPackage pkg)

            throws PackageManagerException {

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

        if (!AndroidPackageLegacyUtils.isPrivileged(pkg)

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

                && !pkg.isLeavingSharedUser()) {

            SharedUserSetting sharedUserSetting = null;

            try {

                synchronized (mPm.mLock) {
@@ -4630,7 +4632,8 @@ final class InstallPackageHelper { 
        if (((scanFlags & SCAN_AS_PRIVILEGED) == 0)

                && !AndroidPackageLegacyUtils.isPrivileged(pkg)

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

                && !skipVendorPrivilegeScan) {

                && !skipVendorPrivilegeScan

                && !pkg.isLeavingSharedUser()) {

            SharedUserSetting sharedUserSetting = null;

            synchronized (mPm.mLock) {

                try {