From 589c4f7ad98ac65e2f7ae854bf46c62077783e51 Mon Sep 17 00:00:00 2001 From: Chun-Wei Wang Date: Thu, 17 Nov 2022 01:37:45 +0000 Subject: [PATCH] Revert "Prevent committing session while in progress" This reverts commit 1138064218b9e29c5ab97d58d963946a8c093828. Reason for revert: b/259333954 Change-Id: I724ec379395237ff8f47c98815d0ea49b7417e5d --- .../server/pm/PackageInstallerSession.java | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/services/core/java/com/android/server/pm/PackageInstallerSession.java b/services/core/java/com/android/server/pm/PackageInstallerSession.java index 06458d1bf50ca..37bfbb11948a8 100644 --- a/services/core/java/com/android/server/pm/PackageInstallerSession.java +++ b/services/core/java/com/android/server/pm/PackageInstallerSession.java @@ -366,14 +366,6 @@ public class PackageInstallerSession extends IPackageInstallerSession.Stub { @GuardedBy("mLock") private boolean mStageDirInUse = false; - /** - * True if the installation is already in progress. This is used to prevent the caller - * from {@link #commit(IntentSender, boolean) committing} the session again while the - * installation is still in progress. - */ - @GuardedBy("mLock") - private boolean mInstallationInProgress = false; - /** Permissions have been accepted by the user (see {@link #setPermissionsResult}) */ @GuardedBy("mLock") private boolean mPermissionsManuallyAccepted = false; @@ -1669,14 +1661,6 @@ public class PackageInstallerSession extends IPackageInstallerSession.Stub { } } - synchronized (mLock) { - if (mInstallationInProgress) { - throw new IllegalStateException("Installation is already in progress. Don't " - + "commit session=" + sessionId + " again."); - } - mInstallationInProgress = true; - } - dispatchSessionSealed(); }