From: Haavard Skinnemoen <hskinnemoen@atmel.com>
Date: Mon, 19 May 2008 10:26:38 +0000 (+0200)
Subject: Conditionally add -fno-stack-protector to CFLAGS
X-Git-Url: http://git.dujemihanovic.xyz/?a=commitdiff_plain;h=28eab0d77352b84885f938759bf2612b7bf0bc44;p=u-boot.git

Conditionally add -fno-stack-protector to CFLAGS

When compile-testing on powerpc, I get errors like this:

net/nfs.c:422: undefined reference to `__stack_chk_fail_local'

This seems to be because -fstack-protector is on by default, so
let's explicitly disable it on all architectures that support the
option.

The Ubuntu toolchain is affected by this problem, and according to
Mike Frysinger, Gentoo has been running with SSP enabled for years.
More and more distros are turning SSP on by default, so this problem
is likely to get worse in the future.

Also, powerpc just happens to be one of the arches I do
compile-testing on. There may be other arches affected by this too.

Signed-off-by: Haavard Skinnemoen <hskinnemoen@atmel.com>
---

diff --git a/config.mk b/config.mk
index b08b7a7593..1dac29be57 100644
--- a/config.mk
+++ b/config.mk
@@ -172,6 +172,8 @@ else
 CFLAGS := $(CPPFLAGS) -Wall -Wstrict-prototypes
 endif
 
+CFLAGS += $(call cc-option,-fno-stack-protector)
+
 # avoid trigraph warnings while parsing pci.h (produced by NIOS gcc-2.9)
 # this option have to be placed behind -Wall -- that's why it is here
 ifeq ($(ARCH),nios)