diff options
author | Rich Felker <dalias@aerifal.cx> | 2015-04-17 21:54:42 -0400 |
---|---|---|
committer | Rich Felker <dalias@aerifal.cx> | 2015-04-17 21:54:42 -0400 |
commit | 583e55122e767b1586286a0d9c35e2a4027998ab (patch) | |
tree | 369bc1f292ee44f7bdd702203aaf535d97569caf /src/signal/or1k | |
parent | 81e18eb3cd61f7b68a8f99b3e6c728f2a4d79221 (diff) | |
download | musl-583e55122e767b1586286a0d9c35e2a4027998ab.tar.gz musl-583e55122e767b1586286a0d9c35e2a4027998ab.tar.bz2 musl-583e55122e767b1586286a0d9c35e2a4027998ab.tar.xz musl-583e55122e767b1586286a0d9c35e2a4027998ab.zip |
redesign sigsetjmp so that signal mask is restored after longjmp
the conventional way to implement sigsetjmp is to save the signal mask
then tail-call to setjmp; siglongjmp then restores the signal mask and
calls longjmp. the problem with this approach is that a signal already
pending, or arriving between unmasking of signals and restoration of
the saved stack pointer, will have its signal handler run on the stack
that was active before siglongjmp was called. this can lead to
unbounded stack usage when siglongjmp is used to leave a signal
handler.
in the new design, sigsetjmp saves its own return address inside the
extended part of the sigjmp_buf (outside the __jmp_buf part used by
setjmp) then calls setjmp to save a jmp_buf inside its own execution.
it then tail-calls to __sigsetjmp_tail, which uses the return value of
setjmp to determine whether to save the current signal mask or restore
a previously-saved mask.
as an added bonus, this design makes it so that siglongjmp and longjmp
are identical. this is useful because the __longjmp_chk function we
need to add for ABI-compatibility assumes siglongjmp and longjmp are
the same, but for different reasons -- it was designed assuming either
can access a flag just past the __jmp_buf indicating whether the
signal masked was saved, and act on that flag. however, early versions
of musl did not have space past the __jmp_buf for the non-sigjmp_buf
version of jmp_buf, so our setjmp cannot store such a flag without
risking clobbering memory on (very) old binaries.
Diffstat (limited to 'src/signal/or1k')
-rw-r--r-- | src/signal/or1k/sigsetjmp.s | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/src/signal/or1k/sigsetjmp.s b/src/signal/or1k/sigsetjmp.s index 70a29223..350c00e2 100644 --- a/src/signal/or1k/sigsetjmp.s +++ b/src/signal/or1k/sigsetjmp.s @@ -6,17 +6,18 @@ sigsetjmp: __sigsetjmp: l.sfeq r4, r0 l.bf plt(setjmp) - l.sw 52(r3), r4 /* buf->__fl = save */ - l.addi r1, r1, -8 - l.sw 0(r1), r9 - l.sw 4(r1), r3 - l.addi r5, r3, 56 /* buf->__ss */ - l.add r4, r0, r0 - l.jal plt(sigprocmask) - l.ori r3, r0, 2 /* SIG_SETMASK */ + l.sw 52(r3), r9 + l.sw 52+4+8(r3), r20 - l.lwz r9, 0(r1) - l.lwz r3, 4(r1) - l.j plt(setjmp) - l.addi r1, r1, 8 + l.jal plt(setjmp) + l.ori r20, r3, 0 + + l.ori r4, r11, 0 + l.ori r3, r20, 0 + + l.lwz r9, 52(r3) + +.hidden __sigsetjmp_tail + l.j plt(__sigsetjmp_tail) + l.lwz r20, 52+4+8(r3) |