-
Notifications
You must be signed in to change notification settings - Fork 196
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
Soft-deprecate GNU_PROPERTY_AARCH64_FEATURE_1_PAC #121
Conversation
@smithp35 @DanielKristofKiss FYI clarifying the meaning of the PAC program property. |
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.
LGTM, thanks!
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.
Thanks for the update. A couple of small suggestions.
b80e9ed
to
220f98a
Compare
Users should not rely on this property to enquire about the status of the protection of an ELF file. If it is present, then yes, the file has been protected. If it is not present, it can still have been protected to some extent. The comment on Custom PLT protection has also been removed. This protection is seen as a separate feature to PAC-ret, since you could ask the linker to generate protected PLTs regardless of the presence of PAC-ret.
220f98a
to
1fd7e96
Compare
@smithp35 I updated a new patch with your two suggestions. |
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.
LGTM, thanks for the update.
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.
Thanks!
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.
One minor change
executable sections have been protected with Return Address Signing. | ||
Its use is optional, meaning that an ELF file where this feature bit | ||
is unset can still have Return Address signing enabled in some or all | ||
its executable sections. |
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.
of its executable sections
Users should not rely on this property to enquire about the status
of the protection of an ELF file. If it is present, then yes, the
file has been protected. If it is not present, it can still have
been protected to some extent.
The comment on Custom PLT protection has also been removed. This
protection is seen as a separate feature to PAC-ret, since you could
ask the linker to generate protected PLTs regardless of the presence
of PAC-ret.