From d9d343cfe090655eb0273a954094097ee20785d0 Mon Sep 17 00:00:00 2001 From: "Philip P. Moltmann" Date: Tue, 3 Jul 2018 15:17:11 -0700 Subject: [PATCH] Do not commit session twice Change-Id: Icee9011cb3bba6d536a86452d1dbca1c5c2313e6 Fixes: 110847215 Test: atest GtsPackageInstallTestCases --- .../java/com/android/server/pm/PackageInstallerSession.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/services/core/java/com/android/server/pm/PackageInstallerSession.java b/services/core/java/com/android/server/pm/PackageInstallerSession.java index fa934fe23fff4..17160be9e862a 100644 --- a/services/core/java/com/android/server/pm/PackageInstallerSession.java +++ b/services/core/java/com/android/server/pm/PackageInstallerSession.java @@ -930,6 +930,10 @@ public class PackageInstallerSession extends IPackageInstallerSession.Stub { @GuardedBy("mLock") private void commitLocked() throws PackageManagerException { + if (mRelinquished) { + Slog.d(TAG, "Ignoring commit after previous commit relinquished control"); + return; + } if (mDestroyed) { throw new PackageManagerException(INSTALL_FAILED_INTERNAL_ERROR, "Session destroyed"); }