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

Qemu-user mmap support #638

Merged
merged 1 commit into from
Apr 11, 2021
Merged

Qemu-user mmap support #638

merged 1 commit into from
Apr 11, 2021

Conversation

hugsy
Copy link
Owner

@hugsy hugsy commented Apr 9, 2021

This PR slightly improves #579: instead of hardcoding some addresses which would not work on non 64b archs, we completely assumes that the debugged binary is emulated by qemu and therefore show the qemu mem maping. This allows for a clean and effective way to have the correct page permissions set in gef (and might help oneself who tries to escape the qemu process 😃 )

The reasoning behind is mostly that when debugging we don't really care that it's qemu-emulated, and have a proper memory mapping is just that much more useful info.

Before
image
image

After
image
image

How Has This Been Tested?

Architecture Yes/No Comments
x86-32 ✔️
x86-64 ✔️
ARM ✔️
AARCH64 ✖️
MIPS ✖️
POWERPC ✖️
SPARC ✖️
RISC-V ✖️
make tests ✔️

Checklist

  • My PR was done against the dev branch, not master.
  • My code follows the code style of this project.
  • My change includes a change to the documentation, if required.
  • My change adds tests as appropriate.
  • I have read and agree to the CONTRIBUTING document.

…- which would not work on non 64b archs, we completely assumes that the debugged binary is emulated by qemu and therefore show the qemu mem maping. This allows for a clean and effective way to have the correct page permissions set in gef (and might help oneself who tries to escape the qemu process :) )
@hugsy hugsy added this to the [Stable Release] 2021.04 milestone Apr 9, 2021
@hugsy hugsy merged commit 00e05d3 into dev Apr 11, 2021
@hugsy hugsy deleted the qemuuser-mmapping branch April 11, 2021 01:55
@hugsy hugsy mentioned this pull request Apr 11, 2021
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants