diff options
author | Calin Juravle <calin@google.com> | 2014-08-27 15:50:09 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2014-08-27 15:50:10 +0000 |
commit | 3bec87c9fab49fc2cc7ad1909331f3978803bdd6 (patch) | |
tree | ce9dd8da92008b4a5a115d530ebc9111e5ffe587 | |
parent | 9b5112c36b5163925cb303c0cd016bf4115c5800 (diff) | |
parent | e8d2ffd733c0598b99a91ce307b8f6411fcdafaa (diff) | |
download | base-3bec87c9fab49fc2cc7ad1909331f3978803bdd6.tar.gz |
Merge "Pass vmSafeMode (manifest attribute) to installd."
-rw-r--r-- | services/java/com/android/server/pm/Installer.java | 4 | ||||
-rwxr-xr-x | services/java/com/android/server/pm/PackageManagerService.java | 6 |
2 files changed, 7 insertions, 3 deletions
diff --git a/services/java/com/android/server/pm/Installer.java b/services/java/com/android/server/pm/Installer.java index 0f384b1c372e..4fd9277b155a 100644 --- a/services/java/com/android/server/pm/Installer.java +++ b/services/java/com/android/server/pm/Installer.java @@ -240,11 +240,12 @@ public final class Installer { builder.append(" *"); // No pkgName arg present builder.append(' '); builder.append(instructionSet); + builder.append(" 0"); // vmSafeMode=false return execute(builder.toString()); } public int dexopt(String apkPath, int uid, boolean isPublic, String pkgName, - String instructionSet) { + String instructionSet, boolean vmSafeMode) { StringBuilder builder = new StringBuilder("dexopt"); builder.append(' '); builder.append(apkPath); @@ -255,6 +256,7 @@ public final class Installer { builder.append(pkgName); builder.append(' '); builder.append(instructionSet); + builder.append(vmSafeMode ? " 1" : " 0"); return execute(builder.toString()); } diff --git a/services/java/com/android/server/pm/PackageManagerService.java b/services/java/com/android/server/pm/PackageManagerService.java index cb41d722fbd9..4cbf8456ff65 100755 --- a/services/java/com/android/server/pm/PackageManagerService.java +++ b/services/java/com/android/server/pm/PackageManagerService.java @@ -4293,6 +4293,7 @@ public class PackageManagerService extends IPackageManager.Stub { } } + final boolean vmSafeMode = (pkg.applicationInfo.flags & ApplicationInfo.FLAG_VM_SAFE_MODE) != 0; boolean performed = false; if ((pkg.applicationInfo.flags&ApplicationInfo.FLAG_HAS_CODE) != 0) { String path = pkg.mScanPath; @@ -4309,10 +4310,11 @@ public class PackageManagerService extends IPackageManager.Stub { // 2.) we are defering a needed dexopt // 3.) we are skipping an unneeded dexopt if (forceDex || (!defer && isDexOptNeededInternal == DexFile.DEXOPT_NEEDED)) { - Log.i(TAG, "Running dexopt on: " + pkg.applicationInfo.packageName); + Log.i(TAG, "Running dexopt on: " + pkg.applicationInfo.packageName + + " vmSafeMode=" + vmSafeMode); final int sharedGid = UserHandle.getSharedAppGid(pkg.applicationInfo.uid); int ret = mInstaller.dexopt(path, sharedGid, !isForwardLocked(pkg), - pkg.packageName, dexCodeInstructionSet); + pkg.packageName, dexCodeInstructionSet, vmSafeMode); // Note that we ran dexopt, since rerunning will // probably just result in an error again. pkg.mDexOptNeeded = false; |