diff --git a/services/java/com/android/server/pm/PackageManagerService.java b/services/java/com/android/server/pm/PackageManagerService.java index 3501e47912652..75b3f0486b10e 100644 --- a/services/java/com/android/server/pm/PackageManagerService.java +++ b/services/java/com/android/server/pm/PackageManagerService.java @@ -8833,7 +8833,7 @@ public class PackageManagerService extends IPackageManager.Stub { // little while. mHandler.post(new Runnable() { public void run() { - updateExternalMediaStatusInner(mediaStatus, reportStatus); + updateExternalMediaStatusInner(mediaStatus, reportStatus, true); } }); } @@ -8843,7 +8843,7 @@ public class PackageManagerService extends IPackageManager.Stub { * Should block until all the ASEC containers are finished being scanned. */ public void scanAvailableAsecs() { - updateExternalMediaStatusInner(true, false); + updateExternalMediaStatusInner(true, false, false); } /* @@ -8852,7 +8852,8 @@ public class PackageManagerService extends IPackageManager.Stub { * Please note that we always have to report status if reportStatus has been * set to true especially when unloading packages. */ - private void updateExternalMediaStatusInner(boolean isMounted, boolean reportStatus) { + private void updateExternalMediaStatusInner(boolean isMounted, boolean reportStatus, + boolean externalStorage) { // Collection of uids int uidArr[] = null; // Collection of stale containers @@ -8890,6 +8891,14 @@ public class PackageManagerService extends IPackageManager.Stub { continue; } + /* + * Skip packages that are not external if we're unmounting + * external storage. + */ + if (externalStorage && !isMounted && !isExternal(ps)) { + continue; + } + final AsecInstallArgs args = new AsecInstallArgs(cid, isForwardLocked(ps)); // The package status is changed only if the code path // matches between settings and the container id.