[Buildroot] [PATCH] tpm2-tss: fix build with BR2_FORTIFY_SOURCE_1

Peter Korsgaard peter at korsgaard.com
Tue Jan 15 07:52:16 UTC 2019


>>>>> "Peter" == Peter Korsgaard <peter at korsgaard.com> writes:

 > The configure script passes -U FORTIFY_SOURCE -D FORTIFY_SOURCE=2 by
 > default, which conflicts with BR2_FORTIFY_SOURCE_1 as -Werror is used:

 > <cross>-gcc ..  -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 .. -D_FORTIFY_SOURCE=1
 > <command-line>:0:0: error: "_FORTIFY_SOURCE" redefined [-Werror]

 > Disable this so the FORTIFY_SOURCE flags in TARGET_CFLAGS (if any) are used
 > instead.

 > Signed-off-by: Peter Korsgaard <peter at korsgaard.com>
 > ---
 >  package/tpm2-tss/tpm2-tss.mk | 6 ++++++
 >  1 file changed, 6 insertions(+)

 > diff --git a/package/tpm2-tss/tpm2-tss.mk b/package/tpm2-tss/tpm2-tss.mk
 > index f005e5d6b8..5b57973c6e 100644
 > --- a/package/tpm2-tss/tpm2-tss.mk
 > +++ b/package/tpm2-tss/tpm2-tss.mk
 > @@ -16,4 +16,10 @@ TPM2_TSS_CONF_OPTS = --with-crypto=ossl --disable-doxygen-doc
 >  # libssp is missing.
 >  TPM2_TSS_CONF_ENV =
 > ax_cv_check_cflags___________Wall__Werror_______fstack_protector_all=$(if
 > $(BR2_TOOLCHAIN_HAS_SSP),yes,no)
 
 > +# configure.ac uses FORTIFY_SOURCE=2 by default. Disable that so the
 > +# BR2_FORTIFY_SOURCE_* options in CFLAGS are used instead
 > +sTPM2_TSS_CONF_ENV += \

Ehh, there's naturally a typo here in the variable name. I'll send an
updated patch.

-- 
Bye, Peter Korsgaard


More information about the buildroot mailing list