summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Treehugger Robot <treehugger-gerrit@google.com> 2017-12-11 15:47:45 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2017-12-11 15:47:45 +0000
commit998ac99db4b421181791beffbfd9de22987dc472 (patch)
treec1eb50a23505a10e67d5f463be77728d3deb15e2
parentc0d951c3e0f7d9495e90b84dcfed0e8ad2604193 (diff)
parent10e5eeb68d4a3c55d761e13016994a537c703c63 (diff)
Merge "[DeadLock]AMS PMS MountService"
-rw-r--r--services/core/java/com/android/server/pm/PackageManagerService.java9
1 files changed, 4 insertions, 5 deletions
diff --git a/services/core/java/com/android/server/pm/PackageManagerService.java b/services/core/java/com/android/server/pm/PackageManagerService.java
index 920deb9cf9e7..08c881316205 100644
--- a/services/core/java/com/android/server/pm/PackageManagerService.java
+++ b/services/core/java/com/android/server/pm/PackageManagerService.java
@@ -14629,14 +14629,13 @@ public class PackageManagerService extends IPackageManager.Stub
if (getInstantAppPackageName(Binder.getCallingUid()) != null) {
return null;
}
- // writer
- synchronized (mPackages) {
- if (!isExternalMediaAvailable()) {
+ if (!isExternalMediaAvailable()) {
// If the external storage is no longer mounted at this point,
// the caller may not have been able to delete all of this
// packages files and can not delete any more. Bail.
- return null;
- }
+ return null;
+ }
+ synchronized (mPackages) {
final ArrayList<PackageCleanItem> pkgs = mSettings.mPackagesToBeCleaned;
if (lastPackage != null) {
pkgs.remove(lastPackage);