-
Notifications
You must be signed in to change notification settings - Fork 82
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
Add support to Hermes version 83, 89 and improve contribution process #25
base: add-hbc-83-89-and-improve-contribution
Are you sure you want to change the base?
Add support to Hermes version 83, 89 and improve contribution process #25
Conversation
Draft/hbc v84
Helo @bongtrop and @anantshri, is there any way i can help with this? Is it possible that https://github.com/cyfinoid/hbctool is still not fully working yet? It appears Happy to know how i can help. |
I am having issues too, trying to decompile an index.android.bundle which contains HBC Version 89, however I get the following error while it tries to unpack the instruction.hasm: Traceback (most recent call last): An updates? Happy to run against a previous version and output some debugging |
OK so I figured this out, for version 89 at least. Turns out the structure.json in the data folder was very outdated, I wet through and manually updated it by looking at the ByteCodeFileFormat.h from the raw folder. Also the Parser needed to account for BigInts, FunctionSourceTable, etc.. I believe similar updates will be required for version 84, and so on, to bring them up to date with the correct file format. Will look to create a pull request for hbc89 files, and someone else can use that as a basis to update other version. Thanks for all those that contributed to this! |
Thank you for your work. When it will be merged? |
|
And the error:
It looking for |
the problem is deeper, bc is not correctly read |
I have fixed the issues related to bytecode v84 with #31 |
Very thank for https://github.com/cyfinoid @anantshri