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

Replace header_size with Header Regions system #479

Open
AaronOpfer opened this Issue Nov 29, 2016 · 1 comment

Comments

Projects
None yet
2 participants
@AaronOpfer
Contributor

AaronOpfer commented Nov 29, 2016

edb currently allows the architecture targets to specify that the PE/ELF headers have a certain number of bytes dedicated to their headers from their base address. This is the header_size call. This gives the GUI the ability to color the header area gray and also can give hints to the analysis engine.

However, the current system is a little naive. PE and ELF both support putting headers in arbitrary order at arbitrary positions. There can be gaps between what parts are considered headers (headers can even overlap!).

I propose that this be replaced with another mechanism that allows giving some sort of list of regions so that the header declaration can be considered more robust. Each region should have a start address, size, and probably a label or name that the GUI can show as a tooltip when mousing over the region.

eteran added a commit that referenced this issue Dec 20, 2016

@eteran

This comment has been minimized.

Show comment
Hide comment
@eteran

eteran Dec 20, 2016

Owner

The core functionality is in there. Now we just need to make use of it in things like the QDisassemblyView

Owner

eteran commented Dec 20, 2016

The core functionality is in there. Now we just need to make use of it in things like the QDisassemblyView

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment