-
Notifications
You must be signed in to change notification settings - Fork 524
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
Dynamic relocations support #353
Dynamic relocations support #353
Conversation
This is a great PR, thank you! |
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.
find some mistakes
__IMAGE_INDIR_CONTROL_TRANSFER_DYNAMIC_RELOCATION_format__ = ( | ||
"IMAGE_INDIR_CONTROL_TRANSFER_DYNAMIC_RELOCATION", | ||
( | ||
"I:12,PageRelativeOffset", |
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.
type error, which is actually WORD
"I,DynamicValueRelocTableOffset", | ||
"H,DynamicValueRelocTableSection", | ||
"H,Reserved2", | ||
"I,GuardRFVerifyStackPointerFunctionPointer" |
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.
missing comma
#374 has fixed them |
Adds support of
IMAGE_DYNAMIC_RELOCATION_TABLE
in PE file.There are also issues about extending format of
IMAGE_LOAD_CONFIG_DIRECTORY
here #220 and here #317.I added support of load config because it contains dynamic relocation information.
From winnt.h:
Some description of dynamic relocations structures could also be found here.
And some example code here.
Also related to that is a retpoline presentation with details about dynamic relocation types like
IMAGE_IMPORT_CONTROL_TRANSFER_DYNAMIC_RELOCATION
.