inetd,ed,msh: data/bss reduction (in mss, more than 9k of it)

diff --git a/editors/ed.c b/editors/ed.c
index 42adca40938e256ffc5d38225764ce8fc474ff24..1706e427174b089d98fe6c71c91636a1169f1645 100644 (file)
--- a/editors/ed.c
+++ b/editors/ed.c
@@ -9,8 +9,12 @@
 
 #include "busybox.h"
 
-#define        USERSIZE        1024    /* max line length typed in by user */
-#define        INITBUF_SIZE    1024    /* initial buffer size */
+enum {
+       USERSIZE = sizeof(bb_common_bufsiz1) > 1024 ? 1024
+                : sizeof(bb_common_bufsiz1) - 1, /* max line length typed in by user */
+       INITBUF_SIZE = 1024, /* initial buffer size */
+};
+
 typedef struct LINE {
        struct LINE *next;
        struct LINE *prev;
@@ -18,9 +22,11 @@ typedef struct LINE {
        char data[1];
 } LINE;
 
+#define searchString bb_common_bufsiz1
+
 static LINE lines, *curLine;
 static int curNum, lastNum, marks[26], dirty;
-static char *bufBase, *bufPtr, *fileName, searchString[USERSIZE];
+static char *bufBase, *bufPtr, *fileName;
 static int bufUsed, bufSize;
 
 static void doCommands(void);

diff --git a/networking/inetd.c b/networking/inetd.c
index 48e23db2ea437211a6cb9f029bc4363b9475f9be..83123463f9db84a9dc1caffacc9840ea07325d14 100644 (file)
--- a/networking/inetd.c
+++ b/networking/inetd.c
@@ -172,7 +172,6 @@
 #include <rpc/pmap_clnt.h>
 #endif
 
-#define _PATH_INETDCONF "/etc/inetd.conf"
 #define _PATH_INETDPID  "/var/run/inetd.pid"
 
 
@@ -327,10 +326,9 @@ static int timingout;
 static struct servent *sp;
 static uid_t uid;
 
-static const char *CONFIG = _PATH_INETDCONF;
+static const char *config_filename = "/etc/inetd.conf";
 
 static FILE *fconfig;
-static char line[1024];
 static char *defhost;
 
 /* xstrdup(NULL) returns NULL, but this one
@@ -350,7 +348,7 @@ static int setconfig(void)
                fseek(fconfig, 0L, SEEK_SET);
                return 1;
        }
-       fconfig = fopen(CONFIG, "r");
+       fconfig = fopen(config_filename, "r");
        return (fconfig != NULL);
 }
 
@@ -511,6 +509,8 @@ static void setup(servtab_t *sep)
 
 static char *nextline(void)
 {
+#define line bb_common_bufsiz1
+
        char *cp;
        FILE *fd = fconfig;
 
@@ -541,10 +541,12 @@ static char *skip(char **cpp) /* int report; */
                int c;
 
                c = getc(fconfig);
-               (void) ungetc(c, fconfig);
-               if (c == ' ' || c == '\t')
-                       if ((cp = nextline()))
+               ungetc(c, fconfig);
+               if (c == ' ' || c == '\t') {
+                       cp = nextline();
+                       if (cp)
                                goto again;
+               }
                *cpp = NULL;
                /* goto erp; */
                return NULL;
@@ -924,7 +926,7 @@ static void config(int sig ATTRIBUTE_UNUSED)
        char protoname[10];
 
        if (!setconfig()) {
-               bb_perror_msg("%s", CONFIG);
+               bb_perror_msg("%s", config_filename);
                return;
        }
        for (sep = servtab; sep; sep = sep->se_next)
@@ -1281,10 +1283,10 @@ int inetd_main(int argc, char *argv[])
 
        uid = getuid();
        if (uid != 0)
-               CONFIG = NULL;
+               config_filename = NULL;
        if (argc > 0)
-               CONFIG = argv[0];
-       if (CONFIG == NULL)
+               config_filename = argv[0];
+       if (config_filename == NULL)
                bb_error_msg_and_die("non-root must specify a config file");
 
 #ifdef BB_NOMMU

diff --git a/shell/msh.c b/shell/msh.c
index d9dd3efb21bdc7eb0e1c8cdecfc81a0f604ccdd5..66b10f346d4d0ac49012bcc59711efd023b138ea 100644 (file)
--- a/shell/msh.c
+++ b/shell/msh.c
@@ -13,12 +13,13 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
+# include <sys/times.h>
+# include <setjmp.h>
+
 #ifdef STANDALONE
 # ifndef _GNU_SOURCE
 #  define _GNU_SOURCE
 # endif
-# include <setjmp.h>
-# include <sys/times.h>
 # include <sys/types.h>
 # include <sys/stat.h>
 # include <sys/wait.h>
@@ -82,8 +83,6 @@ static char *itoa(int n)
        return local_buf;
 }
 #else
