aboutsummaryrefslogtreecommitdiff
path: root/linux-user
diff options
context:
space:
mode:
authorTom Musta <tommusta@gmail.com>2014-06-30 08:13:36 -0500
committerAlexander Graf <agraf@suse.de>2014-09-08 12:50:46 +0200
commitfbdc200ac2d0d29e88ee6af9b77810c0f84265a6 (patch)
tree80d19ddd4d86c40a8f2a4d6b4d1652b4d46c1fb1 /linux-user
parent2e14072f9e859272c7b94b8e189bd30bb4954aa1 (diff)
downloadqemu-fbdc200ac2d0d29e88ee6af9b77810c0f84265a6.zip
qemu-fbdc200ac2d0d29e88ee6af9b77810c0f84265a6.tar.gz
qemu-fbdc200ac2d0d29e88ee6af9b77810c0f84265a6.tar.bz2
linux-user: Fix Stack Pointer Bug in PPC setup_rt_frame
The code that sets the stack frame back pointer is incorrect for the setup_rt_frame() code; qemu will abort (SIGSEGV) in some environments. The setup_frame code was fixed in commit beb526b12134a6b6744125deec5a7fe24a8f92e3 but the setup_rt_frame code was not. Make the setup_rt_frame code consistent with the setup_frame code. Signed-off-by: Tom Musta <tommusta@gmail.com> Signed-off-by: Alexander Graf <agraf@suse.de>
Diffstat (limited to 'linux-user')
-rw-r--r--linux-user/signal.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/linux-user/signal.c b/linux-user/signal.c
index 26929c5..2952956 100644
--- a/linux-user/signal.c
+++ b/linux-user/signal.c
@@ -4751,7 +4751,7 @@ static void setup_rt_frame(int sig, struct target_sigaction *ka,
/* Create a stack frame for the caller of the handler. */
newsp = rt_sf_addr - (SIGNAL_FRAMESIZE + 16);
- __put_user(env->gpr[1], (target_ulong *)(uintptr_t) newsp);
+ err |= put_user(env->gpr[1], newsp, target_ulong);
if (err)
goto sigsegv;