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

Fixes for MachO: @rpath TODO, Issue #44, Issue #6 #45

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

Conversation

titison
Copy link

@titison titison commented Nov 11, 2023

Completes this TODO from the README.md:
@rpath contents into shared::VecRpath similar to DT_RPATH/DT_RUNPATH on ELFs

The whole @rpath is now displayed similar to the RPATH and RUNPATH for ELFs, when one is found.

EDIT: Fixes #44 and #6 aswell now.

@titison titison changed the title Show complete @rpath variable if present Fixes for MachO: @rpath TODO, Issue #44, Issue #6 Nov 11, 2023
@titison
Copy link
Author

titison commented Nov 11, 2023

I included fixes to #44 and #6 in this pull request so i do not completely spam the PR section. I hope thats okay as these are all small fixes.

marcograss added a commit to marcograss/checksec.rs that referenced this pull request May 23, 2024
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.

Issues with has_arc() and has_canary() Functions Not Detecting Symbols in Mach-O Files
1 participant