[hurd,commited]: Fix static initialization with -fstack-protector-strong

Nick Alcock nix@esperi.org.uk
Fri Sep 15 16:38:00 GMT 2017


On 27 Aug 2017, Samuel Thibault outgrape:

>     When linked statically, TLS initialization is not achieved before
>     mach_init and alike, so ssp accesses to tcbhead's stack_guard would
>     crash.  We can just avoid using ssp in the few functions needed before
>     TLS is set up.

Oh blast I forgot all about Hurd didn't I.

Sorry.

> diff --git a/hurd/Makefile b/hurd/Makefile
> index 9205822b24..b44b9b80c6 100644
> --- a/hurd/Makefile
> +++ b/hurd/Makefile
> @@ -81,6 +81,10 @@ $(inlines:%=$(objpfx)%.c): $(objpfx)%-inlines.c: %.h
>  	 echo '#include "$<"') > $@-new
>  	mv -f $@-new $@
>  generated += $(inlines:=.c)
> +
> +# Avoid ssp before TLS is initialized.
> +CFLAGS-hurdstartup.o = $(no-stack-protector)
> +CFLAGS-RPC_exec_startup_get_info.o = $(no-stack-protector)

This certainly looks like the right sort of thing to be doing, though I
don't have anything I can test it on right now.

-fstack-protector-all might well comb out some more cases.

-- 
NULL && (void)



More information about the Libc-alpha mailing list