Messages in this thread | ![/](/images/icornerl.gif) | | Date | Tue, 16 Jun 2020 08:57:25 -0600 | From | Tycho Andersen <> | Subject | Re: [PATCH v4 08/11] selftests/seccomp: Make kcmp() less required |
| |
On Mon, Jun 15, 2020 at 08:25:21PM -0700, Kees Cook wrote: > The seccomp tests are a bit noisy without CONFIG_CHECKPOINT_RESTORE (due > to missing the kcmp() syscall). The seccomp tests are more accurate with > kcmp(), but it's not strictly required. Refactor the tests to use > alternatives (comparing fd numbers), and provide a central test for > kcmp() so there is a single XFAIL instead of many. Continue to produce > warnings for the other tests, though. > > Additionally adds some more bad flag EINVAL tests to the addfd selftest. > > Signed-off-by: Kees Cook <keescook@chromium.org>
This looks fine, but I wonder if this is enough motivation for taking kcmp() out of CONFIG_CHECKPOINT_RESTORE guards?
Tycho
| ![\](/images/icornerr.gif) |