Re: [PATCH] selftests/seccomp: Remove SIGSTOP si_pid check

From: shuah
Date: Tue Dec 11 2018 - 19:54:41 EST


On 12/11/18 5:43 PM, Kees Cook wrote:
On Thu, Dec 6, 2018 at 3:50 PM Kees Cook <keescook@xxxxxxxxxxxx> wrote:

Commit f149b3155744 ("signal: Never allocate siginfo for SIGKILL or SIGSTOP")
means that the seccomp selftest cannot check si_pid under SIGSTOP anymore.
Since it's believed[1] there are no other userspace things depending on the
old behavior, this removes the behavioral check in the selftest, since it's
more a "extra" sanity check (which turns out, maybe, not to have been
useful to test).

[1] https://lkml.kernel.org/r/CAGXu5jJaZAOzP1qFz66tYrtbuywqb+UN2SOA1VLHpCCOiYvYeg@xxxxxxxxxxxxxx

Reported-by: Tycho Andersen <tycho@xxxxxxxx>
Suggested-by: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
Signed-off-by: Kees Cook <keescook@xxxxxxxxxxxx>
---
Shuah, can you make sure that Linus gets this before v4.20 is released? Thanks!

Ping. Shuah, can you get this to Linus (or should I send it directly?)


I will send this. Thanks for the ping.

-- Shuah