On Thu, May 27, 2021 at 5:29 AM Kees Cook <keescook@xxxxxxxxxxxx> wrote: > > Since the open fds might not always start at "4" (especially when > running under kselftest, etc), start counting from the first assigned > fd, rather than using the more permissive EXPECT_GE(fd, 0). > > Signed-off-by: Kees Cook <keescook@xxxxxxxxxxxx> Nice cleanup, thanks! Just in case, tested it here, works fine. Feel free to add: Reviewed-by: Rodrigo Campos <rodrigo@xxxxxxxxxx> I can improve the selftest to test the new addfd flag we just added also in combination existing flags (like setting the fd number to use), and maybe also split the big chunk test, if you think that is valuable. Best, Rodrigo
