From 48748143a34fb534e51378ec82816d7bbfb22df7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timo=20Ter=C3=A4s?= Date: Tue, 30 Jul 2013 09:14:56 -0400 Subject: use separate sigaction buffers for old and new data in signal() it is needed since __sigaction uses restrict in parameters and sharing the buffer is technically an aliasing error. do the same for the syscall, as at least qemu-user does not handle it properly. --- src/signal/sigaction.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src/signal/sigaction.c') diff --git a/src/signal/sigaction.c b/src/signal/sigaction.c index 7a72a44b..1f09bb96 100644 --- a/src/signal/sigaction.c +++ b/src/signal/sigaction.c @@ -14,7 +14,7 @@ weak_alias(dummy, __pthread_self_def); int __libc_sigaction(int sig, const struct sigaction *restrict sa, struct sigaction *restrict old) { - struct k_sigaction ksa; + struct k_sigaction ksa, ksa_old; if (sa) { if ((uintptr_t)sa->sa_handler > 1UL) __pthread_self_def(); @@ -23,12 +23,12 @@ int __libc_sigaction(int sig, const struct sigaction *restrict sa, struct sigact ksa.restorer = (sa->sa_flags & SA_SIGINFO) ? __restore_rt : __restore; memcpy(&ksa.mask, &sa->sa_mask, sizeof ksa.mask); } - if (syscall(SYS_rt_sigaction, sig, sa?&ksa:0, old?&ksa:0, sizeof ksa.mask)) + if (syscall(SYS_rt_sigaction, sig, sa?&ksa:0, old?&ksa_old:0, sizeof ksa.mask)) return -1; if (old) { - old->sa_handler = ksa.handler; - old->sa_flags = ksa.flags; - memcpy(&old->sa_mask, &ksa.mask, sizeof ksa.mask); + old->sa_handler = ksa_old.handler; + old->sa_flags = ksa_old.flags; + memcpy(&old->sa_mask, &ksa_old.mask, sizeof ksa_old.mask); } return 0; } -- cgit v1.2.1