fix wrong sigaction syscall ABI on mips*, or1k, microblaze, riscv64
authorRich Felker <dalias@aerifal.cx>
Thu, 9 Feb 2023 16:52:44 +0000 (11:52 -0500)
committerRich Felker <dalias@aerifal.cx>
Thu, 9 Feb 2023 17:33:35 +0000 (12:33 -0500)
commit269d193820342dc109f39909d78fb30f4c978f76
tree2acbae503e21dca93feb669b5d5a890ee915a797
parentea3b40a321e751e016948087ef23ae7b9e8e0150
fix wrong sigaction syscall ABI on mips*, or1k, microblaze, riscv64

we wrongly defined a dummy SA_RESTORER flag on these archs, despite
the kernel interface not actually having such a feature. on archs
which lack SA_RESTORER, the kernel sigaction structure also lacks the
restorer function pointer member, which means the signal mask appears
at a different offset. the kernel was thereby interpreting the bits of
the code address as part of the signal set to be masked while handling
the signal.

this patch removes the erroneous SA_RESTORER definitions from archs
which do not have it, makes access to the member conditional on
whether SA_RESTORER is defined for the arch, and removes the
now-unused asm for the affected archs.

because there are reportedly versions of qemu-user which also use the
wrong ABI here, the old ksigaction struct size is preserved with an
unused member at the end. this is harmless and mitigates the risk of
such a bug turning into a buffer overflow onto the sigaction
function's stack.
14 files changed:
arch/microblaze/bits/signal.h
arch/mips/bits/signal.h
arch/mips/ksigaction.h
arch/mips64/bits/signal.h
arch/mips64/ksigaction.h
arch/mipsn32/bits/signal.h
arch/mipsn32/ksigaction.h
arch/or1k/bits/signal.h
arch/riscv64/bits/signal.h
src/internal/ksigaction.h
src/signal/mips/restore.s [deleted file]
src/signal/mips64/restore.s [deleted file]
src/signal/mipsn32/restore.s [deleted file]
src/signal/sigaction.c