Skip to content

Commit 7172f1c

Browse files
wtarreaupaulmckrcu
authored andcommitted
selftests/nolibc: condition some tests on /proc existence
If /proc is not available (program run inside a chroot or without sufficient permissions), it's better to disable the associated tests. Some will be preserved like the ones which check for a failure to create some entries there since they're still supposed to fail. Signed-off-by: Willy Tarreau <[email protected]> Signed-off-by: Paul E. McKenney <[email protected]>
1 parent 1a5454f commit 7172f1c

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

tools/testing/selftests/nolibc/nolibc-test.c

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -413,11 +413,15 @@ int test_getdents64(const char *dir)
413413
int run_syscall(int min, int max)
414414
{
415415
struct stat stat_buf;
416+
int proc;
416417
int test;
417418
int tmp;
418419
int ret = 0;
419420
void *p1, *p2;
420421

422+
/* <proc> indicates whether or not /proc is mounted */
423+
proc = stat("/proc", &stat_buf) == 0;
424+
421425
for (test = min; test >= 0 && test <= max; test++) {
422426
int llen = 0; // line length
423427

@@ -438,12 +442,12 @@ int run_syscall(int min, int max)
438442
CASE_TEST(chdir_root); EXPECT_SYSZR(1, chdir("/")); break;
439443
CASE_TEST(chdir_dot); EXPECT_SYSZR(1, chdir(".")); break;
440444
CASE_TEST(chdir_blah); EXPECT_SYSER(1, chdir("/blah"), -1, ENOENT); break;
441-
CASE_TEST(chmod_net); EXPECT_SYSZR(1, chmod("/proc/self/net", 0555)); break;
442-
CASE_TEST(chmod_self); EXPECT_SYSER(1, chmod("/proc/self", 0555), -1, EPERM); break;
443-
CASE_TEST(chown_self); EXPECT_SYSER(1, chown("/proc/self", 0, 0), -1, EPERM); break;
445+
CASE_TEST(chmod_net); EXPECT_SYSZR(proc, chmod("/proc/self/net", 0555)); break;
446+
CASE_TEST(chmod_self); EXPECT_SYSER(proc, chmod("/proc/self", 0555), -1, EPERM); break;
447+
CASE_TEST(chown_self); EXPECT_SYSER(proc, chown("/proc/self", 0, 0), -1, EPERM); break;
444448
CASE_TEST(chroot_root); EXPECT_SYSZR(1, chroot("/")); break;
445449
CASE_TEST(chroot_blah); EXPECT_SYSER(1, chroot("/proc/self/blah"), -1, ENOENT); break;
446-
CASE_TEST(chroot_exe); EXPECT_SYSER(1, chroot("/proc/self/exe"), -1, ENOTDIR); break;
450+
CASE_TEST(chroot_exe); EXPECT_SYSER(proc, chroot("/proc/self/exe"), -1, ENOTDIR); break;
447451
CASE_TEST(close_m1); EXPECT_SYSER(1, close(-1), -1, EBADF); break;
448452
CASE_TEST(close_dup); EXPECT_SYSZR(1, close(dup(0))); break;
449453
CASE_TEST(dup_0); tmp = dup(0); EXPECT_SYSNE(1, tmp, -1); close(tmp); break;
@@ -464,7 +468,7 @@ int run_syscall(int min, int max)
464468
CASE_TEST(link_root1); EXPECT_SYSER(1, link("/", "/"), -1, EEXIST); break;
465469
CASE_TEST(link_blah); EXPECT_SYSER(1, link("/proc/self/blah", "/blah"), -1, ENOENT); break;
466470
CASE_TEST(link_dir); EXPECT_SYSER(1, link("/", "/blah"), -1, EPERM); break;
467-
CASE_TEST(link_cross); EXPECT_SYSER(1, link("/proc/self/net", "/blah"), -1, EXDEV); break;
471+
CASE_TEST(link_cross); EXPECT_SYSER(proc, link("/proc/self/net", "/blah"), -1, EXDEV); break;
468472
CASE_TEST(lseek_m1); EXPECT_SYSER(1, lseek(-1, 0, SEEK_SET), -1, EBADF); break;
469473
CASE_TEST(lseek_0); EXPECT_SYSER(1, lseek(0, 0, SEEK_SET), -1, ESPIPE); break;
470474
CASE_TEST(mkdir_root); EXPECT_SYSER(1, mkdir("/", 0755), -1, EEXIST); break;

0 commit comments

Comments
 (0)