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

Commit 56fbd4e1 authored by Todd Kennedy's avatar Todd Kennedy
Browse files

Don't abort multi-package install early 

The multi-package install flow depends upon all of the packages being
processed. If we ever skip one, any installation status will never be
returned to the installation observer.

Fixes: 134632533
Test: adb install-multi-package app1/base.apk app2/base.apk app3/base.apk app4/base.apk
Test: adb did not hang and returned proper error "Failure [INSTALL_FAILED_VERSION_DOWNGRADE"
Change-Id: I3c0d1df059e1411c2d77f96b8fb656c9d1050ce1
parent a501bbc7

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

+0 −2
Original line number Original line Diff line number Diff line
@@ -15012,7 +15012,6 @@ public class PackageManagerService extends IPackageManager.Stub 
                params.handleStartCopy();

                params.handleStartCopy();

                if (params.mRet != INSTALL_SUCCEEDED) {

                if (params.mRet != INSTALL_SUCCEEDED) {

                    mRet = params.mRet;

                    mRet = params.mRet;

                    break;

                }

                }

            }

            }

        }

        }
@@ -15023,7 +15022,6 @@ public class PackageManagerService extends IPackageManager.Stub 
                params.handleReturnCode();

                params.handleReturnCode();

                if (params.mRet != INSTALL_SUCCEEDED) {

                if (params.mRet != INSTALL_SUCCEEDED) {

                    mRet = params.mRet;

                    mRet = params.mRet;

                    break;

                }

                }

            }

            }

        }

        }