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

Add MEDIA_BUILD_HARDENING option #1242

Merged
merged 1 commit into from Jan 7, 2022
Merged

Add MEDIA_BUILD_HARDENING option #1242

merged 1 commit into from Jan 7, 2022

Conversation

ffontaine
Copy link
Contributor

@ffontaine ffontaine commented Aug 11, 2021

Add MEDIA_BUILD_HARDENING option to allow the user to disable hardening options such as stack-protector-all or FORTIFY SOURCE 2 which are not always available (e.g. fortify source 2 is only available on glibc >= 6 and not musl/uclibc-ng)

Signed-off-by: Fabrice Fontaine fontaine.fabrice@gmail.com

Add MEDIA_BUILD_HARDENING option to allow the user to disable hardening
options such as stack-protector-all or FORTIFY SOURCE 2 which are not
always available (e.g. fortify source 2 is only available on glibc >= 6
and not musl/uclibc-ng)

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Copy link
Contributor

@Sherry-Lin Sherry-Lin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@haizhouxu could you take a look?

@bkuhls
Copy link
Contributor

bkuhls commented Nov 24, 2021

ping

@Sherry-Lin Sherry-Lin added the Common memory, surface, ddi label Nov 26, 2021
@Sherry-Lin
Copy link
Contributor

@cqian2 could you take a look?

@ffontaine
Copy link
Contributor Author

Do you plan to merge this PR or do you have additional comments?

@dvrogozh dvrogozh added the verifying PR: fix ready and verifying with build/test label Jan 7, 2022
@intel-mediadev intel-mediadev merged commit 8eab97a into intel:master Jan 7, 2022
@dvrogozh
Copy link
Contributor

@ffontaine: merged.

@ffontaine
Copy link
Contributor Author

Indeed, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Common memory, surface, ddi verifying PR: fix ready and verifying with build/test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants