|
 |
089393 |
From bba566504901b2c07885ecf325829875a96381a7 Mon Sep 17 00:00:00 2001
|
|
|
089393 |
From: =?UTF-8?q?=C3=81kos=20Uzonyi?= <uzonyi.akos@gmail.com>
|
|
|
089393 |
Date: Tue, 30 Jun 2020 17:20:12 +0200
|
|
|
089393 |
Subject: [PATCH 136/138] Use get_proc_pid for /proc paths
|
|
|
089393 |
|
|
|
089393 |
* mmap_cache.c (mmap_cache_rebuild_if_invalid): Use proc pid instead of
|
|
|
089393 |
tcp->pid for /proc path.
|
|
|
089393 |
* util.c (getfdproto): Likewise.
|
|
|
089393 |
(pidfd_get_pid): Likewise.
|
|
|
089393 |
* pathtrace.c (getfdpath_pid): Likewise.
|
|
|
089393 |
* strace.c (attach_tcb): Likewise.
|
|
|
089393 |
---
|
|
|
089393 |
mmap_cache.c | 2 +-
|
|
|
089393 |
pathtrace.c | 7 ++++++-
|
|
|
089393 |
strace.c | 2 +-
|
|
|
089393 |
util.c | 9 +++++++--
|
|
|
089393 |
4 files changed, 15 insertions(+), 5 deletions(-)
|
|
|
089393 |
|
|
|
089393 |
diff --git a/mmap_cache.c b/mmap_cache.c
|
|
|
089393 |
index 89c6225..9825df2 100644
|
|
|
089393 |
--- a/mmap_cache.c
|
|
|
089393 |
+++ b/mmap_cache.c
|
|
|
089393 |
@@ -84,7 +84,7 @@ mmap_cache_rebuild_if_invalid(struct tcb *tcp, const char *caller)
|
|
|
089393 |
return MMAP_CACHE_REBUILD_READY;
|
|
|
089393 |
|
|
|
089393 |
char filename[sizeof("/proc/4294967296/maps")];
|
|
|
089393 |
- xsprintf(filename, "/proc/%u/maps", tcp->pid);
|
|
|
089393 |
+ xsprintf(filename, "/proc/%u/maps", get_proc_pid(tcp));
|
|
|
089393 |
|
|
|
089393 |
FILE *fp = fopen_stream(filename, "r");
|
|
|
089393 |
if (!fp) {
|
|
|
089393 |
diff --git a/pathtrace.c b/pathtrace.c
|
|
|
089393 |
index f85cf14..87dc64b 100644
|
|
|
089393 |
--- a/pathtrace.c
|
|
|
089393 |
+++ b/pathtrace.c
|
|
|
089393 |
@@ -87,7 +87,12 @@ getfdpath_pid(pid_t pid, int fd, char *buf, unsigned bufsize)
|
|
|
089393 |
if (fd < 0)
|
|
|
089393 |
return -1;
|
|
|
089393 |
|
|
|
089393 |
- xsprintf(linkpath, "/proc/%u/fd/%u", pid, fd);
|
|
|
089393 |
+ int proc_pid = 0;
|
|
|
089393 |
+ translate_pid(NULL, pid, PT_TID, &proc_pid);
|
|
|
089393 |
+ if (!proc_pid)
|
|
|
089393 |
+ return -1;
|
|
|
089393 |
+
|
|
|
089393 |
+ xsprintf(linkpath, "/proc/%u/fd/%u", proc_pid, fd);
|
|
|
089393 |
n = readlink(linkpath, buf, bufsize - 1);
|
|
|
089393 |
/*
|
|
|
089393 |
* NB: if buf is too small, readlink doesn't fail,
|
|
|
089393 |
diff --git a/strace.c b/strace.c
|
|
|
089393 |
index 249533e..ef23f08 100644
|
|
|
089393 |
--- a/strace.c
|
|
|
089393 |
+++ b/strace.c
|
|
|
089393 |
@@ -1196,7 +1196,7 @@ attach_tcb(struct tcb *const tcp)
|
|
|
089393 |
unsigned int ntid = 0, nerr = 0;
|
|
|
089393 |
|
|
|
089393 |
if (followfork && tcp->pid != strace_child &&
|
|
|
089393 |
- xsprintf(procdir, task_path, tcp->pid) > 0 &&
|
|
|
089393 |
+ xsprintf(procdir, task_path, get_proc_pid(tcp)) > 0 &&
|
|
|
089393 |
(dir = opendir(procdir)) != NULL) {
|
|
|
089393 |
struct_dirent *de;
|
|
|
089393 |
|
|
|
089393 |
diff --git a/util.c b/util.c
|
|
|
089393 |
index 2568021..481144b 100644
|
|
|
089393 |
--- a/util.c
|
|
|
089393 |
+++ b/util.c
|
|
|
089393 |
@@ -501,7 +501,7 @@ getfdproto(struct tcb *tcp, int fd)
|
|
|
089393 |
if (fd < 0)
|
|
|
089393 |
return SOCK_PROTO_UNKNOWN;
|
|
|
089393 |
|
|
|
089393 |
- xsprintf(path, "/proc/%u/fd/%u", tcp->pid, fd);
|
|
|
089393 |
+ xsprintf(path, "/proc/%u/fd/%u", get_proc_pid(tcp), fd);
|
|
|
089393 |
r = getxattr(path, "system.sockprotoname", buf, bufsize - 1);
|
|
|
089393 |
if (r <= 0)
|
|
|
089393 |
return SOCK_PROTO_UNKNOWN;
|
|
|
089393 |
@@ -582,8 +582,13 @@ printdev(struct tcb *tcp, int fd, const char *path)
|
|
|
089393 |
pid_t
|
|
|
089393 |
pidfd_get_pid(pid_t pid_of_fd, int fd)
|
|
|
089393 |
{
|
|
|
089393 |
+ int proc_pid = 0;
|
|
|
089393 |
+ translate_pid(NULL, pid_of_fd, PT_TID, &proc_pid);
|
|
|
089393 |
+ if (!proc_pid)
|
|
|
089393 |
+ return -1;
|
|
|
089393 |
+
|
|
|
089393 |
char fdi_path[sizeof("/proc/%u/fdinfo/%u") + 2 * sizeof(int) * 3];
|
|
|
089393 |
- xsprintf(fdi_path, "/proc/%u/fdinfo/%u", pid_of_fd, fd);
|
|
|
089393 |
+ xsprintf(fdi_path, "/proc/%u/fdinfo/%u", proc_pid, fd);
|
|
|
089393 |
|
|
|
089393 |
FILE *f = fopen_stream(fdi_path, "r");
|
|
|
089393 |
if (!f)
|
|
|
089393 |
--
|
|
|
089393 |
2.1.4
|
|
|
089393 |
|