search

catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..
https://chromium.googlesource.com/catapult
BSD 3-Clause "New" or "Revised" License
1.91k stars 563 forks source link

dexdump version too old, fails on new dex files #4590

Closed m0hamed closed 5 years ago

m0hamed commented 5 years ago

Running the bundled version of dexdump on our dex files fails with the following error:

Processing '/tmp/tmpyRsgEQ_dex_extract/classes2.dex'...
E/libdex  (35211): ERROR: unsupported dex version (30 33 37 00)
E/libdex  (35211): ERROR: Byte swap + verify failed
ERROR: Failed structural verification of '/tmp/tmpyRsgEQ_dex_extract/classes2.dex'

Running dexdump that is supplied by the sdk (sdk v27.0.3) works correctly on the same dex file. Unfortunately dexdump does not have a --version arg so I cannot comment on the actual versions of dexdump involved.

m0hamed commented 5 years ago

I am closing the issue since after looking more into this, I found that catapult does not bundle any specific version of dexdump but is given the path to the utilities using configs. So this issue is probably in our build system.