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

cmake: only use _FORTIFY_SOURCE with hardening #1456

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

illwieckz
Copy link
Member

@illwieckz illwieckz commented Dec 6, 2024

That test was almost always true (well, except with non-release MinGW builds no one run).

The previous code means that _FORTIFY_SOURCE was used on MinGW on recent releases because USE_HARDENING is used.

I don't really know what that tes meant anyway.

I noticed this because doing ASAN build on macOS spammed lots of warning that _FORTIFY_SOURCE was redefined. It looks like an ASAN build on macOS forces a different _FORTIFY_SOURCE value so we need to be able to disable it, something we can do by disabling hardening.

@illwieckz illwieckz force-pushed the illwieckz/hardening branch from f7ce889 to aa32c01 Compare December 6, 2024 14:20
@slipher
Copy link
Member

slipher commented Dec 12, 2024

This makes most builds less secure just to fix warnings on one obscure use case.

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

Successfully merging this pull request may close these issues.

2 participants