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 feature to print heap chunks of all arenas #722

Merged
merged 3 commits into from
Sep 22, 2021

Conversation

theguy147
Copy link
Collaborator

Add feature to print heap chunks of all arenas

Description/Motivation/Screenshots

This PR adds a new flag --all/-a to the heap chunks command to print out all chunks of all arenas at once (see Screenshot).

Furthermore the style of the Arena output is adjusted to match the style of the Chunks:

  • The word "Arena" is colorized (blue).
  • The whitespace right after "Arena" has been removed.

image
image

How Has This Been Tested?

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

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.

@theguy147 theguy147 requested review from hugsy and Grazfather and removed request for hugsy September 20, 2021 21:37
@theguy147 theguy147 self-assigned this Sep 20, 2021
Copy link
Owner

@hugsy hugsy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

gef.py Outdated Show resolved Hide resolved
@hugsy hugsy merged commit f5df117 into hugsy:dev Sep 22, 2021
@theguy147 theguy147 deleted the heap-chunks-all branch September 22, 2021 19:57
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.

3 participants