[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-5.2 3/3] linux-user/sparc: Don't zero high half of PC, N
From: |
Laurent Vivier |
Subject: |
Re: [PATCH for-5.2 3/3] linux-user/sparc: Don't zero high half of PC, NPC, PSR in sigreturn |
Date: |
Tue, 10 Nov 2020 07:55:22 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0 |
Le 05/11/2020 à 22:23, Peter Maydell a écrit :
> 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>
> ---
> linux-user/sparc/signal.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/linux-user/sparc/signal.c b/linux-user/sparc/signal.c
> index c315704b389..d12adc8e6ff 100644
> --- a/linux-user/sparc/signal.c
> +++ b/linux-user/sparc/signal.c
> @@ -247,7 +247,7 @@ long do_sigreturn(CPUSPARCState *env)
> {
> abi_ulong sf_addr;
> struct target_signal_frame *sf;
> - uint32_t up_psr, pc, npc;
> + abi_ulong up_psr, pc, npc;
> target_sigset_t set;
> sigset_t host_set;
> int i;
>
Applied to my linux-user-for-5.2 branch.
Thanks,
Laurent
- Re: [PATCH for-5.2 1/3] linux-user/sparc: Fix errors in target_ucontext structures, (continued)
[PATCH for-5.2 2/3] linux-user/sparc: Correct set/get_context handling of fp and i7, Peter Maydell, 2020/11/05
[PATCH for-5.2 3/3] linux-user/sparc: Don't zero high half of PC, NPC, PSR in sigreturn, Peter Maydell, 2020/11/05
Re: [PATCH for-5.2 0/3] linux-user: fix various sparc64 guest bugs, Mark Cave-Ayland, 2020/11/10