about summary refs log tree commit diff
path: root/src/signal/sigsetjmp.c
diff options
context:
space:
mode:
authorRich Felker <dalias@aerifal.cx>2012-07-03 20:07:33 -0400
committerRich Felker <dalias@aerifal.cx>2012-07-03 20:07:33 -0400
commitd6c0efe106b1016108207fb6872820c06dcef4f8 (patch)
tree700300d0b4ff576121241befdcf865678cde4e5c /src/signal/sigsetjmp.c
parente6129e6d836e5f4725d9b14ba7457b32e24adc61 (diff)
downloadmusl-d6c0efe106b1016108207fb6872820c06dcef4f8.tar.gz
musl-d6c0efe106b1016108207fb6872820c06dcef4f8.tar.xz
musl-d6c0efe106b1016108207fb6872820c06dcef4f8.zip
jmp_buf overhaul fixing several issues
on arm, the location of the saved-signal-mask flag and mask were off
by one between sigsetjmp and siglongjmp, causing incorrect behavior
restoring the signal mask. this is because the siglongjmp code assumed
an extra slot was in the non-sig jmp_buf for the flag, but arm did not
have this. now, the extra slot is removed for all archs since it was
useless.

also, arm eabi requires jmp_buf to have 8-byte alignment. we achieve
that using long long as the type rather than with non-portable gcc
attribute tags.
Diffstat (limited to 'src/signal/sigsetjmp.c')
-rw-r--r--src/signal/sigsetjmp.c8
1 files changed, 3 insertions, 5 deletions
diff --git a/src/signal/sigsetjmp.c b/src/signal/sigsetjmp.c
index 5c937074..01ba0dff 100644
--- a/src/signal/sigsetjmp.c
+++ b/src/signal/sigsetjmp.c
@@ -9,9 +9,7 @@
 
 int sigsetjmp(sigjmp_buf buf, int save)
 {
-	unsigned long *flag = buf + sizeof(jmp_buf)/sizeof(long);
-	sigset_t *mask = (void *)(flag + 1);
-	if ((*flag = save))
-		sigprocmask (SIG_SETMASK, NULL, mask);
-	return setjmp((void *)buf);
+	if ((buf->__fl = save))
+		pthread_sigmask(SIG_SETMASK, 0, (sigset_t *)buf->__ss);
+	return setjmp(buf->__jb);
 }