]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
powerpc: Sanitize stack pointer in signal handling code
authorJosh Boyer <jwboyer@linux.vnet.ibm.com>
Tue, 28 Apr 2009 15:11:57 +0000 (11:11 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sat, 2 May 2009 17:24:48 +0000 (10:24 -0700)
commit606624763556fd32a8d68796ce9ef03ae57df5e1
tree8267cb83e2d4188f3d5a084d6382ced3e751bd2a
parent664b8ee764c295ff3bfd9736094a036dcc0ebda2
powerpc: Sanitize stack pointer in signal handling code

This has been backported to 2.6.27.x from commit efbda86098 in Linus' tree.

On powerpc64 machines running 32-bit userspace, we can get garbage bits in the
stack pointer passed into the kernel.  Most places handle this correctly, but
the signal handling code uses the passed value directly for allocating signal
stack frames.

This fixes the issue by introducing a get_clean_sp function that returns a
sanitized stack pointer.  For 32-bit tasks on a 64-bit kernel, the stack
pointer is masked correctly.  In all other cases, the stack pointer is simply
returned.

Additionally, we pass an 'is_32' parameter to get_sigframe now in order to
get the properly sanitized stack.  The callers are know to be 32 or 64-bit
statically.

Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
arch/powerpc/include/asm/processor.h
arch/powerpc/kernel/signal.c
arch/powerpc/kernel/signal.h
arch/powerpc/kernel/signal_32.c
arch/powerpc/kernel/signal_64.c