[PATCH] ARM: decompressor: disable stack protector

Linus Walleij linus.walleij at linaro.org
Tue Oct 26 10:58:23 PDT 2021


On Tue, Oct 26, 2021 at 10:28 AM Ard Biesheuvel <ardb at kernel.org> wrote:

> Enabling the stack protector in the decompressor is of dubious value,
> given that it uses a fixed value for the canary, cannot print any output
> unless CONFIG_DEBUG_LL is enabled (which relies on board specific build
> time settings), and is already disabled for a good chunk of the code
> (libfdt).
>
> So let's just disable it in the decompressor. This will make it easier
> in the future to manage the command line options that would need to be
> removed again in this context for the TLS register based stack
> protector.
>
> Signed-off-by: Ard Biesheuvel <ardb at kernel.org>

Makes sense.
Reviewed-by: Linus Walleij <linus.walleij at linaro.org>

Yours,
Linus Walleij



More information about the linux-arm-kernel mailing list