]> git.baikalelectronics.ru Git - kernel.git/commit
signal: Fail sigqueueinfo if si_signo != sig
authorEric W. Biederman <ebiederm@xmission.com>
Tue, 25 Sep 2018 10:04:47 +0000 (12:04 +0200)
committerEric W. Biederman <ebiederm@xmission.com>
Wed, 3 Oct 2018 14:46:28 +0000 (16:46 +0200)
commit1eb6f7a08e53e226ad94994dfde9df67f4bbcf2d
tree629998c30d946e656f969a40eeaca77da6249fbd
parent4f7152214eef5550b0834755b8c50f2c24153c3f
signal: Fail sigqueueinfo if si_signo != sig

The kernel needs to validate that the contents of struct siginfo make
sense as siginfo is copied into the kernel, so that the proper union
members can be put in the appropriate locations.  The field si_signo
is a fundamental part of that validation.  As such changing the
contents of si_signo after the validation make no sense and can result
in nonsense values in the kernel.

As such simply fail if someone is silly enough to set si_signo out of
sync with the signal number passed to sigqueueinfo.

I don't expect a problem as glibc's sigqueue implementation sets
"si_signo = sig" and CRIU just returns to the kernel what the kernel
gave to it.

If there is some application that calls sigqueueinfo directly that has
a problem with this added sanity check we can revisit this when we see
what kind of crazy that application is doing.

Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
kernel/signal.c