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

Commit f42633be authored by JW Wang's avatar JW Wang
Browse files

Revert "Early detection of mis-configuration (1/n)" 

This reverts commit 04bdf998.

Reason for revert: b/203627495

Change-Id: Iee3c3418bb4b42440751801a998fe4fd02bd9b77
parent 04bdf998

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

+7 −6
Original line number Diff line number Diff line
@@ -2406,6 +2406,13 @@ public class PackageInstallerSession extends IPackageInstallerSession.Stub { 
        final VerificationParams verifyingSession = prepareForVerification();

        if (isMultiPackage()) {

            final List<PackageInstallerSession> childSessions = getChildSessions();

            // Spot check to reject a non-staged multi package install of APEXes and APKs.

            if (!params.isStaged && containsApkSession()

                    && sessionContains(s -> s.isApexSession())) {

                throw new PackageManagerException(

                    PackageManager.INSTALL_FAILED_SESSION_INVALID,

                    "Non-staged multi package install of APEX and APK packages is not supported");

            }

            List<VerificationParams> verifyingChildSessions =

                    new ArrayList<>(childSessions.size());

            boolean success = true;
@@ -4165,12 +4172,6 @@ public class PackageInstallerSession extends IPackageInstallerSession.Stub { 
                    + childSession.sessionId + " and session " + sessionId

                    + " have inconsistent rollback settings");

        }

        boolean hasAPK = containsApkSession() || !childSession.isApexSession();

        boolean hasAPEX = sessionContains(s -> s.isApexSession()) || childSession.isApexSession();

        if (!params.isStaged && hasAPK && hasAPEX) {

            throw new IllegalStateException("Mix of APK and APEX is not supported for "

                    + "non-staged multi-package session");

        }



        try {

            acquireTransactionLock();