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 resolution_addr parameter to Ret2dlresolvePayload #2430

Closed
wants to merge 3 commits into from

Conversation

k4lizen
Copy link
Contributor

@k4lizen k4lizen commented Jul 24, 2024

This PR implements point number two from #2429 (thus it shouldn't close the issue).

  • Adds an optional parameter called resolution_addr to Ret2dlresolvePayload making the loaders _dl_runtime_resolve function write the resolved function address to that location instead of the beginning of the payload (as is the default).

@peace-maker
Copy link
Member

Thank you for the contribution! Can you add documentation for the parameter please? Best case would be to add an example on how and when to use it too.

@k4lizen
Copy link
Contributor Author

k4lizen commented Aug 6, 2024

Aa I messed up a git operation, I'll open a new branch.
Edit: here #2436

Eeeeh? Now that I closed it and created a new PR it looks fine? Maybe I didn't refresh the page after the force push...

@k4lizen k4lizen closed this Aug 6, 2024
@k4lizen k4lizen deleted the dlr branch August 6, 2024 23:05
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