diff options
author | Narayan Kamath <narayan@google.com> | 2014-08-08 12:44:12 +0100 |
---|---|---|
committer | Narayan Kamath <narayan@google.com> | 2014-08-15 12:50:10 +0100 |
commit | 6c4b9de8f1fd594038793c3924b52a44138c319e (patch) | |
tree | 7fe3145e6dbc6e35be1ce083e6128a40e1ee015b /core | |
parent | 4b2d0f20db2e0f9395a0c12ed5d4b6020eb272cb (diff) | |
download | frameworks_base-6c4b9de8f1fd594038793c3924b52a44138c319e.zip frameworks_base-6c4b9de8f1fd594038793c3924b52a44138c319e.tar.gz frameworks_base-6c4b9de8f1fd594038793c3924b52a44138c319e.tar.bz2 |
Validate instruction sets passed to installd.
We don't want folks passing down arbitrary strings.
bug: 16837404
Change-Id: I73ac66b376f1401f9f95f3c6323da6242ac8ed3d
Diffstat (limited to 'core')
-rw-r--r-- | core/java/android/app/LoadedApk.java | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/core/java/android/app/LoadedApk.java b/core/java/android/app/LoadedApk.java index 24c2835..aa1f021 100644 --- a/core/java/android/app/LoadedApk.java +++ b/core/java/android/app/LoadedApk.java @@ -262,10 +262,6 @@ public final class LoadedApk { if (!Objects.equals(mPackageName, ActivityThread.currentPackageName())) { final String isa = VMRuntime.getRuntime().vmInstructionSet(); try { - // TODO: We can probably do away with the isa argument since - // the AM and PM have enough information to figure this out - // themselves. If we do need it, we should match it against the - // list of devices ISAs before sending it down to installd. ActivityThread.getPackageManager().performDexOptIfNeeded(mPackageName, isa); } catch (RemoteException re) { // Ignored. |