8000 expose frida-gum underlying writer object to allow users to use funct… by jejuisland87654 · Pull Request #198 · frida/frida-rust · GitHub
[go: up one dir, main page]

Skip to content

expose frida-gum underlying writer object to allow users to use funct… - #198

Merged
s1341 merged 3 commits into
frida:mainfrom
jejuisland87654:expose-writer-gum-object
May 5, 2025
Merged

s1341 merged 3 commits into
frida:mainfrom
jejuisland87654:expose-writer-gum-object

Conversation

@jejuisland87654
Copy link
Copy Markdown
Contributor

…ions that are not exposed in

@jejuisland87654
jejuisland87654 force-pushed the expose-writer-gum-object branch from 388c859 to b7d0cdb Compare March 6, 2025 17:09
@s1341
s1341 commented Mar 9, 2025
Copy link
Copy Markdown
Contributor

I'll accept this... but I'd prefer we add all the missing methods, can you open a PR to do so?

@jejuisland87654
Copy link
Copy Markdown
Contributor Author

@s1341

I added bindings for gum_arm64_writer_put_nop which is the main one I needed but there are dozens of functions for the writer and the stalker in Frida so I don't think it's realistic to add a wrapper for all of them...

@s1341
s1341 commented Mar 30, 2025
Copy link
Copy Markdown
Contributor

I think we really should wrap them all....

@jejuisland87654
Copy link
Copy Markdown
Contributor Author

@s1341 From a quick look seems there are 83 functions for the GumArm64Writer alone and 47 functions for the GumStalker, while some of them accept non-trivial parameters, so I think it's a bit out of the scope of this PR to wrap them all... any chance to merge the current change in the meantime?

@s1341
s1341 merged commit 269f0c6 into frida:main May 5, 2025
@s1341
s1341 commented May 5, 2025
Copy link
Copy Markdown
Contributor

Please open a new PR for the rest.

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.

2 participants

0