[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