diff options
author | Mark Wielaard <mark@klomp.org> | 2022-06-26 22:17:17 +0200 |
---|---|---|
committer | Mark Wielaard <mark@klomp.org> | 2022-07-29 18:52:12 +0200 |
commit | 325ba824b0611e14e96b38dbc486acf255f9e568 (patch) | |
tree | 6321e599741ffe5a4a0e157c07bbc105cd5a6b8d | |
parent | 35363b53cef00c3b9886b2c1f45612869f304960 (diff) | |
download | glibc-325ba824b0611e14e96b38dbc486acf255f9e568.tar.gz glibc-325ba824b0611e14e96b38dbc486acf255f9e568.tar.xz glibc-325ba824b0611e14e96b38dbc486acf255f9e568.zip |
tst-pidfd.c: UNSUPPORTED if we get EPERM on valid pidfd_getfd call
pidfd_getfd can fail for a valid pidfd with errno EPERM for various reasons in a restricted environment. Use FAIL_UNSUPPORTED in that case. Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org> Reviewed-by: Carlos O'Donell <carlos@redhat.com>
-rw-r--r-- | sysdeps/unix/sysv/linux/tst-pidfd.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/sysdeps/unix/sysv/linux/tst-pidfd.c b/sysdeps/unix/sysv/linux/tst-pidfd.c index d93b6faa6f..037af22290 100644 --- a/sysdeps/unix/sysv/linux/tst-pidfd.c +++ b/sysdeps/unix/sysv/linux/tst-pidfd.c @@ -142,6 +142,13 @@ do_test (void) xrecvfrom (sockets[0], &remote_fd, sizeof (remote_fd), 0, NULL, 0); int fd = pidfd_getfd (pidfd, remote_fd, 0); + /* pidfd_getfd may fail with EPERM if the process does not have + PTRACE_MODE_ATTACH_REALCREDS permissions. This means the call + may be denied if the process doesn't have CAP_SYS_PTRACE or + if a LSM security_ptrace_access_check denies access. */ + if (fd == -1 && errno == EPERM) + FAIL_UNSUPPORTED ("don't have permission to use pidfd_getfd on pidfd, " + "skipping test"); TEST_VERIFY (fd > 0); char *path = xasprintf ("/proc/%d/fd/%d", pid, remote_fd); |