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

[Lifter] Addresses representation in the decompiled code #52

Closed
mari-mari opened this issue Feb 2, 2022 · 2 comments · Fixed by #397
Closed

[Lifter] Addresses representation in the decompiled code #52

mari-mari opened this issue Feb 2, 2022 · 2 comments · Fixed by #397
Assignees
Labels
feature-request New feature or request priority-low Low priority issue

Comments

@mari-mari
Copy link
Collaborator

mari-mari commented Feb 2, 2022

Proposal

Currently we represent addresses within the binary as hexadecimal:
extractme.zip main

...
var_3 = i + 0x804c020;
...

For instance, Binary Ninja prepends in such cases data_ prefix for data.
It would be nice to somehow integrate section information into decompiled code.

Current suggestions were:
data_0x804c020
0x804c020 \* data *\
0x804c020 \* data + 12 *\ - with offset from the section start.
???

Approach

Collect couple of ideas, note them, chose the most optimal 😈 and implement.

@mari-mari mari-mari added feature-request New feature or request priority-low Low priority issue labels Feb 2, 2022
@fnhartmann
Copy link
Collaborator

/cib

Copy link
Contributor

github-actions bot added a commit that referenced this issue Mar 13, 2024
@fnhartmann fnhartmann self-assigned this Mar 13, 2024
@fnhartmann fnhartmann changed the title [GUI] Addresses representation in the decompiled code [Lifter] Addresses representation in the decompiled code Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request New feature or request priority-low Low priority issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants