These messages can be .rodata, so make them even more const.
[oweals/busybox.git] / ps.c
diff --git a/ps.c b/ps.c
index b9ff5bd0bc1ce5db29091e306256241ca159429b..24f806c4b35d3e5dc45bd23c18e6b1883efa4d88 100644 (file)
--- a/ps.c
+++ b/ps.c
@@ -2,8 +2,8 @@
 /*
  * Mini ps implementation(s) for busybox
  *
- * Copyright (C) 1999 by Lineo, inc.  Written by Erik Andersen
- * <andersen@lineo.com>, <andersee@debian.org>
+ * Copyright (C) 1999,2000,2001 by Lineo, inc.  
+ * Written by Erik Andersen <andersen@lineo.com>, <andersee@debian.org>
  *
  *
  * This contains _two_ implementations of ps for Linux.  One uses the
  *
  */
 
-#include "internal.h"
 #include <stdio.h>
+#include <stdlib.h>
 #include <unistd.h>
 #include <dirent.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <ctype.h>
+#include <string.h>
 #include <sys/ioctl.h>
+#include "busybox.h"
 
+static const int TERMINAL_WIDTH = 79;      /* not 80 in case terminal has linefold bug */
 
-#if ! defined BB_FEATURE_USE_DEVPS_N_DEVMTAB
+
+
+#if ! defined BB_FEATURE_USE_DEVPS_PATCH
 
 /* The following is the first ps implementation --
  * the one using the /proc virtual filesystem.
  */
 
-#if ! defined BB_FEATURE_USE_PROCFS
-#error Sorry, I depend on the /proc filesystem right now.
-#endif
-
 typedef struct proc_s {
        char
         cmd[16];                                       /* basename of executable file in call to exec(2) */
@@ -91,24 +92,23 @@ static void parse_proc_status(char *S, proc_t * P)
        if (tmp)
                sscanf(tmp, "Pid:\t%d\n" "PPid:\t%d\n", &P->pid, &P->ppid);
        else
-               fprintf(stderr, "Internal error!\n");
+               error_msg("Internal error!");
 
        /* For busybox, ignoring effective, saved, etc */
        tmp = strstr(S, "Uid:");
        if (tmp)
                sscanf(tmp, "Uid:\t%d", &P->ruid);
        else
-               fprintf(stderr, "Internal error!\n");
+               error_msg("Internal error!");
 
        tmp = strstr(S, "Gid:");
        if (tmp)
                sscanf(tmp, "Gid:\t%d", &P->rgid);
        else
-               fprintf(stderr, "Internal error!\n");
+               error_msg("Internal error!");
 
 }
 
-
 extern int ps_main(int argc, char **argv)
 {
        proc_t p;
@@ -116,24 +116,21 @@ extern int ps_main(int argc, char **argv)
        FILE *file;
        struct dirent *entry;
        char path[32], sbuf[512];
-       char uidName[10] = "";
-       char groupName[10] = "";
+       char uidName[9];
+       char groupName[9];
        int len, i, c;
 #ifdef BB_FEATURE_AUTOWIDTH
-       struct winsize win = { 0, 0 };
-       int terminal_width = 0;
+       struct winsize win = { 0, 0, 0, 0 };
+       int terminal_width = TERMINAL_WIDTH;
 #else
-#define terminal_width  79
+#define terminal_width  TERMINAL_WIDTH
 #endif
 
 
 
-       if (argc > 1 && **(argv + 1) == '-')
-               usage ("ps\n\nReport process status\n\nThis version of ps accepts no options.\n");
-
        dir = opendir("/proc");
        if (!dir)
-               fatalError("Can't open /proc");
+               error_msg_and_die("Can't open /proc");
 
 #ifdef BB_FEATURE_AUTOWIDTH
                ioctl(fileno(stdout), TIOCGWINSZ, &win);
@@ -141,12 +138,8 @@ extern int ps_main(int argc, char **argv)
                        terminal_width = win.ws_col - 1;
 #endif
 
-       fprintf(stdout, "%5s  %-8s %-3s %5s %s\n", "PID", "Uid", "Gid",
-                       "State", "Command");
+       printf("%5s  %-8s %-3s %5s %s\n", "PID", "Uid", "Gid", "State", "Command");
        while ((entry = readdir(dir)) != NULL) {
-               uidName[0] = '\0';
-               groupName[0] = '\0';
-
                if (!isdigit(*entry->d_name))
                        continue;
                sprintf(path, "/proc/%s/status", entry->d_name);
@@ -162,36 +155,36 @@ extern int ps_main(int argc, char **argv)
                if (*groupName == '\0')
                        sprintf(groupName, "%d", p.rgid);
 
-               len = fprintf(stdout, "%5d %-8s %-8s %c ", p.pid, uidName, groupName,
-                               p.state);
                sprintf(path, "/proc/%s/cmdline", entry->d_name);
                file = fopen(path, "r");
                if (file == NULL)
-                       fatalError("Can't open %s: %s\n", path, strerror(errno));
+                       continue;
                i = 0;
+               len = printf("%5d %-8s %-8s %c ", p.pid, uidName, groupName, p.state);
                while (((c = getc(file)) != EOF) && (i < (terminal_width-len))) {
                        i++;
                        if (c == '\0')
                                c = ' ';
                        putc(c, stdout);
                }
+               fclose(file);
                if (i == 0)
-                       fprintf(stdout, "[%s]", p.cmd);
-               fprintf(stdout, "\n");
+                       printf("[%s]", p.cmd);
+               putchar('\n');
        }
        closedir(dir);
-       exit(TRUE);
+       return EXIT_SUCCESS;
 }
 
 
