diff options
| author | Richard Henderson | 2017-10-31 13:53:52 +0100 |
|---|---|---|
| committer | Riku Voipio | 2017-11-07 20:58:12 +0100 |
| commit | 7f047de18cf6cd51915d4fdbe1a7e66c38b45e98 (patch) | |
| tree | afc7a3c0835b773f9ef8aa1e9fba8d6b736e68d1 /linux-user/signal.c | |
| parent | Update version for v2.11.0-rc0 release (diff) | |
| download | qemu-7f047de18cf6cd51915d4fdbe1a7e66c38b45e98.tar.gz qemu-7f047de18cf6cd51915d4fdbe1a7e66c38b45e98.tar.xz qemu-7f047de18cf6cd51915d4fdbe1a7e66c38b45e98.zip | |
linux-user: Restrict usage of sa_restorer
Reading and writing to an sa_restorer member that isn't supposed to
exist corrupts user memory. Introduce TARGET_ARCH_HAS_SA_RESTORER,
similar to the kernel's __ARCH_HAS_SA_RESTORER.
Reported-by: Helge Deller <deller@gmx.de>
Signed-off-by: Richard Henderson <rth@twiddle.net>
Signed-off-by: Riku Voipio <riku.voipio@linaro.org>
Diffstat (limited to 'linux-user/signal.c')
| -rw-r--r-- | linux-user/signal.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/linux-user/signal.c b/linux-user/signal.c index 7a238aaea1..cf35473671 100644 --- a/linux-user/signal.c +++ b/linux-user/signal.c @@ -777,7 +777,7 @@ int do_sigaction(int sig, const struct target_sigaction *act, if (oact) { __put_user(k->_sa_handler, &oact->_sa_handler); __put_user(k->sa_flags, &oact->sa_flags); -#if !defined(TARGET_MIPS) +#ifdef TARGET_ARCH_HAS_SA_RESTORER __put_user(k->sa_restorer, &oact->sa_restorer); #endif /* Not swapped. */ @@ -787,7 +787,7 @@ int do_sigaction(int sig, const struct target_sigaction *act, /* FIXME: This is not threadsafe. */ __get_user(k->_sa_handler, &act->_sa_handler); __get_user(k->sa_flags, &act->sa_flags); -#if !defined(TARGET_MIPS) +#ifdef TARGET_ARCH_HAS_SA_RESTORER __get_user(k->sa_restorer, &act->sa_restorer); #endif /* To be swapped in target_to_host_sigset. */ |