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

vmmap in the right way #1

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

Conversation

andreafioraldi
Copy link

@andreafioraldi andreafioraldi commented Dec 7, 2019

  • ugly camel case API is not supported in recent IDA versions. They were deprected in IDA 7.0 and abandoned later.
  • vmmap is intended to be parsed as a json. json.loads(idc.send_dbg_command("vmmap")) is better than hand crafted stuffs
  • idaapi.SEGPERM_* may not be consistent with the permissions in vmmap. idaapi.SEGPERM_* are the right constants, not SEG_PROT_*

@andreafioraldi
Copy link
Author

andreafioraldi commented Dec 7, 2019

I didn't tested the changes (don't force me to install Visual Studio plis)

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