[gentoo-commits] proj/pax-utils:master commit in: /

[Mike Frysinger]

commit:     a8a823e6acf88625fd482e15b2ba69c5f165fe46
Author:     Mike Frysinger <vapier <AT> gentoo <DOT> org>
AuthorDate: Wed Jan 24 22:51:40 2024 +0000
Commit:     Mike Frysinger <vapier <AT> gentoo <DOT> org>
CommitDate: Wed Jan 24 22:51:40 2024 +0000
URL:        https://gitweb.gentoo.org/proj/pax-utils.git/commit/?id=a8a823e6

pspax: replace proc_fopen with fopenat_r

Switch to the common helper we have in paxinc already that does
exactly the same thing as this proc_fopen.

Signed-off-by: Mike Frysinger <vapier <AT> gentoo.org>

 pspax.c | 37 ++++++++-----------------------------
 1 file changed, 8 insertions(+), 29 deletions(-)

diff --git a/pspax.c b/pspax.c
index 97d51c6..81392b1 100644
--- a/pspax.c
+++ b/pspax.c
@@ -40,33 +40,12 @@ static pid_t show_pid = 0;
 static uid_t show_uid = (uid_t)-1;
 static gid_t show_gid = (gid_t)-1;
 
-static int proc_open(int pfd, const char *file)
-{
-       return openat(pfd, file, O_RDONLY|O_CLOEXEC);
-}
-
-static FILE *proc_fopen(int pfd, const char *file)
-{
-       int fd;
-       FILE *fp;
-
-       fd = proc_open(pfd, file);
-       if (fd == -1)
-               return NULL;
-
-       fp = fdopen(fd, "re");
-       if (fp == NULL)
-               close(fd);
-
-       return fp;
-}
-
 static elfobj *proc_readelf(int pfd)
 {
        int fd;
        elfobj *elf;
 
-       fd = proc_open(pfd, "exe");
+       fd = openat(pfd, "exe", O_RDONLY|O_CLOEXEC);
        if (fd == -1)
                return NULL;
 
@@ -80,7 +59,7 @@ static const char *get_proc_name_cmdline(int pfd)
        FILE *fp;
        static char str[1024];
 
-       fp = proc_fopen(pfd, "cmdline");
+       fp = fopenat_r(pfd, "cmdline");
        if (fp == NULL)
                return NULL;
 
@@ -107,7 +86,7 @@ static const char *get_proc_name(int pfd)
        if (wide_output)
                return get_proc_name_cmdline(pfd);
 
-       fp = proc_fopen(pfd, "stat");
+       fp = fopenat_r(pfd, "stat");
        if (fp == NULL)
                return NULL;
 
@@ -142,7 +121,7 @@ static int get_proc_maps(int pfd)
        FILE *fp;
        static char str[BUFSIZ];
 
-       if ((fp = proc_fopen(pfd, "maps")) == NULL)
+       if ((fp = fopenat_r(pfd, "maps")) == NULL)
                return -1;
 
        while (fgets(str, sizeof(str), fp)) {
@@ -178,7 +157,7 @@ static int print_executable_mappings(int pfd)
        FILE *fp;
        static char str[BUFSIZ];
 
-       if ((fp = proc_fopen(pfd, "maps")) == NULL)
+       if ((fp = fopenat_r(pfd, "maps")) == NULL)
                return -1;
 
        while (fgets(str, sizeof(str), fp)) {
@@ -224,7 +203,7 @@ static const char *get_proc_status(int pfd, const char 
*name)
        size_t len;
        static char str[BUFSIZ];
 
-       if ((fp = proc_fopen(pfd, "status")) == NULL)
+       if ((fp = fopenat_r(pfd, "status")) == NULL)
                return NULL;
 
        len = strlen(name);
@@ -248,7 +227,7 @@ static const char *get_pid_attr(int pfd)
        char *p;
        static char buf[BUFSIZ];
 
-       if ((fp = proc_fopen(pfd, "attr/current")) == NULL)
+       if ((fp = fopenat_r(pfd, "attr/current")) == NULL)
                return NULL;
 
        if (fgets(buf, sizeof(buf), fp) != NULL)
@@ -265,7 +244,7 @@ static const char *get_pid_addr(int pfd)
        char *p;
        static char buf[BUFSIZ];
 
-       if ((fp = proc_fopen(pfd, "ipaddr")) == NULL)
+       if ((fp = fopenat_r(pfd, "ipaddr")) == NULL)
                return NULL;
 
        if (fgets(buf, sizeof(buf), fp) != NULL)