From e949b78c30d949f4789db5d798c3179849ed688c Mon Sep 17 00:00:00 2001 From: Jim Warner Date: Sun, 23 Aug 2015 00:00:00 -0500 Subject: [PATCH] library: revert changes to 'look_up_our_self' function The patch below is where most proc_t fixed size arrays became simple pointers to char. In that commit changes to the above function were made so that dynamic memory was freed which included the program name (cmd) field. That change was prompted by a valgrind reported memory leak during development that no longer seems to exist. However, by keeping the look_up_our_self() changes the ps command without args then fails to report anything. So this patch just restores the expected old behavior. Reference(s): commit 3881a0844afe4d1b3cd512b2c2fd79e11bb0ed06 Signed-off-by: Jim Warner --- proc/readproc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/proc/readproc.c b/proc/readproc.c index f0de5699..061dde12 100644 --- a/proc/readproc.c +++ b/proc/readproc.c @@ -1426,9 +1426,7 @@ void look_up_our_self(proc_t *p) { fprintf(stderr, "Error, do this: mount -t proc proc /proc\n"); _exit(47); } - memset(p, 0, sizeof(*p)); stat2proc(ub.buf, p); // parse /proc/self/stat - free_acquired(p, 0); free(ub.buf); }