Fix moving of apps into legacy ASEC. Derive the correct current volume UUID for comparison, and only check for cluster style installs when moving from internal storage. Bug: 22616484 Change-Id: Idb6be2aa4aaa9b9f47ebbeeebd65c15a60d5d164
diff --git a/services/core/java/com/android/server/pm/PackageManagerService.java b/services/core/java/com/android/server/pm/PackageManagerService.java index 26304ab..4fe5846 100644 --- a/services/core/java/com/android/server/pm/PackageManagerService.java +++ b/services/core/java/com/android/server/pm/PackageManagerService.java
@@ -15813,16 +15813,27 @@ "Cannot move system application"); } - if (Objects.equals(ps.volumeUuid, volumeUuid)) { - throw new PackageManagerException(MOVE_FAILED_INTERNAL_ERROR, - "Package already moved to " + volumeUuid); + if (pkg.applicationInfo.isExternalAsec()) { + currentAsec = true; + currentVolumeUuid = StorageManager.UUID_PRIMARY_PHYSICAL; + } else if (pkg.applicationInfo.isForwardLocked()) { + currentAsec = true; + currentVolumeUuid = "forward_locked"; + } else { + currentAsec = false; + currentVolumeUuid = ps.volumeUuid; + + final File probe = new File(pkg.codePath); + final File probeOat = new File(probe, "oat"); + if (!probe.isDirectory() || !probeOat.isDirectory()) { + throw new PackageManagerException(MOVE_FAILED_INTERNAL_ERROR, + "Move only supported for modern cluster style installs"); + } } - final File probe = new File(pkg.codePath); - final File probeOat = new File(probe, "oat"); - if (!probe.isDirectory() || !probeOat.isDirectory()) { + if (Objects.equals(currentVolumeUuid, volumeUuid)) { throw new PackageManagerException(MOVE_FAILED_INTERNAL_ERROR, - "Move only supported for modern cluster style installs"); + "Package already moved to " + volumeUuid); } if (ps.frozen) { @@ -15831,9 +15842,6 @@ } ps.frozen = true; - currentAsec = pkg.applicationInfo.isForwardLocked() - || pkg.applicationInfo.isExternalAsec(); - currentVolumeUuid = ps.volumeUuid; codeFile = new File(pkg.codePath); installerPackageName = ps.installerPackageName; packageAbiOverride = ps.cpuAbiOverrideString;