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
Relocatable ELF Support #531
Conversation
rakuy0
commented
Jun 10, 2022
- Cleanup a bunch of duplicate classes for Elf section info and whitespace/typos/general spelling
- Make relocatable ELF work. Or at least a very simple example. ELF treats certain section info slightly different when the file is marked as relocatable.
- There's relocations (like R_AMD64_32S) that while we don't support quite yet (because they induce other issues), I would like to, so I added some sizing info to the relocation type so that we could one day support them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
love all the reduction of duplication between the different Elf32/64 classes.
i'm cringing somewhat about taking the logic out of Elf and shoving it into Vstruct. i get it... but i'm not particularly joyful about it.
i think the only thing i need to see is a comment at the duplicated constants between elf_lookup.py and vstruct.defs.elf.py