about summary refs log tree commit diff
diff options
context:
space:
mode:
authorMark Wielaard <mark@klomp.org>2022-06-26 22:17:17 +0200
committerMark Wielaard <mark@klomp.org>2022-07-29 18:52:12 +0200
commit325ba824b0611e14e96b38dbc486acf255f9e568 (patch)
tree6321e599741ffe5a4a0e157c07bbc105cd5a6b8d
parent35363b53cef00c3b9886b2c1f45612869f304960 (diff)
downloadglibc-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.c7
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);