[Buildroot] [git commit] boot/arm-trusted-firmware: change ENABLE_STACK_PROTECTOR value when disabled

Peter Korsgaard peter at korsgaard.com
Wed Sep 1 20:52:31 UTC 2021


>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni at bootlin.com> writes:

 > commit: https://git.buildroot.net/buildroot/commit/?id=5f432df7e2d26d6d57d750cbee1a573c1f886e26
 > branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master

 > Since commit
 > cf176128ec4ad60c29c8828cab1887f69edb9473 ("boot/arm-trusted-firmware:
 > add SSP option"), we are passing ENABLE_STACK_PROTECTOR=none when we
 > want to disable SSP usage in TF-A. While this works fine in recent
 > versions of TF-A, older versions such as TF-A will end up passing
 > -fstack-protector-none in this situation, which fails as this is not a
 > valid gcc option (the valid gcc option is -fno-stack-protector).

 > To solve this, we pass ENABLE_STACK_PROTECTOR=0 which was in older
 > TF-A versions used to say "don't do anything with SSP", and is also
 > still supported in newer versions of TF-A.

 > Fixes:

 >  https://gitlab.com/buildroot.org/buildroot/-/jobs/1478738580

 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni at bootlin.com>

Committed to 2021.02.x and 2021.05.x, thanks.

-- 
Bye, Peter Korsgaard


More information about the buildroot mailing list