mbox series

[0/2] gcc-plugins: fixes for arm_ssp_per_task_plugin

Message ID 20190118105807.26425-1-ard.biesheuvel@linaro.org
Headers show
Series gcc-plugins: fixes for arm_ssp_per_task_plugin | expand

Message

Ard Biesheuvel Jan. 18, 2019, 10:58 a.m. UTC 
A couple of fixes to permit newer versions of GCC to use the stack
protector plugin for ARM.

Ard Biesheuvel (2):
  gcc-plugins: arm_ssp_per_task_plugin: sign extend the SP mask
  gcc-plugins: arm_ssp_per_task_plugin: fix for GCC 9+

 scripts/gcc-plugins/arm_ssp_per_task_plugin.c | 23 ++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

-- 
2.20.1

Comments

Kees Cook Jan. 20, 2019, 1:51 a.m. UTC | #1 
On Fri, Jan 18, 2019 at 2:58 AM Ard Biesheuvel
<ard.biesheuvel@linaro.org> wrote:
>

> A couple of fixes to permit newer versions of GCC to use the stack

> protector plugin for ARM.

>

> Ard Biesheuvel (2):

>   gcc-plugins: arm_ssp_per_task_plugin: sign extend the SP mask

>   gcc-plugins: arm_ssp_per_task_plugin: fix for GCC 9+


Nice; thanks! It seems like these should go into -rc4, yes? I'll get
them queued up for Linus...

-Kees

>

>  scripts/gcc-plugins/arm_ssp_per_task_plugin.c | 23 ++++++++++++++++++--

>  1 file changed, 21 insertions(+), 2 deletions(-)

>

> --

> 2.20.1

>



-- 
Kees Cook
Ard Biesheuvel Jan. 20, 2019, 3:43 p.m. UTC | #2 
On Sun, 20 Jan 2019 at 02:51, Kees Cook <keescook@chromium.org> wrote:
>

> On Fri, Jan 18, 2019 at 2:58 AM Ard Biesheuvel

> <ard.biesheuvel@linaro.org> wrote:

> >

> > A couple of fixes to permit newer versions of GCC to use the stack

> > protector plugin for ARM.

> >

> > Ard Biesheuvel (2):

> >   gcc-plugins: arm_ssp_per_task_plugin: sign extend the SP mask

> >   gcc-plugins: arm_ssp_per_task_plugin: fix for GCC 9+

>

> Nice; thanks! It seems like these should go into -rc4, yes? I'll get

> them queued up for Linus...

>


Yes, given that it is new code anyway, let's merge it as fixes.