motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-12-18 05:28:36 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions

linux-user/sparc: Don't zero high half of PC, NPC, PSR in sigreturn

Browse source 
The function do_sigreturn() tries to store the PC, NPC and PSR in
uint32_t local variables, which implicitly drops the high half of
these fields for 64-bit guests.

The usual effect was that a guest which used signals would crash on
return from a signal unless it was lucky enough to take it while the
PC was in the low 4GB of the address space.  In particular, Debian
/bin/dash and /bin/bash would segfault after executing external
commands.

Use abi_ulong, which is the type these fields all have in the
__siginfo_t struct.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20201105212314.9628-4-peter.maydell@linaro.org>
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
This commit is contained in:
Peter Maydell 2020-11-05 21:23:14 +00:00 • committed by Laurent Vivier
parent 266b41582e
commit c3ab5df2f5
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
linux-user/sparc/signal.c
View file

@ -247,7 +247,7 @@ long do_sigreturn(CPUSPARCState *env)
{ {
abi_ulong sf_addr; abi_ulong sf_addr;
struct target_signal_frame *sf; struct target_signal_frame *sf;
uint32_t up_psr, pc, npc; abi_ulong up_psr, pc, npc;
target_sigset_t set; target_sigset_t set;
sigset_t host_set; sigset_t host_set;
int i; int i;