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

feat: create hooking code for all internal methods in the class #2328

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

RuffaloLavoisier
Copy link
Contributor

@RuffaloLavoisier RuffaloLavoisier commented Oct 29, 2024

Description

When writing hooking codes for a class, write hooking codes for all internal methods for the class.

hook_m

@jpstotz
Copy link
Collaborator

jpstotz commented Oct 29, 2024

If you want to hook all methods of a class I would recommend to use frida-trace instead of frida. There you can select classes and methods by pattern (wildcard).

@RuffaloLavoisier
Copy link
Contributor Author

@jpstotz
Hi there 😄
IMHO - It would be great if this feature could be added to JADX. For beginners, it would make writing hooking code quick and easy. Also, the current code snippet for the class simply returns the name in a JavaScript-like format, rather than actually hooking the class, which makes it less meaningful for writing hooking code.

@skylot
Copy link
Owner

skylot commented Oct 30, 2024

If you want to hook all methods of a class I would recommend to use frida-trace

I agree with @jpstotz here, also class can have lots of methods, so hooking all of them will spam a lot of messages in logs.
But as long as current method snippets already includes class hook, class snippet become useless.
So, I think, it is fine to make it more useful. And in case you need several methods, removing is easier rather than copy snippet for each method 😄

@skylot skylot merged commit 2661b91 into skylot:master Oct 30, 2024
4 checks passed
@RuffaloLavoisier RuffaloLavoisier deleted the feature/FridaHook4AllMethod branch November 1, 2024 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants