]> granicus.if.org Git - procps-ng/commitdiff
pwdx & libprocps-ng: Hurd does not have MAX_PATH defined
authorDaniel Novotny <unknown@example.com>
Wed, 18 Feb 2009 12:05:22 +0000 (13:05 +0100)
committerCraig Small <csmall@enc.com.au>
Sun, 18 Dec 2011 11:30:00 +0000 (22:30 +1100)
A patch from Debian.

Bug-Redhat: https://bugzilla.redhat.com/show_bug.cgi?id=485243
Bug-Debian: http://bugs.debian.org/588677
Backported-by: Sami Kerola <kerolasa@iki.fi>
proc/readproc.c
pwdx.c

index 3d5e877c4f5902da21c83ccd7319051b8c03ab63..b85cc887683ae31236e1a6d4c07ad7b8b94d91a0 100644 (file)
@@ -1354,7 +1354,7 @@ proc_data_t *readproctab3 (int(*want_task)(proc_t *buf), PROCTAB *restrict const
  * and filled out proc_t structure.
  */
 proc_t * get_proc_stats(pid_t pid, proc_t *p) {
-       static char path[PATH_MAX], sbuf[1024];
+       static char path[32], sbuf[1024];
        struct stat statbuf;
 
        sprintf(path, "/proc/%d", pid);
diff --git a/pwdx.c b/pwdx.c
index ac42736bf5e04a0887223264b5c62be98c6441e2..755c17806f82b5bdedb8e7e892858bda3d6e4837 100644 (file)
--- a/pwdx.c
+++ b/pwdx.c
@@ -35,7 +35,6 @@ static void version(void)
 
 int main(int argc, char* argv[])
 {
-     static char buf[PATH_MAX+1];      // null terminate string via static
      regex_t re;
      int i;
 
@@ -59,6 +58,8 @@ int main(int argc, char* argv[])
 
      for (i = 1; i < argc; i++) {
           if (regexec(&re, argv[i], 0, NULL, 0) != 0) {
+               /* Constant 27 is the length of the error string "pwdx: ... " */
+               char buf[27 + strlen (argv[i]) + 1];
                snprintf(buf, sizeof buf, "pwdx: invalid process id: %s\n", argv[i]);
                buf[sizeof(buf)-1] = '\0';
                die(buf);
@@ -69,9 +70,14 @@ int main(int argc, char* argv[])
 
      regfree(&re);
 
+     int alloclen = 128;
+     char *pathbuf = malloc(alloclen);
+
      for (i = 1; i < argc; i++) {
-          char * s = buf;
+          char * s;
           int len;
+          /* Constant 10 is the length of strings "/proc/" + "/cwd" + 1 */
+          char buf[10 + strlen(argv[i]) + 1];
           
           // At this point, all arguments are in the form /proc/nnnn
           // or nnnn, so a simple check based on the first char is
@@ -83,10 +89,16 @@ int main(int argc, char* argv[])
 
           // buf contains /proc/nnnn/cwd symlink name on entry, the
           // target of that symlink on return
-          if ((len = readlink(buf, buf, PATH_MAX)) < 0) {
+          while ((len = readlink(buf, pathbuf, alloclen)) == alloclen) {
+               alloclen *= 2;
+               pathbuf = realloc(pathbuf, alloclen);
+          }
+
+          if (len < 0) {
                s = strerror(errno == ENOENT ? ESRCH : errno);
           } else {
-               buf[len] = 0;
+               pathbuf[len] = 0;
+               s = pathbuf;
           }
 
           printf("%s: %s\n", argv[i], s);