Skip to content

Conversation

@XXMA16
Copy link

@XXMA16 XXMA16 commented Nov 10, 2025

Prerequisite Checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Updated relevant documentation (if needed).

Description of changes

The canary and return address overwrites were done two frames higher (instead of only one).

An explicit statement that the canary overwrite should be commented out before proceeding with replacing the return address with &pawned has also been added.

Explanation

addr points to [RBP-16], since at [RBP-8] resides the canary, meaning that addr[1] will overwrite it. The return address is found two machine words higher than that, at [RBP+8]

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.

1 participant