|
 |
fecd69 |
From 401d587d2deb78adcf62517654f955a24b2d537d Mon Sep 17 00:00:00 2001
|
|
|
fecd69 |
From: Todd Lewis <todd_lewis@unc.edu>
|
|
|
fecd69 |
Date: Mon, 25 Oct 2021 19:38:10 -0400
|
|
|
fecd69 |
Subject: [PATCH] fix uid/gid > 2^31
|
|
|
fecd69 |
|
|
|
fecd69 |
---
|
|
|
fecd69 |
pgrep.c | 10 +++++++---
|
|
|
fecd69 |
proc/readproc.h | 12 ++++++------
|
|
|
fecd69 |
2 files changed, 13 insertions(+), 9 deletions(-)
|
|
|
fecd69 |
|
|
|
fecd69 |
diff --git a/pgrep.c b/pgrep.c
|
|
|
fecd69 |
index 1905cd1d..7d731620 100644
|
|
|
fecd69 |
--- a/pgrep.c
|
|
|
fecd69 |
+++ b/pgrep.c
|
|
|
fecd69 |
@@ -229,8 +229,12 @@ static int strict_atol (const char *restrict str, long *restrict value)
|
|
|
fecd69 |
|
|
|
fecd69 |
for ( ; *str; ++str) {
|
|
|
fecd69 |
if (! isdigit (*str))
|
|
|
fecd69 |
- return (0);
|
|
|
fecd69 |
+ return 0;
|
|
|
fecd69 |
+ if (res >= LONG_MAX / 10)
|
|
|
fecd69 |
+ return 0;
|
|
|
fecd69 |
res *= 10;
|
|
|
fecd69 |
+ if (res >= LONG_MAX - (*str - '0'))
|
|
|
fecd69 |
+ return 0;
|
|
|
fecd69 |
res += *str - '0';
|
|
|
fecd69 |
}
|
|
|
fecd69 |
*value = sign * res;
|
|
|
fecd69 |
@@ -305,7 +309,7 @@ static int conv_uid (const char *restrict name, struct el *restrict e)
|
|
|
fecd69 |
xwarnx(_("invalid user name: %s"), name);
|
|
|
fecd69 |
return 0;
|
|
|
fecd69 |
}
|
|
|
fecd69 |
- e->num = (int) pwd->pw_uid;
|
|
|
fecd69 |
+ e->num = pwd->pw_uid;
|
|
|
fecd69 |
return 1;
|
|
|
fecd69 |
}
|
|
|
fecd69 |
|
|
|
fecd69 |
@@ -322,7 +326,7 @@ static int conv_gid (const char *restrict name, struct el *restrict e)
|
|
|
fecd69 |
xwarnx(_("invalid group name: %s"), name);
|
|
|
fecd69 |
return 0;
|
|
|
fecd69 |
}
|
|
|
fecd69 |
- e->num = (int) grp->gr_gid;
|
|
|
fecd69 |
+ e->num = grp->gr_gid;
|
|
|
fecd69 |
return 1;
|
|
|
fecd69 |
}
|
|
|
fecd69 |
|
|
|
fecd69 |
diff --git a/proc/readproc.h b/proc/readproc.h
|
|
|
fecd69 |
index 7905ea99..8d9ae392 100644
|
|
|
fecd69 |
--- a/proc/readproc.h
|
|
|
fecd69 |
+++ b/proc/readproc.h
|
|
|
fecd69 |
@@ -159,12 +159,12 @@ typedef struct proc_t {
|
|
|
fecd69 |
session, // stat session id
|
|
|
fecd69 |
nlwp, // stat,status number of threads, or 0 if no clue
|
|
|
fecd69 |
tgid, // (special) thread group ID, the POSIX PID (see also: tid)
|
|
|
fecd69 |
- tty, // stat full device number of controlling terminal
|
|
|
fecd69 |
- /* FIXME: int uids & gids should be uid_t or gid_t from pwd.h */
|
|
|
fecd69 |
- euid, egid, // stat(),status effective
|
|
|
fecd69 |
- ruid, rgid, // status real
|
|
|
fecd69 |
- suid, sgid, // status saved
|
|
|
fecd69 |
- fuid, fgid, // status fs (used for file access only)
|
|
|
fecd69 |
+ tty; // stat full device number of controlling terminal
|
|
|
fecd69 |
+ uid_t euid; gid_t egid; // stat(),status effective
|
|
|
fecd69 |
+ uid_t ruid; gid_t rgid; // status real
|
|
|
fecd69 |
+ uid_t suid; gid_t sgid; // status saved
|
|
|
fecd69 |
+ uid_t fuid; gid_t fgid; // status fs (used for file access only)
|
|
|
fecd69 |
+ int
|
|
|
fecd69 |
tpgid, // stat terminal process group id
|
|
|
fecd69 |
exit_signal, // stat might not be SIGCHLD
|
|
|
fecd69 |
processor; // stat current (or most recent?) CPU
|
|
|
fecd69 |
--
|
|
|
fecd69 |
GitLab
|
|
|
fecd69 |
|