Re: [PATCH] lkdtm: add test for executing .rodata

From: Arnd Bergmann
Date: Wed Feb 17 2016 - 16:44:50 EST


On Tuesday 16 February 2016 13:49:04 Kees Cook wrote:
> }
>
> +static void __attribute__((__section__(".rodata,\"a\",@progbits#")))
> +do_nothing_rodata(void)
> +{
> + return;
> +}
> +
> static void do_nothing(void)
> {
>

I think this also needs to be marked "noinline" to ensure that the
function does not get eliminated. I've seen clang do that on
execute_location() recently and submitted a patch for that.

Arnd