-#else /* BB_FEATURE_USE_DEVPS_N_DEVMTAB */
+#else /* BB_FEATURE_USE_DEVPS_PATCH */
 
 
 /* The following is the second ps implementation --
  * this one uses the nifty new devps kernel device.
  */
 
-#include <linux/devps.h>
+#include <linux/devps.h> /* For Erik's nifty devps device driver */
 
 
 extern int ps_main(int argc, char **argv)
@@ -201,37 +194,37 @@ extern int ps_main(int argc, char **argv)
        pid_t num_pids;
        pid_t* pid_array = NULL;
        struct pid_info info;
-       char uidName[10] = "";
-       char groupName[10] = "";
+       char uidName[9];
+       char groupName[9];
 #ifdef BB_FEATURE_AUTOWIDTH
-       struct winsize win = { 0, 0 };
-       int terminal_width = 0;
+       struct winsize win = { 0, 0, 0, 0 };
+       int terminal_width = TERMINAL_WIDTH;
 #else
-#define terminal_width  79
+#define terminal_width  TERMINAL_WIDTH
 #endif
 
        if (argc > 1 && **(argv + 1) == '-') 
-               usage("ps-devps\n\nReport process status\n\nThis version of ps accepts no options.\n\n");
+               show_usage();
 
        /* open device */ 
        fd = open(device, O_RDONLY);
        if (fd < 0) 
-               fatalError( "open failed for `%s': %s\n", device, strerror (errno));
+               perror_msg_and_die( "open failed for `%s'", device);
 
        /* Find out how many processes there are */
        if (ioctl (fd, DEVPS_GET_NUM_PIDS, &num_pids)<0) 
-               fatalError( "\nDEVPS_GET_PID_LIST: %s\n", strerror (errno));
+               perror_msg_and_die( "\nDEVPS_GET_PID_LIST");
        
        /* Allocate some memory -- grab a few extras just in case 
         * some new processes start up while we wait. The kernel will
         * just ignore any extras if we give it too many, and will trunc.
         * the list if we give it too few.  */
-       pid_array = (pid_t*) calloc( num_pids+10, sizeof(pid_t));
+       pid_array = (pid_t*) xcalloc( num_pids+10, sizeof(pid_t));
        pid_array[0] = num_pids+10;
 
        /* Now grab the pid list */
        if (ioctl (fd, DEVPS_GET_PID_LIST, pid_array)<0) 
-               fatalError("\nDEVPS_GET_PID_LIST: %s\n", strerror (errno));
+               perror_msg_and_die("\nDEVPS_GET_PID_LIST");
 
 #ifdef BB_FEATURE_AUTOWIDTH
                ioctl(fileno(stdout), TIOCGWINSZ, &win);
@@ -240,16 +233,13 @@ extern int ps_main(int argc, char **argv)
 #endif
 
        /* Print up a ps listing */
-       fprintf(stdout, "%5s  %-8s %-3s %5s %s\n", "PID", "Uid", "Gid",
-                       "State", "Command");
+       printf("%5s  %-8s %-3s %5s %s\n", "PID", "Uid", "Gid", "State", "Command");
 
        for (i=1; i<pid_array[0] ; i++) {
-               uidName[0] = '\0';
-               groupName[0] = '\0';
            info.pid = pid_array[i];
 
            if (ioctl (fd, DEVPS_GET_PID_INFO, &info)<0)
-                       fatalError("\nDEVPS_GET_PID_INFO: %s\n", strerror (errno));
+                       perror_msg_and_die("\nDEVPS_GET_PID_INFO");
            
                /* Make some adjustments as needed */
                my_getpwuid(uidName, info.euid);
@@ -259,7 +249,7 @@ extern int ps_main(int argc, char **argv)
                if (*groupName == '\0')
                        sprintf(groupName, "%ld", info.egid);
 
-               len = fprintf(stdout, "%5d %-8s %-8s %c ", info.pid, uidName, groupName, info.state);
+               len = printf("%5d %-8s %-8s %c ", info.pid, uidName, groupName, info.state);
 
                if (strlen(info.command_line) > 1) {
                        for( j=0; j<(sizeof(info.command_line)-1) && j < (terminal_width-len); j++) {
@@ -268,9 +258,9 @@ extern int ps_main(int argc, char **argv)
                                }
                        }
                        *(info.command_line+j) = '\0';
-                       fprintf(stdout, "%s\n", info.command_line);
+                       puts(info.command_line);
                } else {
-                       fprintf(stdout, "[%s]\n", info.name);
+                       printf("[%s]\n", info.name);
                }
        }
 
@@ -279,10 +269,10 @@ extern int ps_main(int argc, char **argv)
 
        /* close device */
        if (close (fd) != 0) 
-               fatalError("close failed for `%s': %s\n", device, strerror (errno));
+               perror_msg_and_die("close failed for `%s'", device);
  
        exit (0);
 }
 
-#endif /* BB_FEATURE_USE_DEVPS_N_DEVMTAB */
+#endif /* BB_FEATURE_USE_DEVPS_PATCH */