Re: [PATCH 1/2] kbuild: remove cc-option test of -fno-stack-protector

From: Kees Cook
Date: Fri Jun 26 2020 - 16:13:23 EST


On Fri, Jun 26, 2020 at 12:09:37PM -0700, Nick Desaulniers wrote:
> On Fri, Jun 26, 2020 at 12:00 PM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote:
> > CFLAGS_NO_HARDENING := $(call cc-option, -fno-PIC,) $(call cc-option, -fno-pic,) \
> > - $(call cc-option, -fno-stack-protector,) \
> > - $(call cc-option, -fno-stack-protector-all,)
> > + -fno-stack-protector $(call cc-option, -fno-stack-protector-all)
>
> Just curious, looks like we could do the same for
> `-fno-stack-protector-all`, here or tree-wide, right? Wait, what
> compiler recognizes -fno-stack-protector-all?
> https://godbolt.org/z/QFQKE_

That is from ancient times, IIRC. -fno-stack-protector should be
sufficient.

--
Kees Cook