>From 543f2a7c00ca8bc3a050ffb3ab936ba6fb1d67c2 Mon Sep 17 00:00:00 2001 From: Bruno Haible Date: Wed, 27 May 2020 19:41:38 +0200 Subject: [PATCH 06/18] get_ppid_of: Make more robust in multithreaded applications. * lib/get_ppid_of.c (get_ppid_of): Pass an O_CLOEXEC flag to open(). * modules/get_ppid_of (Depends-on): Add 'open'. --- ChangeLog | 6 ++++++ lib/get_ppid_of.c | 10 +++++----- modules/get_ppid_of | 1 + 3 files changed, 12 insertions(+), 5 deletions(-) diff --git a/ChangeLog b/ChangeLog index 4139e7c..fec6b52 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2020-05-27 Bruno Haible + get_ppid_of: Make more robust in multithreaded applications. + * lib/get_ppid_of.c (get_ppid_of): Pass an O_CLOEXEC flag to open(). + * modules/get_ppid_of (Depends-on): Add 'open'. + +2020-05-27 Bruno Haible + get-rusage-as: Make more robust in multithreaded applications. * lib/get-rusage-as.c (get_rusage_as_via_setrlimit): Pass an O_CLOEXEC flag to open(). diff --git a/lib/get_ppid_of.c b/lib/get_ppid_of.c index 1560e7d..5579cd9 100644 --- a/lib/get_ppid_of.c +++ b/lib/get_ppid_of.c @@ -74,7 +74,7 @@ get_ppid_of (pid_t pid) int fd; sprintf (filename, "/proc/%u/status", (unsigned int) pid); - fd = open (filename, O_RDONLY); + fd = open (filename, O_RDONLY | O_CLOEXEC); if (fd >= 0) { char buf[4096 + 1]; @@ -115,7 +115,7 @@ get_ppid_of (pid_t pid) int fd; sprintf (filename, "/proc/%u/status", (unsigned int) pid); - fd = open (filename, O_RDONLY); + fd = open (filename, O_RDONLY | O_CLOEXEC); if (fd >= 0) { char buf[4096 + 1]; @@ -152,7 +152,7 @@ get_ppid_of (pid_t pid) int fd; sprintf (filename, "/proc/%u/psinfo", (unsigned int) pid); - fd = open (filename, O_RDONLY); + fd = open (filename, O_RDONLY | O_CLOEXEC); if (fd >= 0) { char buf[4096 + 1]; @@ -190,7 +190,7 @@ get_ppid_of (pid_t pid) int fd; sprintf (filename, "/proc/%u/psinfo", (unsigned int) pid); - fd = open (filename, O_RDONLY); + fd = open (filename, O_RDONLY | O_CLOEXEC); if (fd >= 0) { /* The contents is a 'struct psinfo'. But since 'struct psinfo' @@ -283,7 +283,7 @@ get_ppid_of (pid_t pid) int fd; sprintf (filename, "/proc/pinfo/%u", pid); - fd = open (filename, O_RDONLY); + fd = open (filename, O_RDONLY | O_CLOEXEC); if (0 <= fd) { prpsinfo_t buf; diff --git a/modules/get_ppid_of b/modules/get_ppid_of index 4f2cad1..84e3d79 100644 --- a/modules/get_ppid_of +++ b/modules/get_ppid_of @@ -9,6 +9,7 @@ Depends-on: extensions sys_types unistd +open configure.ac: -- 2.7.4