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

Add basic libhooker API support #76

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

ren7995
Copy link

@ren7995 ren7995 commented Jun 12, 2021

DISCLAIMER: This is not in any way finished, I don't even know perl. It is my hopes this can be a good starting point that other, smarter people can use to build from
This pull request adds support for using the libhooker APIs directly if available, and falling back to MobileSubstrate

Does this provide any real benefit for end users? No, not really, the shims for MobileSubstrate functions on libhooker still work fine; the only case the shim could really be less efficient than directly supporting the libhooker API is when hooking C functions.

Quite simply, I made this pull request because Coolstar said "why don't you make a PR for logos then if it's that easy". It was that easy.

What needs to be done before merging:

  • Actually batch hooking for C functions as to not dlsym and call LHHookFunctions for every one (this probably would require a lot of work, but this would allow Logos hookf to utilize the benefits of libhooker's API)
  • Perhaps clean up the code

@kirb kirb marked this pull request as draft February 6, 2022 02:24
@L1ghtmann
Copy link
Member

Now that libhooker support has been merged for some time, do we want to close this?

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