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

Commit 06775341 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 7945a6e6

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

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


    private void assertPackageWithSharedUserIdIsPrivileged(AndroidPackage pkg)

            throws PackageManagerException {

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

        if (!AndroidPackageUtils.isPrivileged(pkg)

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

                && !pkg.isLeavingSharedUser()) {

            SharedUserSetting sharedUserSetting = null;

            try {

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

                && !AndroidPackageUtils.isPrivileged(pkg)

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

                && !skipVendorPrivilegeScan) {

                && !skipVendorPrivilegeScan

                && !pkg.isLeavingSharedUser()) {

            SharedUserSetting sharedUserSetting = null;

            synchronized (mPm.mLock) {

                try {