Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[feature] add verify dex file checksum switch #1385

Closed
Jackjun724 opened this issue Feb 21, 2022 · 4 comments
Closed

[feature] add verify dex file checksum switch #1385

Jackjun724 opened this issue Feb 21, 2022 · 4 comments

Comments

@Jackjun724
Copy link

I have bad dex file, it's checksum is error. It comes from the frida-DEXDUMP, I used to use 1.2.0 version of jadx, which can normally open this DEX (maybe there are some errors). In 1.3.3 version, I can't open it. I think decompiling some code is better than unable to decompile. I think jadx need a switch to control verify dex file checksum

@skylot
Copy link
Owner

skylot commented Feb 21, 2022

@Jackjun724 done. New option to disable checksum verification added for cli and gui.
Actually, it is a plugin option, so there are a lot of changes to automatically pull options data from plugins and show/parse in cli and gui. Hope it will work as expected 🙂

@skylot skylot closed this as completed Feb 21, 2022
@Jackjun724
Copy link
Author

Jackjun724 commented Feb 22, 2022

Thank you for you 🙂

@Jackjun724
Copy link
Author

@skylot I found a new bug, If Jadx-gui does not open any files, a NullPointerException will be thrown when opening the setting window, I think it is because the decompiler is uninitialized when the file is not open. bug code

@skylot
Copy link
Owner

skylot commented Feb 22, 2022

I found a new bug

@Jackjun724 thanks! NPE fixed 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants