Don't cast returns from malloc()
[openafs.git] / src / config / mc.c
index 4df6283..97fc26e 100644 (file)
@@ -1,24 +1,25 @@
 /*
  * Copyright 2000, International Business Machines Corporation and others.
  * All Rights Reserved.
- * 
+ *
  * This software has been released under the terms of the IBM Public
  * License.  For details, see the LICENSE file in the top-level source
  * directory or online at http://www.openafs.org/dl/license10.html
  */
 
+#include "afsconfig.h"
+
 #define        MAXLINELEN      1024
 #define        MAXTOKLEN       100
 #include <sys/param.h>
 #include <sys/types.h>
 #include <sys/file.h>
 #include <stdio.h>
+#include <stdlib.h>
 
-#if  defined(__alpha)
-extern void *malloc(int size);
-#endif
+#include <string.h>
 
-#define TOK_DONTUSE 1 /* Don't copy if match and this flag is set. */
+#define TOK_DONTUSE 1          /* Don't copy if match and this flag is set. */
 struct token {
     struct token *next;
     char *key;
@@ -26,10 +27,11 @@ struct token {
 };
 
 /* free token list returned by parseLine */
-static FreeTokens(alist)
-    register struct token *alist; {
-    register struct token *nlist;
-    for(; alist; alist = nlist) {
+static int
+FreeTokens(struct token *alist)
+{
+    struct token *nlist;
+    for (; alist; alist = nlist) {
        nlist = alist->next;
        free(alist->key);
        free(alist);
@@ -38,48 +40,48 @@ static FreeTokens(alist)
 }
 
 #define        space(x)    ((x) == ' ' || (x) == '\t' || (x) == '<' || (x) == '>')
-static ParseLine(aline, alist)
-    char *aline;
-    struct token **alist; {
-    char tbuffer[MAXTOKLEN+1];
-    register char *tptr;
+static int
+ParseLine(char *aline, struct token **alist)
+{
+    char tbuffer[MAXTOKLEN + 1];
+    char *tptr = NULL;
     int inToken;
     struct token *first, *last;
-    register struct token *ttok;
-    register int tc;
+    struct token *ttok;
+    int tc;
     int dontUse = 0;
-    
-    inToken = 0;       /* not copying token chars at start */
-    first = (struct token *) 0;
-    last = (struct token *) 0;
+
+    inToken = 0;               /* not copying token chars at start */
+    first = NULL;
+    last = NULL;
     while (1) {
        tc = *aline++;
-       if (tc == 0 || space(tc)) {    /* terminating null gets us in here, too */
+       if (tc == 0 || space(tc)) {     /* terminating null gets us in here, too */
            if (inToken) {
-               inToken = 0;    /* end of this token */
-               *tptr++ = 0;
-               ttok = (struct token *) malloc(sizeof(struct token));
-               ttok->next = (struct token *) 0;
+               inToken = 0;    /* end of this token */
+               if (!tptr)
+                   return -1;  /* should never get here */
+               else
+                   *tptr++ = 0;
+               ttok = malloc(sizeof(struct token));
+               ttok->next = NULL;
                if (dontUse) {
-                   ttok->key = (char *) malloc(strlen(tbuffer));
-                   strcpy(ttok->key, tbuffer+1);
+                   ttok->key = strdup(tbuffer + 1); /* Skip first char */
                    ttok->flags = TOK_DONTUSE;
                    dontUse = 0;
-               }
-               else {
-                   ttok->key = (char *) malloc(strlen(tbuffer)+1);
-                   strcpy(ttok->key, tbuffer);
+               } else {
+                   ttok->key = strdup(tbuffer);
                    ttok->flags = 0;
                }
                if (last) {
                    last->next = ttok;
                    last = ttok;
-               }
-               else last = ttok;
-               if (!first) first = ttok;
+               } else
+                   last = ttok;
+               if (!first)
+                   first = ttok;
            }
-       }
-       else {
+       } else {
            /* an alpha character */
            if (!inToken) {
                if (tc == '-') {
@@ -88,34 +90,38 @@ static ParseLine(aline, alist)
                tptr = tbuffer;
                inToken = 1;
            }
-           if (tptr - tbuffer >= MAXTOKLEN) return -1;   /* token too long */
+           if (tptr - tbuffer >= MAXTOKLEN)
+               return -1;      /* token too long */
            *tptr++ = tc;
        }
        if (tc == 0) {
            /* last token flushed 'cause space(0) --> true */
-           if (last) last->next = (struct token *) 0;
+           if (last)
+               last->next = NULL;
            *alist = first;
            return 0;
        }
     }
 }
+
 /* read a line into a buffer, putting in null termination and stopping on appropriate
     end of line char.  Returns 0 at eof, > 0 at normal line end, and < 0 on error */
-static GetLine(afile, abuffer, amax)
-    FILE *afile;
-    int amax;
-    register char *abuffer; {
-    register int tc;
+static int
+GetLine(FILE * afile, char *abuffer, int amax)
+{
+    int tc;
     int first;
 
     first = 1;
     while (1) {
        tc = getc(afile);
-       if (first && tc < 0) return 0;
+       if (first && tc < 0)
+           return 0;
        first = 0;
        if (tc <= 0 || tc == '\012') {
-           if (amax > 0) *abuffer++ = 0;
-           return (amax > 0? 1 : -1);
+           if (amax > 0)
+               *abuffer++ = 0;
+           return (amax > 0 ? 1 : -1);
        }
        if (amax > 0) {
            /* keep reading to end of line so next one isn't bogus */
@@ -124,33 +130,35 @@ static GetLine(afile, abuffer, amax)
        }
     }
 }
-mc_copy(ain, aout, alist)
-    register FILE *ain;
-    register FILE *aout;
-    char *alist[]; {
+
+int
+mc_copy(FILE * ain, FILE * aout, char *alist[])
+{
     char tbuffer[MAXLINELEN];
     struct token *tokens;
-    register char **tp;
-    register struct token *tt;
-    register int code;
+    char **tp;
+    struct token *tt;
+    int code;
     int copying;
     int done;
 
-    copying = 1;       /* start off copying data */
+    copying = 1;               /* start off copying data */
     while (1) {
        /* copy lines, handling modes appropriately */
        code = GetLine(ain, tbuffer, MAXLINELEN);
-       if (code <= 0) break;
+       if (code <= 0)
+           break;
        /* otherwise process the line */
        if (tbuffer[0] == '<') {
            /* interpret the line as a set of options, any one of which will cause us
-               to start copying the data again. */
+            * to start copying the data again. */
            code = ParseLine(tbuffer, &tokens);
-           if (code != 0) return -1;
+           if (code != 0)
+               return -1;
            copying = 0;
            done = 0;
-           for(tp = alist; (!done) && (*tp != (char *)0) ; tp++) {
-               for(tt = tokens; tt; tt=tt->next) {
+           for (tp = alist; (!done) && (*tp != NULL); tp++) {
+               for (tt = tokens; tt; tt = tt->next) {
                    if (!strcmp(*tp, tt->key)) {
                        /* Need to search all tokens in case a dont use
                         * flag is set. But we can stop on the first
@@ -160,16 +168,14 @@ mc_copy(ain, aout, alist)
                            copying = 0;
                            done = 1;
                            break;
-                       }
-                       else {
+                       } else {
                            copying = 1;
                        }
                    }
                }
            }
            FreeTokens(tokens);
-       }
-       else {
+       } else {
            /* just copy the line */
            if (copying) {
                fwrite(tbuffer, 1, strlen(tbuffer), aout);