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

Compiler Hardening Guide should recommend -fhardened option #240

Open
thomasnyman opened this issue Sep 21, 2023 · 3 comments · May be fixed by #492
Open

Compiler Hardening Guide should recommend -fhardened option #240

thomasnyman opened this issue Sep 21, 2023 · 3 comments · May be fixed by #492

Comments

@thomasnyman
Copy link
Contributor

thomasnyman commented Sep 21, 2023

Soon the Best Practices guide should recommend the newly proposed -fhardened option:
patch and thread

Originally posted by @edelsohn in #234 (comment)

@thomasnyman
Copy link
Contributor Author

This was discussed during the C/C++ Compiler BP Call 2023-09.27.

The proposed-fhardened flag seems like a good addition to the guide but given that it's currently targeting GCC 14 (expected in May 2024) the overall consensus among the participants to the discussion was to postpone its addition until it is available in an official GCC release.

In the meantime, as -fhardened is an 'umbrella option' that enables a set of flags already available in current GCC (and Clang) releases the constituent options not yet covered by the guide can be considered already now.

I've opened issues #245 and #247 for -ftrivial-auto-var-init and -fcf-protection respectively.

Leaving this issue open to track -fhardened as the discussion around the functionality stabilizes in the GCC community.

@SecurityCRob
Copy link
Contributor

Has this been addressed by the C/C++ Compiler Hardening options guide? @gkunz @thomasnyman @david-a-wheeler

@thomasnyman
Copy link
Contributor Author

@SecurityCRob Yes, this is being worked on in #492 now that GCC 14 has been released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants