[PATCH] ARM: fix atags_to_fdt with stack-protector-strong
Arnd Bergmann
arnd at arndb.de
Fri Dec 18 13:09:12 PST 2015
On Friday 18 December 2015 13:04:59 Kees Cook wrote:
> +ifeq ($(CONFIG_ARM_ATAG_DTB_COMPAT),y)
> +CFLAGS_atags_to_fdt.o := -fno-stack-protector
> +CFLAGS_fdt.o := -fno-stack-protector
> +CFLAGS_fdt_ro.o := -fno-stack-protector
> +CFLAGS_fdt_rw.o := -fno-stack-protector
> +CFLAGS_fdt_wip.o := -fno-stack-protector
> +endif
I'm pretty sure you don't need the ifeq there, you can simply define those
flags unconditionally.
You can't just add -fno-stack-protector unconditionally, because that
breaks building the kernel with toolchains that are older than stack-protector,
so this should be
CFLAGS_obj.o += $(call cc-option, -fno-stack-protector)
Other than that, the patch looks ok.
Arnd
More information about the linux-arm-kernel
mailing list