-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Frida snippet don't work for constructors anymore #1714
Comments
According to #1605 What version of Frida and frida-server do you use? |
The current one, |
Hm. I am not a Frida expert, but I believe that a As a workaround, I can move template for this snippet to the preferences, so it will be possible to edit it, but this solution is not flexible enough, because a lot of code used to form a strings in snippet. So maybe script for that will be better (I will try to make it). |
@skylot I created a controlled environment to test this. I created new Android Studio Project with a basic Activity and a basic class with constructor. Here is the failing example with I am happy to provide the code or condunct further tests. Seems like the frida workflow here is strange, created a ticket for it in the frida repo: frida/frida#2302 |
See frida/frida#2302 (comment) and So I think even |
@xxr0ss thank you for clarifying Therefore the FridaAction needs to be adapted further. |
…(PR #1715) * Change callMethodName of constructors in Frida action * Fix format violation in FridaAction * Fix format violation in FridaAction
Thanks @skylot! The generated code for the example above is now
Looking good! |
Great! Thanks for verify. |
When generating a Frida snippet via jadx for a constructor, the code looks like this:
This generates the following error in the latest frida versions (haven't tested how far back that dates):
When using
let ret = this.$init();
instead, it works for me.The text was updated successfully, but these errors were encountered: