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/signal.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/signal/signal.c') diff --git a/src/signal/signal.c b/src/signal/signal.c index 07346a8a..9d2f23ab 100644 --- a/src/signal/signal.c +++ b/src/signal/signal.c @@ -7,10 +7,10 @@ int __sigaction(int, const struct sigaction *, struct sigaction *); void (*signal(int sig, void (*func)(int)))(int) { - struct sigaction sa = { .sa_handler = func, .sa_flags = SA_RESTART }; - if (__sigaction(sig, &sa, &sa) < 0) + struct sigaction sa_old, sa = { .sa_handler = func, .sa_flags = SA_RESTART }; + if (__sigaction(sig, &sa, &sa_old) < 0) return SIG_ERR; - return sa.sa_handler; + return sa_old.sa_handler; } weak_alias(signal, bsd_signal); -- cgit v1.2.1