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

rop: Improve density #2048

Merged
merged 1 commit into from
Apr 19, 2022
Merged

rop: Improve density #2048

merged 1 commit into from
Apr 19, 2022

Conversation

Arusekk
Copy link
Member

@Arusekk Arusekk commented Apr 3, 2022

When the last gadget has no arguments and the gadget before has stack
arguments, put the last gadget directly in its return address slot
(instead of a stack adjustment gadget).

@Arusekk Arusekk force-pushed the dense-rop branch 3 times, most recently from ad901ca to cdaf936 Compare April 19, 2022 20:06
When the last gadget has no arguments and the gadget before has stack
arguments, put the last gadget directly in its return address slot
(instead of a stack adjustment gadget).
@Arusekk Arusekk merged commit fc372d1 into Gallopsled:dev Apr 19, 2022
@Arusekk Arusekk deleted the dense-rop branch April 19, 2022 21:25
gogo2464 pushed a commit to gogo2464/pwntools that referenced this pull request Sep 10, 2023
When the last gadget has no arguments and the gadget before has stack
arguments, put the last gadget directly in its return address slot
(instead of a stack adjustment gadget).
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.

None yet

1 participant