Merge "Re-seal the sessions on reboot." (8468378f) · Commits · e / os / android_frameworks_base

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

+2 −2

Original line number	Diff line number	Diff line
		@@ -400,10 +400,10 @@ public class PackageInstallerService extends IPackageInstaller.Stub implements
		} finally {
		IoUtils.closeQuietly(fis);
		}
		// After all of the sessions were loaded, they are ready to be sealed and validated
		// Re-sealing the sealed sessions.
		for (int i = 0; i < mSessions.size(); ++i) {
		PackageInstallerSession session = mSessions.valueAt(i);
		session.sealAndValidateIfNecessary();
		session.sealIfNecessary();
		}
		}

+4 −8

Original line number	Diff line number	Diff line
		@@ -1374,15 +1374,13 @@ public class PackageInstallerSession extends IPackageInstallerSession.Stub {
		}

		/**
		* If session should be sealed, then it's sealed to prevent further modification
		* and then it's validated.
		*
		* If the session was sealed but something went wrong then it's destroyed.
		* If session should be sealed, then it's sealed to prevent further modification.
		* If the session can't be sealed then it's destroyed.
		*
		* <p> This is meant to be called after all of the sessions are loaded and added to
		* PackageInstallerService
		*/
		void sealAndValidateIfNecessary() {
		void sealIfNecessary() {
		synchronized (mLock) {
		if (!mShouldBeSealed \|\| isStagedAndInTerminalState()) {
		return;
		@@ -1391,9 +1389,7 @@ public class PackageInstallerSession extends IPackageInstallerSession.Stub {
		List<PackageInstallerSession> childSessions = getChildSessions();
		synchronized (mLock) {
		try {
		sealAndValidateLocked(childSessions);
		} catch (StreamingException e) {
		Slog.e(TAG, "Streaming failed", e);
		sealLocked(childSessions);
		} catch (PackageManagerException e) {
		Slog.e(TAG, "Package not valid", e);
		}