-# include <setjmp.h>
-# include <sys/times.h>
 # include "busybox.h"
 extern char **environ;
 #endif
@@ -162,12 +161,10 @@ int mshdbg_rc = 0;
  */
 typedef void xint;                             /* base type of jmp_buf, for not broken compilers */
 
+
 /*
  * shell components
  */
-
-#define        QUOTE   0200
-
 #define        NOBLOCK ((struct op *)NULL)
 #define        NOWORD  ((char *)NULL)
 #define        NOWORDS ((char **)NULL)
@@ -297,24 +294,21 @@ static char *flag = flags - 'a';
 static char *null;                             /* null value for variable */
 static int intr;                               /* interrupt pending */
 
-static char *trap[_NSIG + 1];
-static char ourtrap[_NSIG + 1];
+/* moved to G: static char *trap[_NSIG + 1]; */
+/* moved to G: static char ourtrap[_NSIG + 1]; */
 static int trapset;                            /* trap pending */
 
 static int heedint;                            /* heed interrupt signals */
 
 static int yynerrs;                            /* yacc */
 
-static char line[LINELIM];
-static char *elinep;
+/* moved to G: static char line[LINELIM]; */
 
 #if ENABLE_FEATURE_EDITING
 static char *current_prompt;
 static line_input_t *line_input_state;
 #endif
 
-static int areanum;                            /* current allocation area */
-
 
 /*
  * other functions
@@ -483,29 +477,15 @@ struct io {
 #define        INSUB() (e.iop->task == XGRAVE || e.iop->task == XDOLL)
 
 static struct ioarg temparg = { 0, 0, 0, AFID_NOBUF, 0 };      /* temporary for PUSHIO */
-static struct ioarg ioargstack[NPUSH];
+/* moved to G: static struct ioarg ioargstack[NPUSH]; */
 static struct io iostack[NPUSH];
-static struct iobuf sharedbuf = { AFID_NOBUF };
-static struct iobuf mainbuf = { AFID_NOBUF };
+/* moved to G: static struct iobuf sharedbuf = { AFID_NOBUF }; */
+/* moved to G: static struct iobuf mainbuf = { AFID_NOBUF }; */
 static unsigned bufid = AFID_ID;       /* buffer id counter */
 
-#define        PUSHIO(what,arg,gen) ((temparg.what = (arg)), pushio(&temparg,(gen)))
 #define        RUN(what,arg,gen) ((temparg.what = (arg)), run(&temparg,(gen)))
 
 
-/*
- * parsing & execution environment
- */
-static struct env {
-       char *linep;
-       struct io *iobase;
-       struct io *iop;
-       xint *errpt;                            /* void * */
-       int iofd;
-       struct env *oenv;
-} e;
-
-
 /*
  * input generators for IO structure
  */
@@ -537,6 +517,7 @@ static void ioecho(char c);
  * IO control
  */
 static void pushio(struct ioarg *argp, int (*f) (struct ioarg *));
+#define PUSHIO(what,arg,gen) ((temparg.what = (arg)), pushio(&temparg,(gen)))
 static int remap(int fd);
 static int openpipe(int *pv);
 static void closepipe(int *pv);
@@ -599,7 +580,6 @@ static int xstrcmp(char *p1, char *p2);
 static void glob0(char *a0, unsigned a1, int a2,
                                  int (*a3) (char *, char *));
 static void readhere(char **name, char *s, int ec);
