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

Search for binding symbols also via LC_DYLD_CHAINED_FIXUPS #123

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

p-x9
Copy link
Contributor

@p-x9 p-x9 commented Apr 19, 2024

Hello, I have implemented the following Issue.
#117

To check if it works correctly, follow the steps below.

  1. Before introducing this pull request, set the Minimum Deployment of the app to higher than iOS14
  2. Execute the executeAntiHook function in the ViewController to check the print operation.
    (This function seems to be commented out now.)
    (The contents of `myPrint' must be commented out, otherwise it seems to loop infinitely.)
  3. Perhaps print("print has been antiHooked") doesn't actually print.
  4. It will be printed after the introduction of this pullquist.

https://github.com/p-x9/swift-anti-fishhook

@p-x9 p-x9 changed the title Search for binding symbols also via LC_DYLD_CHAINED_FIXUPS Search for binding symbols also via LC_DYLD_CHAINED_FIXUPS Apr 19, 2024
@r3ggi
Copy link
Collaborator

r3ggi commented May 20, 2024

Hey! Thanks for the PR. We'll definitely accept your PR but in the later time (probably when iOS 18 will be out). Your change requires the minimum app deployment of the app to iOS 14 which can break the backwards compatibility

@p-x9
Copy link
Contributor Author

p-x9 commented May 20, 2024

Thanks

Even in iOS14 or later, if -no_fixup_chains is set in the linker flags, the binding symbol will be included in the dyld info as before.

Therefore, I think backward compatibility must be maintained.

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