-
Notifications
You must be signed in to change notification settings - Fork 339
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
rust: Add support for parsing ELF files #200
Conversation
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.
Neat! Could we add a test to make sure this can get some info out of a compiled method?
Yes, tests are upcoming. I have them but they are in later PRs which include integration with cirgen. |
This is a PR that's reminding me we need at least some place to write down itemized threats for our libraries. I don't see a need to block this PR right now, but anything doing parsing of as complex a format as ELF we will want written review and documentation of how we are using the library and potential points for things parser confusion if there's drift between our parser and the VM loader, weird header tricks that could lead to unexpected behavior, etc. |
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.
Ok!
No description provided.