diff options
| author | Oleg Nesterov <oleg@redhat.com> | 2016-05-23 16:24:02 -0700 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2017-11-26 13:50:14 +0000 |
| commit | 1e4a536feedf2f37157db8d54b97a1223fa8cd03 (patch) | |
| tree | 03fb32f72f4e50d1ec2133868d746de0dd1cca60 /include/uapi | |
| parent | 1c40e4aa7d573c7f52c48a46ea789541f6c16a06 (diff) | |
signal: move the "sig < SIGRTMIN" check into siginmask(sig)
commit 5c8ccefdf46c5f87d87b694c7fbc04941c2c99a5 upstream.
All the users of siginmask() must ensure that sig < SIGRTMIN. sig_fatal()
doesn't and this is wrong:
UBSAN: Undefined behaviour in kernel/signal.c:911:6
shift exponent 32 is too large for 32-bit type 'long unsigned int'
the patch doesn't add the neccesary check to sig_fatal(), it moves the
check into siginmask() and updates other callers.
Link: http://lkml.kernel.org/r/20160517195052.GA15187@redhat.com
Reported-by: Meelis Roos <mroos@linux.ee>
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'include/uapi')
0 files changed, 0 insertions, 0 deletions