-static void pushio(struct ioarg *argp, int (*f) (struct ioarg *));
 static int xxchar(struct ioarg *ap);
 
 struct here {
@@ -705,9 +685,6 @@ static struct brkcon *brklist;
 static int isbreak;
 static struct wdblock *wdlist;
 static struct wdblock *iolist;
-static char *trap[_NSIG + 1];
-static char ourtrap[_NSIG + 1];
-static int trapset;                            /* trap pending */
 
 #ifdef MSHDEBUG
 static struct var *mshdbg_var;
@@ -731,8 +708,7 @@ static int peeksym;
 static int nlseen;
 static int iounit = IODEFAULT;
 static YYSTYPE yylval;
-static char *elinep = line + sizeof(line) - 5;
-
+static char *elinep; /* done in main(): = line + sizeof(line) - 5 */
 
 static struct here *inhere;     /* list of hear docs while parsing */
 static struct here *acthere;    /* list of active here documents */
@@ -742,8 +718,20 @@ static struct region *areanxt;  /* starting point of scan */
 static void *brktop;
 static void *brkaddr;
 
+/*
+ * parsing & execution environment
+ */
+struct env {
+       char *linep;
+       struct io *iobase;
+       struct io *iop;
+       xint *errpt;            /* void * */
+       int iofd;
+       struct env *oenv;
+};
+
 static struct env e = {
-       line,                   /* linep:  char ptr */
+       NULL /* set to line in main() */, /* linep:  char ptr */
        iostack,                /* iobase:  struct io ptr */
        iostack - 1,            /* iop:  struct io ptr */
        (xint *) NULL,          /* errpt:  void ptr for errors? */
@@ -751,6 +739,29 @@ static struct env e = {
        (struct env *) NULL     /* oenv:  struct env ptr */
 };
 
+
+struct globals {
+       char ourtrap[_NSIG + 1];
+       char *trap[_NSIG + 1];
+       struct iobuf sharedbuf; /* in main(): set to { AFID_NOBUF } */
+       struct iobuf mainbuf; /* in main(): set to { AFID_NOBUF } */
+       struct ioarg ioargstack[NPUSH];
+       char filechar_cmdbuf[BUFSIZ];
+       char line[LINELIM];
+       char child_cmd[LINELIM];
+};
+
+#define G (*ptr_to_globals)
+#define ourtrap         (G.ourtrap        )
+#define trap            (G.trap           )
+#define sharedbuf       (G.sharedbuf      )
+#define mainbuf         (G.mainbuf        )
+#define ioargstack      (G.ioargstack     )
+#define filechar_cmdbuf (G.filechar_cmdbuf)
+#define line            (G.line           )
+#define child_cmd       (G.child_cmd      )
+
+
 #ifdef MSHDEBUG
 void print_t(struct op *t)
 {
@@ -1517,7 +1528,7 @@ static void onintr(int s)                                 /* ANSI C requires a parameter */
 
 #define        CMASK   0377
 #define        QUOTE   0200
-#define        QMASK   (CMASK&~QUOTE)
+#define        QMASK   (CMASK & ~QUOTE)
 #define        NOT     '!'                                     /* might use ^ */
 
 static const char *cclass(const char *p, int sub)
@@ -4011,11 +4022,12 @@ static int dollar(int quoted)
 
 static int grave(int quoted)
 {
+       /* moved to G: static char child_cmd[LINELIM]; */
+
        const char *cp;
        int i;
        int j;
        int pf[2];
-       static char child_cmd[LINELIM];
        const char *src;
        char *dest;
        int count;
@@ -4823,15 +4835,15 @@ static int filechar(struct ioarg *ap)
        }
 #if ENABLE_FEATURE_EDITING
        if (interactive && isatty(ap->afile)) {
-               static char mycommand[BUFSIZ];
+               /* moved to G: static char filechar_cmdbuf[BUFSIZ]; */
                static int position = 0, size = 0;
 
                while (size == 0 || position >= size) {
-                       read_line_input(current_prompt, mycommand, BUFSIZ, line_input_state);
-                       size = strlen(mycommand);
+                       read_line_input(current_prompt, filechar_cmdbuf, BUFSIZ, line_input_state);
+                       size = strlen(filechar_cmdbuf);
                        position = 0;
                }
-               c = mycommand[position];
+               c = filechar_cmdbuf[position];
                position++;
                return c;
        }
@@ -5176,6 +5188,12 @@ int msh_main(int argc, char **argv)
        char *name, **ap;
        int (*iof) (struct ioarg *);
 
+       PTR_TO_GLOBALS = xzalloc(sizeof(G));
+       sharedbuf.id = AFID_NOBUF;
+       mainbuf.id = AFID_NOBUF;
+       e.linep = line;
+       elinep = line + sizeof(line) - 5;
+
 #if ENABLE_FEATURE_EDITING
        line_input_state = new_line_input_t(FOR_SHELL);
 #endif