-
Notifications
You must be signed in to change notification settings - Fork 214
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Prepare Vdex processing backend for Android-P
Vdex version shipped with Android P preview images appears to fully break backwards compatibility. Therefore, the common Vdex structures from Oreo cannot be used anymore. After this commit, each supported Vdex version is required to define a different frontend (Vdex header/structs), backend (processor) and decompiler. The functions required to be exported to the main tool, will be initialised as an env struct with function pointers according to the found Vdex version. Signed-off-by: Anestis Bechtsoudis <anestis@census-labs.com>
- Loading branch information
Showing
23 changed files
with
960 additions
and
571 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
src/*.o | ||
src/*.dSYM | ||
*.o | ||
*.dSYM | ||
src/vdexExtractor | ||
obj | ||
libs |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
fe078c9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The native oatdump implimentation released in P can handle this also, yes?
fe078c9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@frap129 yes (https://android.googlesource.com/platform/art/+/android-9.0.0_r3/oatdump/oatdump.cc#611)
fe078c9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like google isn't too set on maintaining that, as it doesnt work.
oatdump E 08-08 17:12:36 49256 49256 oatdump.cc:3106] Failed to open oat file from '/tmp/tmp.FtKDBekPj3/com.google.android.camera.experimental2017.vdex': Failed to find expected EI_CLASS value 1 or 2 in /tmp/tmp.FtKDBekPj3/com.google.android.camera.experimental2017.vdex, found 48
Looks like the header format got changed but no one informed oatdump. Not asking for support or anything since I can just use odex instead. just thought you might be interested to know.
fe078c9
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Damn it. The whole reason I bothered to upstream the oatdump patches is that they are maintained while ART evolves. Thanks for letting me know (saved some time from expected broken tools).