diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2016-06-12 20:57:41 -0400 |
|---|---|---|
| committer | Sven Eden <yamakuzure@gmx.net> | 2017-06-16 10:12:59 +0200 |
| commit | cf527c6873b86266ab75e56f28e2eada23866296 (patch) | |
| tree | 8e0fabe69c9999bda0faa37c21fc372a54e3da78 /src/basic/process-util.c | |
| parent | 6bc15ba0015c5c35d2d3e68d0ed193e6c5e5f81d (diff) | |
process-util: remove broken support for pid==0
Our functions that query /proc/pid/ support using pid==0 to mean
self. get_process_id also seemed to support that, but it was not implemented
correctly: the result should be in *uid, not returned, and also it gave
completely bogus result when called from get_process_gid(). But afaict,
get_process_{uid,gid} were never called with pid==0, so it's not an actual
bug. Remove the broken code to avoid confusion.
Diffstat (limited to 'src/basic/process-util.c')
| -rw-r--r-- | src/basic/process-util.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/basic/process-util.c b/src/basic/process-util.c index f418b85b9..faadb756c 100644 --- a/src/basic/process-util.c +++ b/src/basic/process-util.c @@ -321,9 +321,6 @@ static int get_process_id(pid_t pid, const char *field, uid_t *uid) { assert(field); assert(uid); - if (pid == 0) - return getuid(); - p = procfs_file_alloca(pid, "status"); f = fopen(p, "re"); if (!f) { |