diff options
author | Peter Jensen <jensenp@google.com> | 2015-03-25 11:00:02 -0700 |
---|---|---|
committer | Peter Jensen <jensenp@google.com> | 2015-03-27 09:23:25 -0700 |
commit | d024c06726c7e119ee76395611aa7cfe3b0a8b7e (patch) | |
tree | c825d21400bcca31d52e62c6a9f1d68de6a6ce50 | |
parent | 8205cac070234acf9715c756593d5a5e297c6961 (diff) | |
download | dalvik-d024c06726c7e119ee76395611aa7cfe3b0a8b7e.tar.gz |
Fix potential dex index overflow in dx
Revert change to dex rotation algorithm made as part of commit:
845d9d0eed0f6556e11ee7f7204fda9c8dd41154
Bug: https://code.google.com/p/android/issues/detail?id=161887
Change-Id: I255aa52b74c921a78e77671ae6187157fef011e1
Signed-off-by: Peter Jensen <jensenp@google.com>
-rw-r--r-- | dx/src/com/android/dx/command/dexer/Main.java | 12 | ||||
-rw-r--r-- | dx/tests/128-multidex-option-overflow/expected.txt | 1 |
2 files changed, 7 insertions, 6 deletions
diff --git a/dx/src/com/android/dx/command/dexer/Main.java b/dx/src/com/android/dx/command/dexer/Main.java index 36b9e2059..b9c6ec952 100644 --- a/dx/src/com/android/dx/command/dexer/Main.java +++ b/dx/src/com/android/dx/command/dexer/Main.java @@ -1731,15 +1731,15 @@ public class Main { // Calculate max number of indices this class will add to the // dex file. - // The constant pool contains at least one entry per method - // (name and signature), at least one entry per field (name - // and type), and at least per method/field reference (typed - // method ref). + // The possibility of overloading means that we can't easily + // know how many constant are needed for declared methods and + // fields. We therefore make the simplifying assumption that + // all constants are external method or field references. int constantPoolSize = cf.getConstantPool().size(); - maxMethodIdsInClass = constantPoolSize - cf.getFields().size() + maxMethodIdsInClass = constantPoolSize + cf.getMethods().size() + MAX_METHOD_ADDED_DURING_DEX_CREATION; - maxFieldIdsInClass = constantPoolSize - cf.getMethods().size() + maxFieldIdsInClass = constantPoolSize + cf.getFields().size() + MAX_FIELD_ADDED_DURING_DEX_CREATION; synchronized(dexRotationLock) { diff --git a/dx/tests/128-multidex-option-overflow/expected.txt b/dx/tests/128-multidex-option-overflow/expected.txt index ac448a63d..3d7a649b8 100644 --- a/dx/tests/128-multidex-option-overflow/expected.txt +++ b/dx/tests/128-multidex-option-overflow/expected.txt @@ -1,2 +1,3 @@ classes2.dex +classes3.dex classes.dex |