Hardcoded mhmail instead of mailproc.
[mmh] / uip / mhparse.c
index 62fc765..ece4e8f 100644 (file)
@@ -13,7 +13,6 @@
 #include <errno.h>
 #include <setjmp.h>
 #include <signal.h>
-#include <h/mts.h>
 #include <h/tws.h>
 #include <h/mime.h>
 #include <h/mhparse.h>
@@ -89,49 +88,46 @@ struct k2v SubApplication[] = {
 };
 
 
-/* ftpsbr.c */
-int ftp_get (char *, char *, char *, char *, char *, char *, int, int);
-
 /* mhcachesbr.c */
-int find_cache (CT, int, int *, char *, char *, int);
+int find_cache(CT, int, int *, char *, char *, int);
 
 /* mhmisc.c */
-int part_ok (CT, int);
-int type_ok (CT, int);
-int make_intermediates (char *);
-void content_error (char *, CT, char *, ...);
+int part_ok(CT, int);
+int type_ok(CT, int);
+int make_intermediates(char *);
+void content_error(char *, CT, char *, ...);
 
 /* mhfree.c */
-void free_content (CT);
-void free_encoding (CT, int);
+void free_content(CT);
+void free_encoding(CT, int);
 
 /*
 ** static prototypes
 */
-static CT get_content (FILE *, char *, int);
-static int get_comment (CT, unsigned char **, int);
-
-static int InitGeneric (CT);
-static int InitText (CT);
-static int InitMultiPart (CT);
-static void reverse_parts (CT);
-static int InitMessage (CT);
-static int InitApplication (CT);
-static int init_encoding (CT, OpenCEFunc);
-static unsigned long size_encoding (CT);
-static int InitBase64 (CT);
-static int openBase64 (CT, char **);
-static int InitQuoted (CT);
-static int openQuoted (CT, char **);
-static int Init7Bit (CT);
-static int openExternal (CT, CT, CE, char **, int *);
-static int InitFile (CT);
-static int openFile (CT, char **);
-static int InitFTP (CT);
-static int openFTP (CT, char **);
-static int InitMail (CT);
-static int openMail (CT, char **);
-static int readDigest (CT, char *);
+static CT get_content(FILE *, char *, int);
+static int get_comment(CT, unsigned char **, int);
+
+static int InitGeneric(CT);
+static int InitText(CT);
+static int InitMultiPart(CT);
+static void reverse_parts(CT);
+static int InitMessage(CT);
+static int InitApplication(CT);
+static int init_encoding(CT, OpenCEFunc);
+static unsigned long size_encoding(CT);
+static int InitBase64(CT);
+static int openBase64(CT, char **);
+static int InitQuoted(CT);
+static int openQuoted(CT, char **);
+static int Init7Bit(CT);
+static int openExternal(CT, CT, CE, char **, int *);
+static int InitFile(CT);
+static int openFile(CT, char **);
+static int InitFTP(CT);
+static int openFTP(CT, char **);
+static int InitMail(CT);
+static int openMail(CT, char **);
+static int readDigest(CT, char *);
 
 struct str2init str2cts[] = {
        { "application", CT_APPLICATION, InitApplication },
@@ -171,14 +167,14 @@ struct str2init str2methods[] = {
 
 
 int
-pidcheck (int status)
+pidcheck(int status)
 {
        if ((status & 0xff00) == 0xff00 || (status & 0x007f) != SIGQUIT)
                return status;
 
-       fflush (stdout);
-       fflush (stderr);
-       done (1);
+       fflush(stdout);
+       fflush(stderr);
+       done(1);
        return 1;
 }
 
@@ -189,7 +185,7 @@ pidcheck (int status)
 ** entity in the file.
 */
 CT
-parse_mime (char *file)
+parse_mime(char *file)
 {
        int is_stdin;
        char buffer[BUFSIZ];
@@ -199,39 +195,39 @@ parse_mime (char *file)
        /*
        ** Check if file is actually standard input
        */
-       if ((is_stdin = !(strcmp (file, "-")))) {
+       if ((is_stdin = (strcmp(file, "-")==0))) {
                char *tfile = m_mktemp2(NULL, invo_name, NULL, &fp);
                if (tfile == NULL) {
                        advise("mhparse", "unable to create temporary file");
                        return NULL;
                }
-               file = add (tfile, NULL);
-               chmod (file, 0600);
+               file = getcpy(tfile);
+               chmod(file, 0600);
 
-               while (fgets (buffer, sizeof(buffer), stdin))
-                       fputs (buffer, fp);
-               fflush (fp);
+               while (fgets(buffer, sizeof(buffer), stdin))
+                       fputs(buffer, fp);
+               fflush(fp);
 
-               if (ferror (stdin)) {
-                       unlink (file);
-                       advise ("stdin", "error reading");
+               if (ferror(stdin)) {
+                       unlink(file);
+                       advise("stdin", "error reading");
                        return NULL;
                }
-               if (ferror (fp)) {
-                       unlink (file);
-                       advise (file, "error writing");
+               if (ferror(fp)) {
+                       unlink(file);
+                       advise(file, "error writing");
                        return NULL;
                }
-               fseek (fp, 0L, SEEK_SET);
-       } else if ((fp = fopen (file, "r")) == NULL) {
-               advise (file, "unable to read");
+               fseek(fp, 0L, SEEK_SET);
+       } else if ((fp = fopen(file, "r")) == NULL) {
+               advise(file, "unable to read");
                return NULL;
        }
 
-       if (!(ct = get_content (fp, file, 1))) {
+       if (!(ct = get_content(fp, file, 1))) {
                if (is_stdin)
-                       unlink (file);
-               advise (NULL, "unable to decode %s", file);
+                       unlink(file);
+               advise(NULL, "unable to decode %s", file);
                return NULL;
        }
 
@@ -241,17 +237,17 @@ parse_mime (char *file)
        ct->c_fp = NULL;
 
        if (ct->c_end == 0L) {
-               fseek (fp, 0L, SEEK_END);
-               ct->c_end = ftell (fp);
+               fseek(fp, 0L, SEEK_END);
+               ct->c_end = ftell(fp);
        }
 
        if (ct->c_ctinitfnx && (*ct->c_ctinitfnx) (ct) == NOTOK) {
-               fclose (fp);
-               free_content (ct);
+               fclose(fp);
+               free_content(ct);
                return NULL;
        }
 
-       fclose (fp);
+       fclose(fp);
        return ct;
 }
 
@@ -268,7 +264,7 @@ parse_mime (char *file)
 */
 
 static CT
-get_content (FILE *in, char *file, int toplevel)
+get_content(FILE *in, char *file, int toplevel)
 {
        int compnum, state;
        char buf[BUFSIZ], name[NAMESZ];
@@ -277,59 +273,61 @@ get_content (FILE *in, char *file, int toplevel)
        HF hp;
 
        /* allocate the content structure */
-       if (!(ct = (CT) calloc (1, sizeof(*ct))))
-               adios (NULL, "out of memory");
+       if (!(ct = (CT) calloc(1, sizeof(*ct))))
+               adios(NULL, "out of memory");
 
        ct->c_fp = in;
-       ct->c_file = add (file, NULL);
-       ct->c_begin = ftell (ct->c_fp) + 1;
+       ct->c_file = getcpy(file);
+       ct->c_begin = ftell(ct->c_fp) + 1;
 
        /*
        ** Parse the header fields for this
        ** content into a linked list.
        */
        for (compnum = 1, state = FLD;;) {
-               switch (state = m_getfld (state, name, buf, sizeof(buf), in)) {
+               switch (state = m_getfld(state, name, buf, sizeof(buf), in)) {
                case FLD:
                case FLDPLUS:
                case FLDEOF:
                        compnum++;
 
                        /* get copies of the buffers */
-                       np = add (name, NULL);
-                       vp = add (buf, NULL);
+                       np = getcpy(name);
+                       vp = getcpy(buf);
 
                        /* if necessary, get rest of field */
                        while (state == FLDPLUS) {
-                               state = m_getfld (state, name, buf, sizeof(buf), in);
-                               vp = add (buf, vp);  /* add to previous value */
+                               state = m_getfld(state, name, buf,
+                                               sizeof(buf), in);
+                               vp = add(buf, vp);  /* add to previous value */
                        }
 
                        /* Now add the header data to the list */
-                       add_header (ct, np, vp);
+                       add_header(ct, np, vp);
 
                        /* continue, if this isn't the last header field */
                        if (state != FLDEOF) {
-                               ct->c_begin = ftell (in) + 1;
+                               ct->c_begin = ftell(in) + 1;
                                continue;
                        }
                        /* else fall... */
 
                case BODY:
                case BODYEOF:
-                       ct->c_begin = ftell (in) - strlen (buf);
+                       ct->c_begin = ftell(in) - strlen(buf);
                        break;
 
                case FILEEOF:
-                       ct->c_begin = ftell (in);
+                       ct->c_begin = ftell(in);
                        break;
 
                case LENERR:
                case FMTERR:
-                       adios (NULL, "message format error in component #%d", compnum);
+                       adios(NULL, "message format error in component #%d",
+                                       compnum);
 
                default:
-                       adios (NULL, "getfld() returned %d", state);
+                       adios(NULL, "getfld() returned %d", state);
                }
 
                /* break out of the loop */
@@ -346,60 +344,57 @@ get_content (FILE *in, char *file, int toplevel)
        hp = ct->c_first_hf;  /* start at first header field */
        while (hp) {
                /* Get MIME-Version field */
-               if (!mh_strcasecmp (hp->name, VRSN_FIELD)) {
+               if (!mh_strcasecmp(hp->name, VRSN_FIELD)) {
                        int ucmp;
                        char c;
                        unsigned char *cp, *dp;
 
                        if (ct->c_vrsn) {
-                               advise (NULL, "message %s has multiple %s: fields",
-                                       ct->c_file, VRSN_FIELD);
+                               advise(NULL, "message %s has multiple %s: fields", ct->c_file, VRSN_FIELD);
                                goto next_header;
                        }
-                       ct->c_vrsn = add (hp->value, NULL);
+                       ct->c_vrsn = getcpy(hp->value);
 
                        /* Now, cleanup this field */
                        cp = ct->c_vrsn;
 
-                       while (isspace (*cp))
+                       while (isspace(*cp))
                                cp++;
                        for (dp = strchr(cp, '\n'); dp; dp = strchr(dp, '\n'))
                                *dp++ = ' ';
-                       for (dp = cp + strlen (cp) - 1; dp >= cp; dp--)
-                               if (!isspace (*dp))
+                       for (dp = cp + strlen(cp) - 1; dp >= cp; dp--)
+                               if (!isspace(*dp))
                                        break;
                        *++dp = '\0';
                        if (debugsw)
-                               fprintf (stderr, "%s: %s\n", VRSN_FIELD, cp);
+                               fprintf(stderr, "%s: %s\n", VRSN_FIELD, cp);
 
-                       if (*cp == '(' && get_comment (ct, &cp, 0) == NOTOK)
+                       if (*cp == '(' && get_comment(ct, &cp, 0) == NOTOK)
                                goto out;
 
-                       for (dp = cp; istoken (*dp); dp++)
+                       for (dp = cp; istoken(*dp); dp++)
                                continue;
                        c = *dp;
                        *dp = '\0';
-                       ucmp = !mh_strcasecmp (cp, VRSN_VALUE);
+                       ucmp = !mh_strcasecmp(cp, VRSN_VALUE);
                        *dp = c;
                        if (!ucmp) {
-                               admonish (NULL, "message %s has unknown value for %s: field (%s)",
-                               ct->c_file, VRSN_FIELD, cp);
+                               admonish(NULL, "message %s has unknown value for %s: field (%s)", ct->c_file, VRSN_FIELD, cp);
                        }
 
-               } else if (!mh_strcasecmp (hp->name, TYPE_FIELD)) {
+               } else if (!mh_strcasecmp(hp->name, TYPE_FIELD)) {
                        /* Get Content-Type field */
                        struct str2init *s2i;
                        CI ci = &ct->c_ctinfo;
 
                        /* Check if we've already seen a Content-Type header */
                        if (ct->c_ctline) {
-                               advise (NULL, "message %s has multiple %s: fields",
-                                       ct->c_file, TYPE_FIELD);
+                               advise(NULL, "message %s has multiple %s: fields", ct->c_file, TYPE_FIELD);
                                goto next_header;
                        }
 
                        /* Parse the Content-Type field */
-                       if (get_ctinfo (hp->value, ct, 0) == NOTOK)
+                       if (get_ctinfo(hp->value, ct, 0) == NOTOK)
                                goto out;
 
                        /*
@@ -407,14 +402,14 @@ get_content (FILE *in, char *file, int toplevel)
                        ** flag for this content type.
                        */
                        for (s2i = str2cts; s2i->si_key; s2i++)
-                               if (!mh_strcasecmp (ci->ci_type, s2i->si_key))
+                               if (!mh_strcasecmp(ci->ci_type, s2i->si_key))
                                        break;
-                       if (!s2i->si_key && !uprf (ci->ci_type, "X-"))
+                       if (!s2i->si_key && !uprf(ci->ci_type, "X-"))
                                s2i++;
                        ct->c_type = s2i->si_val;
                        ct->c_ctinitfnx = s2i->si_init;
 
-               } else if (!mh_strcasecmp (hp->name, ENCODING_FIELD)) {
+               } else if (!mh_strcasecmp(hp->name, ENCODING_FIELD)) {
                        /* Get Content-Transfer-Encoding field */
                        char c;
                        unsigned char *cp, *dp;
@@ -425,17 +420,16 @@ get_content (FILE *in, char *file, int toplevel)
                        ** Content-Transfer-Encoding field
                        */
                        if (ct->c_celine) {
-                               advise (NULL, "message %s has multiple %s: fields",
-                                       ct->c_file, ENCODING_FIELD);
+                               advise(NULL, "message %s has multiple %s: fields", ct->c_file, ENCODING_FIELD);
                                goto next_header;
                        }
 
                        /* get copy of this field */
-                       ct->c_celine = cp = add (hp->value, NULL);
+                       ct->c_celine = cp = getcpy(hp->value);
 
-                       while (isspace (*cp))
+                       while (isspace(*cp))
                                cp++;
-                       for (dp = cp; istoken (*dp); dp++)
+                       for (dp = cp; istoken(*dp); dp++)
                                continue;
                        c = *dp;
                        *dp = '\0';
@@ -445,9 +439,9 @@ get_content (FILE *in, char *file, int toplevel)
                        ** for this transfer encoding.
                        */
                        for (s2i = str2ces; s2i->si_key; s2i++)
-                               if (!mh_strcasecmp (cp, s2i->si_key))
+                               if (!mh_strcasecmp(cp, s2i->si_key))
                                        break;
-                       if (!s2i->si_key && !uprf (cp, "X-"))
+                       if (!s2i->si_key && !uprf(cp, "X-"))
                                s2i++;
                        *dp = c;
                        ct->c_encoding = s2i->si_val;
@@ -456,7 +450,7 @@ get_content (FILE *in, char *file, int toplevel)
                        if (s2i->si_init && (*s2i->si_init) (ct) == NOTOK)
                                goto out;
 
-               } else if (!mh_strcasecmp (hp->name, MD5_FIELD)) {
+               } else if (!mh_strcasecmp(hp->name, MD5_FIELD)) {
                        /* Get Content-MD5 field */
                        unsigned char *cp, *dp;
                        char *ep;
@@ -465,48 +459,47 @@ get_content (FILE *in, char *file, int toplevel)
                                goto next_header;
 
                        if (ct->c_digested) {
-                               advise (NULL, "message %s has multiple %s: fields",
-                                       ct->c_file, MD5_FIELD);
+                               advise(NULL, "message %s has multiple %s: fields", ct->c_file, MD5_FIELD);
                                goto next_header;
                        }
 
-                       ep = cp = add (hp->value, NULL);  /* get a copy */
+                       ep = cp = getcpy(hp->value);
 
-                       while (isspace (*cp))
+                       while (isspace(*cp))
                                cp++;
                        for (dp = strchr(cp, '\n'); dp; dp = strchr(dp, '\n'))
                                *dp++ = ' ';
-                       for (dp = cp + strlen (cp) - 1; dp >= cp; dp--)
-                               if (!isspace (*dp))
+                       for (dp = cp + strlen(cp) - 1; dp >= cp; dp--)
+                               if (!isspace(*dp))
                                        break;
                        *++dp = '\0';
                        if (debugsw)
-                               fprintf (stderr, "%s: %s\n", MD5_FIELD, cp);
+                               fprintf(stderr, "%s: %s\n", MD5_FIELD, cp);
 
-                       if (*cp == '(' && get_comment (ct, &cp, 0) == NOTOK) {
-                               free (ep);
+                       if (*cp == '(' && get_comment(ct, &cp, 0) == NOTOK) {
+                               free(ep);
                                goto out;
                        }
 
-                       for (dp = cp; *dp && !isspace (*dp); dp++)
+                       for (dp = cp; *dp && !isspace(*dp); dp++)
                                continue;
                        *dp = '\0';
 
-                       readDigest (ct, cp);
-                       free (ep);
+                       readDigest(ct, cp);
+                       free(ep);
                        ct->c_digested++;
 
-               } else if (!mh_strcasecmp (hp->name, ID_FIELD)) {
+               } else if (!mh_strcasecmp(hp->name, ID_FIELD)) {
                        /* Get Content-ID field */
-                       ct->c_id = add (hp->value, ct->c_id);
+                       ct->c_id = add(hp->value, ct->c_id);
 
-               } else if (!mh_strcasecmp (hp->name, DESCR_FIELD)) {
+               } else if (!mh_strcasecmp(hp->name, DESCR_FIELD)) {
                        /* Get Content-Description field */
-                       ct->c_descr = add (hp->value, ct->c_descr);
+                       ct->c_descr = add(hp->value, ct->c_descr);
 
-               } else if (!mh_strcasecmp (hp->name, DISPO_FIELD)) {
+               } else if (!mh_strcasecmp(hp->name, DISPO_FIELD)) {
                        /* Get Content-Disposition field */
-                       ct->c_dispo = add (hp->value, ct->c_dispo);
+                       ct->c_dispo = add(hp->value, ct->c_dispo);
                }
 
 next_header:
@@ -524,7 +517,7 @@ next_header:
                ** so default type is message/rfc822
                */
                if (toplevel < 0) {
-                       if (get_ctinfo ("message/rfc822", ct, 0) == NOTOK)
+                       if (get_ctinfo("message/rfc822", ct, 0) == NOTOK)
                                goto out;
                        ct->c_type = CT_MESSAGE;
                        ct->c_ctinitfnx = InitMessage;
@@ -532,7 +525,7 @@ next_header:
                        /*
                        ** Else default type is text/plain
                        */
-                       if (get_ctinfo ("text/plain", ct, 0) == NOTOK)
+                       if (get_ctinfo("text/plain", ct, 0) == NOTOK)
                                goto out;
                        ct->c_type = CT_TEXT;
                        ct->c_ctinitfnx = InitText;
@@ -542,13 +535,13 @@ next_header:
        /* Use default Transfer-Encoding, if necessary */
        if (!ct->c_celine) {
                ct->c_encoding = CE_7BIT;
-               Init7Bit (ct);
+               Init7Bit(ct);
        }
 
        return ct;
 
 out:
-       free_content (ct);
+       free_content(ct);
        return NULL;
 }
 
@@ -558,12 +551,12 @@ out:
 */
 
 int
-add_header (CT ct, char *name, char *value)
+add_header(CT ct, char *name, char *value)
 {
        HF hp;
 
        /* allocate header field structure */
-       hp = mh_xmalloc (sizeof(*hp));
+       hp = mh_xmalloc(sizeof(*hp));
 
        /* link data into header structure */
        hp->name = name;
@@ -593,50 +586,49 @@ add_header (CT ct, char *name, char *value)
 ** that.
 */
 static char *
-incl_name_value (unsigned char *buf, char *name, char *value) {
+incl_name_value(unsigned char *buf, char *name, char *value) {
        char *newbuf = buf;
 
        /* Assume that name is non-null. */
        if (buf && value) {
-               char *name_plus_equal = concat (name, "=", NULL);
+               char *name_plus_equal = concat(name, "=", NULL);
 
-               if (! strstr (buf, name_plus_equal)) {
+               if (!strstr(buf, name_plus_equal)) {
                        char *insertion;
                        unsigned char *cp;
                        char *prefix, *suffix;
 
                        /* Trim trailing space, esp. newline. */
-                       for (cp = &buf[strlen (buf) - 1];
-                                cp >= buf && isspace (*cp);
-                                --cp) {
+                       for (cp = &buf[strlen(buf) - 1];
+                                        cp >= buf && isspace(*cp); --cp) {
                                *cp = '\0';
                        }
 
-                       insertion = concat ("; ", name, "=", "\"", value, "\"",
+                       insertion = concat("; ", name, "=", "\"", value, "\"",
                                        NULL);
 
                        /*
                        ** Insert at first semicolon, if any.
                        ** If none, append to end.
                        */
-                       prefix = add (buf, NULL);
-                       if ((cp = strchr (prefix, ';'))) {
-                               suffix = concat (cp, NULL);
+                       prefix = getcpy(buf);
+                       if ((cp = strchr(prefix, ';'))) {
+                               suffix = concat(cp, NULL);
                                *cp = '\0';
-                               newbuf = concat (prefix, insertion, suffix,
+                               newbuf = concat(prefix, insertion, suffix,
                                                "\n", NULL);
-                               free (suffix);
+                               free(suffix);
                        } else {
                                /* Append to end. */
-                               newbuf = concat (buf, insertion, "\n", NULL);
+                               newbuf = concat(buf, insertion, "\n", NULL);
                        }
 
-                       free (prefix);
-                       free (insertion);
-                       free (buf);
+                       free(prefix);
+                       free(insertion);
+                       free(buf);
                }
 
-               free (name_plus_equal);
+               free(name_plus_equal);
        }
 
        return newbuf;
@@ -648,25 +640,27 @@ incl_name_value (unsigned char *buf, char *name, char *value) {
 ** of name will match filename="foo", and return foo.
 */
 static char *
-extract_name_value (char *name_suffix, char *value) {
+extract_name_value(char *name_suffix, char *value) {
        char *extracted_name_value = value;
-       char *name_suffix_plus_quote = concat (name_suffix, "=\"", NULL);
-       char *name_suffix_equals = strstr (value, name_suffix_plus_quote);
+       char *name_suffix_plus_quote = concat(name_suffix, "=\"", NULL);
+       char *name_suffix_equals = strstr(value, name_suffix_plus_quote);
        char *cp;
 
-       free (name_suffix_plus_quote);
+       free(name_suffix_plus_quote);
        if (name_suffix_equals) {
                char *name_suffix_begin;
 
                /* Find first \". */
-               for (cp = name_suffix_equals; *cp != '"'; ++cp) /* empty */;
+               for (cp = name_suffix_equals; *cp != '"'; ++cp)
+                       ;
                name_suffix_begin = ++cp;
                /* Find second \". */
-               for (; *cp != '"'; ++cp) /* empty */;
+               for (; *cp != '"'; ++cp)
+                       ;
 
-               extracted_name_value = mh_xmalloc (cp - name_suffix_begin + 1);
-               memcpy (extracted_name_value, name_suffix_begin,
-                       cp - name_suffix_begin);
+               extracted_name_value = mh_xmalloc(cp - name_suffix_begin + 1);
+               memcpy(extracted_name_value, name_suffix_begin,
+                               cp - name_suffix_begin);
                extracted_name_value[cp - name_suffix_begin] = '\0';
        }
 
@@ -678,7 +672,7 @@ extract_name_value (char *name_suffix, char *value) {
 ** directives.  Fills in the information of the CTinfo structure.
 */
 int
-get_ctinfo (unsigned char *cp, CT ct, int magic)
+get_ctinfo(unsigned char *cp, CT ct, int magic)
 {
        int i;
        unsigned char *dp;
@@ -687,12 +681,12 @@ get_ctinfo (unsigned char *cp, CT ct, int magic)
        CI ci;
 
        ci = &ct->c_ctinfo;
-       i = strlen (invo_name) + 2;
+       i = strlen(invo_name) + 2;
 
        /* store copy of Content-Type line */
-       cp = ct->c_ctline = add (cp, NULL);
+       cp = ct->c_ctline = getcpy(cp);
 
-       while (isspace (*cp))  /* trim leading spaces */
+       while (isspace(*cp))  /* trim leading spaces */
                cp++;
 
        /* change newlines to spaces */
@@ -700,76 +694,74 @@ get_ctinfo (unsigned char *cp, CT ct, int magic)
                *dp++ = ' ';
 
        /* trim trailing spaces */
-       for (dp = cp + strlen (cp) - 1; dp >= cp; dp--)
-               if (!isspace (*dp))
+       for (dp = cp + strlen(cp) - 1; dp >= cp; dp--)
+               if (!isspace(*dp))
                        break;
        *++dp = '\0';
 
        if (debugsw)
-               fprintf (stderr, "%s: %s\n", TYPE_FIELD, cp);
+               fprintf(stderr, "%s: %s\n", TYPE_FIELD, cp);
 
-       if (*cp == '(' && get_comment (ct, &cp, 1) == NOTOK)
+       if (*cp == '(' && get_comment(ct, &cp, 1) == NOTOK)
                return NOTOK;
 
-       for (dp = cp; istoken (*dp); dp++)
+       for (dp = cp; istoken(*dp); dp++)
                continue;
        c = *dp, *dp = '\0';
-       ci->ci_type = add (cp, NULL);  /* store content type */
+       ci->ci_type = getcpy(cp);  /* store content type */
        *dp = c, cp = dp;
 
        if (!*ci->ci_type) {
-               advise (NULL, "invalid %s: field in message %s (empty type)",
+               advise(NULL, "invalid %s: field in message %s (empty type)",
                                TYPE_FIELD, ct->c_file);
                return NOTOK;
        }
 
        /* down case the content type string */
        for (dp = ci->ci_type; *dp; dp++)
-               if (isalpha(*dp) && isupper (*dp))
-                       *dp = tolower (*dp);
+               if (isalpha(*dp) && isupper(*dp))
+                       *dp = tolower(*dp);
 
-       while (isspace (*cp))
+       while (isspace(*cp))
                cp++;
 
-       if (*cp == '(' && get_comment (ct, &cp, 1) == NOTOK)
+       if (*cp == '(' && get_comment(ct, &cp, 1) == NOTOK)
                return NOTOK;
 
        if (*cp != '/') {
                if (!magic)
-                       ci->ci_subtype = add ("", NULL);
+                       ci->ci_subtype = getcpy("");
                goto magic_skip;
        }
 
        cp++;
-       while (isspace (*cp))
+       while (isspace(*cp))
                cp++;
 
-       if (*cp == '(' && get_comment (ct, &cp, 1) == NOTOK)
+       if (*cp == '(' && get_comment(ct, &cp, 1) == NOTOK)
                return NOTOK;
 
-       for (dp = cp; istoken (*dp); dp++)
+       for (dp = cp; istoken(*dp); dp++)
                continue;
        c = *dp, *dp = '\0';
-       ci->ci_subtype = add (cp, NULL);  /* store the content subtype */
+       ci->ci_subtype = getcpy(cp);  /* store the content subtype */
        *dp = c, cp = dp;
 
        if (!*ci->ci_subtype) {
-               advise (NULL,
-                       "invalid %s: field in message %s (empty subtype for \"%s\")",
-                       TYPE_FIELD, ct->c_file, ci->ci_type);
+               advise(NULL, "invalid %s: field in message %s (empty subtype for \"%s\")", TYPE_FIELD, ct->c_file, ci->ci_type);
                return NOTOK;
        }
 
        /* down case the content subtype string */
        for (dp = ci->ci_subtype; *dp; dp++)
-               if (isalpha(*dp) && isupper (*dp))
-                       *dp = tolower (*dp);
+               if (isalpha(*dp) && isupper(*dp))
+                       *dp = tolower(*dp);
 
 magic_skip:
-       while (isspace (*cp))
+       while (isspace(*cp))
                cp++;
 
-       if (*cp == '(' && get_comment (ct, &cp, 1) == NOTOK)
+       if (*cp == '(' && get_comment(ct, &cp, 1) == NOTOK)
                return NOTOK;
 
        /*
@@ -781,43 +773,37 @@ magic_skip:
                unsigned char *up;
 
                if (ap >= ep) {
-                       advise (NULL,
-                               "too many parameters in message %s's %s: field (%d max)",
-                               ct->c_file, TYPE_FIELD, NPARMS);
+                       advise(NULL, "too many parameters in message %s's %s: field (%d max)", ct->c_file, TYPE_FIELD, NPARMS);
                        return NOTOK;
                }
 
                cp++;
-               while (isspace (*cp))
+               while (isspace(*cp))
                        cp++;
 
-               if (*cp == '(' && get_comment (ct, &cp, 1) == NOTOK)
+               if (*cp == '(' && get_comment(ct, &cp, 1) == NOTOK)
                        return NOTOK;
 
                if (*cp == 0) {
-                       advise (NULL,
-                               "extraneous trailing ';' in message %s's %s: parameter list",
-                               ct->c_file, TYPE_FIELD);
+                       advise (NULL, "extraneous trailing ';' in message %s's %s: parameter list", ct->c_file, TYPE_FIELD);
                        return OK;
                }
 
                /* down case the attribute name */
-               for (dp = cp; istoken (*dp); dp++)
-                       if (isalpha(*dp) && isupper (*dp))
-                               *dp = tolower (*dp);
+               for (dp = cp; istoken(*dp); dp++)
+                       if (isalpha(*dp) && isupper(*dp))
+                               *dp = tolower(*dp);
 
-               for (up = dp; isspace (*dp);)
+               for (up = dp; isspace(*dp);)
                        dp++;
                if (dp == cp || *dp != '=') {
-                       advise (NULL,
-                               "invalid parameter in message %s's %s: field\n%*.*sparameter %s (error detected at offset %d)",
-                               ct->c_file, TYPE_FIELD, i, i, "", cp, dp - cp);
+                       advise(NULL, "invalid parameter in message %s's %s: field\n%*.*sparameter %s (error detected at offset %d)", ct->c_file, TYPE_FIELD, i, i, "", cp, dp - cp);
                        return NOTOK;
                }
 
-               vp = (*ap = add (cp, NULL)) + (up - cp);
+               vp = (*ap = getcpy(cp)) + (up - cp);
                *vp = '\0';
-               for (dp++; isspace (*dp);)
+               for (dp++; isspace(*dp);)
                        dp++;
 
                /* now add the attribute value */
@@ -826,46 +812,42 @@ magic_skip:
                if (*dp == '"') {
                        for (cp = ++dp, dp = vp;;) {
                                switch (c = *cp++) {
-                                       case '\0':
+                               case '\0':
 bad_quote:
-                                               advise (NULL,
-                                                       "invalid quoted-string in message %s's %s: field\n%*.*s(parameter %s)",
-                                                       ct->c_file, TYPE_FIELD, i, i, "", *ap);
-                                               return NOTOK;
-
-                                       case '\\':
-                                               *dp++ = c;
-                                               if ((c = *cp++) == '\0')
-                                                       goto bad_quote;
-                                               /* else fall... */
-
-                                       default:
-                                               *dp++ = c;
-                                               continue;
+                                       advise(NULL, "invalid quoted-string in message %s's %s: field\n%*.*s(parameter %s)", ct->c_file, TYPE_FIELD, i, i, "", *ap);
+                                       return NOTOK;
 
-                                       case '"':
-                                               *dp = '\0';
-                                               break;
+                               case '\\':
+                                       *dp++ = c;
+                                       if ((c = *cp++) == '\0')
+                                               goto bad_quote;
+                                       /* else fall... */
+
+                               default:
+                                       *dp++ = c;
+                                       continue;
+
+                               case '"':
+                                       *dp = '\0';
+                                       break;
                                }
                                break;
                        }
                } else {
-                       for (cp = dp, dp = vp; istoken (*cp); cp++, dp++)
+                       for (cp = dp, dp = vp; istoken(*cp); cp++, dp++)
                                continue;
                        *dp = '\0';
                }
                if (!*vp) {
-                       advise (NULL,
-                               "invalid parameter in message %s's %s: field\n%*.*s(parameter %s)",
-                               ct->c_file, TYPE_FIELD, i, i, "", *ap);
+                       advise(NULL, "invalid parameter in message %s's %s: field\n%*.*s(parameter %s)", ct->c_file, TYPE_FIELD, i, i, "", *ap);
                        return NOTOK;
                }
                ap++;
 
-               while (isspace (*cp))
+               while (isspace(*cp))
                        cp++;
 
-               if (*cp == '(' && get_comment (ct, &cp, 1) == NOTOK)
+               if (*cp == '(' && get_comment(ct, &cp, 1) == NOTOK)
                        return NOTOK;
        }
 
@@ -874,23 +856,23 @@ bad_quote:
        */
        if (magic && *cp == '<') {
                if (ct->c_id) {
-                       free (ct->c_id);
+                       free(ct->c_id);
                        ct->c_id = NULL;
                }
                if (!(dp = strchr(ct->c_id = ++cp, '>'))) {
-                       advise (NULL, "invalid ID in message %s", ct->c_file);
+                       advise(NULL, "invalid ID in message %s", ct->c_file);
                        return NOTOK;
                }
                c = *dp;
                *dp = '\0';
                if (*ct->c_id)
-                       ct->c_id = concat ("<", ct->c_id, ">\n", NULL);
+                       ct->c_id = concat("<", ct->c_id, ">\n", NULL);
                else
                        ct->c_id = NULL;
                *dp++ = c;
                cp = dp;
 
-               while (isspace (*cp))
+               while (isspace(*cp))
                        cp++;
        }
 
@@ -899,11 +881,12 @@ bad_quote:
        */
        if (magic && *cp == '[') {
                ct->c_descr = ++cp;
-               for (dp = cp + strlen (cp) - 1; dp >= cp; dp--)
+               for (dp = cp + strlen(cp) - 1; dp >= cp; dp--)
                        if (*dp == ']')
                                break;
                if (dp < cp) {
-                       advise (NULL, "invalid description in message %s", ct->c_file);
+                       advise(NULL, "invalid description in message %s",
+                                       ct->c_file);
                        ct->c_descr = NULL;
                        return NOTOK;
                }
@@ -911,13 +894,13 @@ bad_quote:
                c = *dp;
                *dp = '\0';
                if (*ct->c_descr)
-                       ct->c_descr = concat (ct->c_descr, "\n", NULL);
+                       ct->c_descr = concat(ct->c_descr, "\n", NULL);
                else
                        ct->c_descr = NULL;
                *dp++ = c;
                cp = dp;
 
-               while (isspace (*cp))
+               while (isspace(*cp))
                        cp++;
        }
 
@@ -926,11 +909,12 @@ bad_quote:
        */
        if (magic && *cp == '{') {
                ct->c_dispo = ++cp;
-               for (dp = cp + strlen (cp) - 1; dp >= cp; dp--)
+               for (dp = cp + strlen(cp) - 1; dp >= cp; dp--)
                        if (*dp == '}')
                                break;
                if (dp < cp) {
-                       advise (NULL, "invalid disposition in message %s", ct->c_file);
+                       advise(NULL, "invalid disposition in message %s",
+                                       ct->c_file);
                        ct->c_dispo = NULL;
                        return NOTOK;
                }
@@ -938,13 +922,13 @@ bad_quote:
                c = *dp;
                *dp = '\0';
                if (*ct->c_dispo)
-                       ct->c_dispo = concat (ct->c_dispo, "\n", NULL);
+                       ct->c_dispo = concat(ct->c_dispo, "\n", NULL);
                else
                        ct->c_dispo = NULL;
                *dp++ = c;
                cp = dp;
 
-               while (isspace (*cp))
+               while (isspace(*cp))
                        cp++;
        }
 
@@ -953,20 +937,18 @@ bad_quote:
        */
        if (*cp) {
                if (magic) {
-                       ci->ci_magic = add (cp, NULL);
+                       ci->ci_magic = getcpy(cp);
 
                        /*
                        ** If there is a Content-Disposition header and
                        ** it doesn't have a *filename=, extract it from
-                       ** the magic contents.  The r1bindex call skips
+                       ** the magic contents.  The mhbasename call skips
                        ** any leading directory components.
                        */
                        if (ct->c_dispo)
-                               ct->c_dispo = incl_name_value (ct->c_dispo, "filename", r1bindex (extract_name_value ("name", ci->ci_magic), '/'));
+                               ct->c_dispo = incl_name_value(ct->c_dispo, "filename", mhbasename(extract_name_value("name", ci->ci_magic)));
                        } else
-                               advise (NULL,
-                                       "extraneous information in message %s's %s: field\n%*.*s(%s)",
-                                       ct->c_file, TYPE_FIELD, i, i, "", cp);
+                               advise(NULL, "extraneous information in message %s's %s: field\n%*.*s(%s)", ct->c_file, TYPE_FIELD, i, i, "", cp);
        }
 
        return OK;
@@ -974,7 +956,7 @@ bad_quote:
 
 
 static int
-get_comment (CT ct, unsigned char **ap, int istype)
+get_comment(CT ct, unsigned char **ap, int istype)
 {
        int i;
        char *bp;
@@ -991,7 +973,7 @@ get_comment (CT ct, unsigned char **ap, int istype)
                switch (c = *cp++) {
                case '\0':
 invalid:
-               advise (NULL, "invalid comment in message %s's %s: field",
+               advise(NULL, "invalid comment in message %s's %s: field",
                                ct->c_file, istype ? TYPE_FIELD : VRSN_FIELD);
                return NOTOK;
 
@@ -1021,14 +1003,14 @@ invalid:
 
        if (istype) {
                if ((dp = ci->ci_comment)) {
-                       ci->ci_comment = concat (dp, " ", buffer, NULL);
-                       free (dp);
+                       ci->ci_comment = concat(dp, " ", buffer, NULL);
+                       free(dp);
                } else {
-                       ci->ci_comment = add (buffer, NULL);
+                       ci->ci_comment = getcpy(buffer);
                }
        }
 
-       while (isspace (*cp))
+       while (isspace(*cp))
                cp++;
 
        *ap = cp;
@@ -1044,7 +1026,7 @@ invalid:
 */
 
 static int
-InitGeneric (CT ct)
+InitGeneric(CT ct)
 {
        return OK;  /* not much to do here */
 }
@@ -1055,7 +1037,7 @@ InitGeneric (CT ct)
 */
 
 static int
-InitText (CT ct)
+InitText(CT ct)
 {
        char buffer[BUFSIZ];
        char *chset = NULL;
@@ -1066,29 +1048,29 @@ InitText (CT ct)
 
        /* check for missing subtype */
        if (!*ci->ci_subtype)
-               ci->ci_subtype = add ("plain", ci->ci_subtype);
+               ci->ci_subtype = add("plain", ci->ci_subtype);
 
        /* match subtype */
        for (kv = SubText; kv->kv_key; kv++)
-               if (!mh_strcasecmp (ci->ci_subtype, kv->kv_key))
+               if (!mh_strcasecmp(ci->ci_subtype, kv->kv_key))
                        break;
        ct->c_subtype = kv->kv_value;
 
        /* allocate text character set structure */
-       if ((t = (struct text *) calloc (1, sizeof(*t))) == NULL)
-               adios (NULL, "out of memory");
+       if ((t = (struct text *) calloc(1, sizeof(*t))) == NULL)
+               adios(NULL, "out of memory");
        ct->c_ctparams = (void *) t;
 
        /* scan for charset parameter */
        for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++)
-               if (!mh_strcasecmp (*ap, "charset"))
+               if (!mh_strcasecmp(*ap, "charset"))
                        break;
 
        /* check if content specified a character set */
        if (*ap) {
                /* match character set or set to CHARSET_UNKNOWN */
                for (kv = Charset; kv->kv_key; kv++) {
-                       if (!mh_strcasecmp (*ep, kv->kv_key)) {
+                       if (!mh_strcasecmp(*ep, kv->kv_key)) {
                                chset = *ep;
                                break;
                        }
@@ -1105,10 +1087,11 @@ InitText (CT ct)
        **
        ** termproc is for mhshow, though mhlist -debug prints it, too.
        */
-       if (chset != NULL && !check_charset (chset, strlen (chset))) {
-               snprintf (buffer, sizeof(buffer), "%s-charset-%s", invo_name, chset);
-               if ((cp = context_find (buffer)))
-                       ct->c_termproc = getcpy (cp);
+       if (chset != NULL && !check_charset(chset, strlen(chset))) {
+               snprintf(buffer, sizeof(buffer), "%s-charset-%s",
+                               invo_name, chset);
+               if ((cp = context_find(buffer)))
+                       ct->c_termproc = getcpy(cp);
        }
 
        return OK;
@@ -1120,7 +1103,7 @@ InitText (CT ct)
 */
 
 static int
-InitMultiPart (CT ct)
+InitMultiPart(CT ct)
 {
        int inout;
        long last, pos;
@@ -1140,15 +1123,13 @@ InitMultiPart (CT ct)
        */
        if (ct->c_encoding != CE_7BIT && ct->c_encoding != CE_8BIT
                && ct->c_encoding != CE_BINARY) {
-               admonish (NULL,
-                         "\"%s/%s\" type in message %s must be encoded in 7bit, 8bit, or binary",
-                         ci->ci_type, ci->ci_subtype, ct->c_file);
+               admonish(NULL, "\"%s/%s\" type in message %s must be encoded in 7bit, 8bit, or binary", ci->ci_type, ci->ci_subtype, ct->c_file);
                return NOTOK;
        }
 
        /* match subtype */
        for (kv = SubMultiPart; kv->kv_key; kv++)
-               if (!mh_strcasecmp (ci->ci_subtype, kv->kv_key))
+               if (!mh_strcasecmp(ci->ci_subtype, kv->kv_key))
                        break;
        ct->c_subtype = kv->kv_value;
 
@@ -1158,7 +1139,7 @@ InitMultiPart (CT ct)
        */
        bp = 0;
        for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++) {
-               if (!mh_strcasecmp (*ap, "boundary")) {
+               if (!mh_strcasecmp(*ap, "boundary")) {
                        bp = *ep;
                        break;
                }
@@ -1166,64 +1147,62 @@ InitMultiPart (CT ct)
 
        /* complain if boundary parameter is missing */
        if (!*ap) {
-               advise (NULL,
-                       "a \"boundary\" parameter is mandatory for \"%s/%s\" type in message %s's %s: field",
-                       ci->ci_type, ci->ci_subtype, ct->c_file, TYPE_FIELD);
+               advise (NULL, "a \"boundary\" parameter is mandatory for \"%s/%s\" type in message %s's %s: field", ci->ci_type, ci->ci_subtype, ct->c_file, TYPE_FIELD);
                return NOTOK;
        }
 
        /* allocate primary structure for multipart info */
-       if ((m = (struct multipart *) calloc (1, sizeof(*m))) == NULL)
-               adios (NULL, "out of memory");
+       if ((m = (struct multipart *) calloc(1, sizeof(*m))) == NULL)
+               adios(NULL, "out of memory");
        ct->c_ctparams = (void *) m;
 
        /* check if boundary parameter contains only whitespace characters */
-       for (cp = bp; isspace (*cp); cp++)
+       for (cp = bp; isspace(*cp); cp++)
                continue;
        if (!*cp) {
-               advise (NULL, "invalid \"boundary\" parameter for \"%s/%s\" type in message %s's %s: field",
-                       ci->ci_type, ci->ci_subtype, ct->c_file, TYPE_FIELD);
+               advise(NULL, "invalid \"boundary\" parameter for \"%s/%s\" type in message %s's %s: field", ci->ci_type, ci->ci_subtype, ct->c_file, TYPE_FIELD);
                return NOTOK;
        }
 
        /* remove trailing whitespace from boundary parameter */
-       for (cp = bp, dp = cp + strlen (cp) - 1; dp > cp; dp--)
-               if (!isspace (*dp))
+       for (cp = bp, dp = cp + strlen(cp) - 1; dp > cp; dp--)
+               if (!isspace(*dp))
                        break;
        *++dp = '\0';
 
        /* record boundary separators */
-       m->mp_start = concat (bp, "\n", NULL);
-       m->mp_stop = concat (bp, "--\n", NULL);
+       m->mp_start = concat(bp, "\n", NULL);
+       m->mp_stop = concat(bp, "--\n", NULL);
 
-       if (!ct->c_fp && (ct->c_fp = fopen (ct->c_file, "r")) == NULL) {
-               advise (ct->c_file, "unable to open for reading");
+       if (!ct->c_fp && (ct->c_fp = fopen(ct->c_file, "r")) == NULL) {
+               advise(ct->c_file, "unable to open for reading");
                return NOTOK;
        }
 
-       fseek (fp = ct->c_fp, pos = ct->c_begin, SEEK_SET);
+       fseek(fp = ct->c_fp, pos = ct->c_begin, SEEK_SET);
        last = ct->c_end;
        next = &m->mp_parts;
        part = NULL;
        inout = 1;
 
-       while (fgets (buffer, sizeof(buffer) - 1, fp)) {
+       while (fgets(buffer, sizeof(buffer) - 1, fp)) {
                if (pos > last)
                        break;
 
-               pos += strlen (buffer);
+               pos += strlen(buffer);
                if (buffer[0] != '-' || buffer[1] != '-')
                        continue;
                if (inout) {
-                       if (strcmp (buffer + 2, m->mp_start))
+                       if (strcmp(buffer + 2, m->mp_start)!=0)
                                continue;
 next_part:
-                       if ((part = (struct part *) calloc (1, sizeof(*part))) == NULL)
-                               adios (NULL, "out of memory");
+                       if ((part = (struct part *) calloc(1, sizeof(*part)))
+                                       == NULL)
+                               adios(NULL, "out of memory");
                        *next = part;
                        next = &part->mp_next;
 
-                       if (!(p = get_content (fp, ct->c_file,
+                       if (!(p = get_content(fp, ct->c_file,
                                ct->c_subtype == MULTI_DIGEST ? -1 : 0))) {
                                ct->c_fp = NULL;
                                return NOTOK;
@@ -1231,10 +1210,10 @@ next_part:
                        p->c_fp = NULL;
                        part->mp_part = p;
                        pos = p->c_begin;
-                       fseek (fp, pos, SEEK_SET);
+                       fseek(fp, pos, SEEK_SET);
                        inout = 0;
                } else {
-                       if (strcmp (buffer + 2, m->mp_start) == 0) {
+                       if (strcmp(buffer + 2, m->mp_start) == 0) {
                                inout = 1;
 end_part:
                                p = part->mp_part;
@@ -1245,13 +1224,13 @@ end_part:
                                        goto next_part;
                                goto last_part;
                        } else {
-                               if (strcmp (buffer + 2, m->mp_stop) == 0)
+                               if (strcmp(buffer + 2, m->mp_stop) == 0)
                                        goto end_part;
                        }
                }
        }
 
-       advise (NULL, "bogus multipart content in message %s", ct->c_file);
+       advise(NULL, "bogus multipart content in message %s", ct->c_file);
        if (!inout && part) {
                p = part->mp_part;
                p->c_end = ct->c_end;
@@ -1261,15 +1240,15 @@ end_part:
                                next = &((*next)->mp_next))
                                continue;
                        *next = NULL;
-                       free_content (p);
-                       free ((char *) part);
+                       free_content(p);
+                       free((char *) part);
                }
        }
 
 last_part:
        /* reverse the order of the parts for multipart/alternative */
        if (ct->c_subtype == MULTI_ALTERNATE)
-               reverse_parts (ct);
+               reverse_parts(ct);
 
        /*
        ** label all subparts with part number, and
@@ -1281,8 +1260,9 @@ last_part:
                char partnam[BUFSIZ];
 
                if (ct->c_partno) {
-                       snprintf (partnam, sizeof(partnam), "%s.", ct->c_partno);
-                       pp = partnam + strlen (partnam);
+                       snprintf(partnam, sizeof(partnam), "%s.",
+                                       ct->c_partno);
+                       pp = partnam + strlen(partnam);
                } else {
                        pp = partnam;
                }
@@ -1291,19 +1271,19 @@ last_part:
                        part = part->mp_next, partnum++) {
                        p = part->mp_part;
 
-                       sprintf (pp, "%d", partnum);
-                       p->c_partno = add (partnam, NULL);
+                       sprintf(pp, "%d", partnum);
+                       p->c_partno = getcpy(partnam);
 
                        /* initialize the content of the subparts */
                        if (p->c_ctinitfnx && (*p->c_ctinitfnx) (p) == NOTOK) {
-                               fclose (ct->c_fp);
+                               fclose(ct->c_fp);
                                ct->c_fp = NULL;
                                return NOTOK;
                        }
                }
        }
 
-       fclose (ct->c_fp);
+       fclose(ct->c_fp);
        ct->c_fp = NULL;
        return OK;
 }
@@ -1314,7 +1294,7 @@ last_part:
 */
 
 static void
-reverse_parts (CT ct)
+reverse_parts(CT ct)
 {
        int i;
        struct multipart *m;
@@ -1332,8 +1312,8 @@ reverse_parts (CT ct)
                i++;
 
        /* allocate array of pointers to the parts */
-       if (!(base = (struct part **) calloc ((size_t) (i + 1), sizeof(*base))))
-               adios (NULL, "out of memory");
+       if (!(base = (struct part **) calloc((size_t) (i + 1), sizeof(*base))))
+               adios(NULL, "out of memory");
        bmp = base;
 
        /* point at all the parts */
@@ -1351,7 +1331,7 @@ reverse_parts (CT ct)
        *next = NULL;
 
        /* free array of pointers */
-       free ((char *) base);
+       free((char *) base);
 }
 
 
@@ -1360,165 +1340,154 @@ reverse_parts (CT ct)
 */
 
 static int
-InitMessage (CT ct)
+InitMessage(CT ct)
 {
        struct k2v *kv;
        CI ci = &ct->c_ctinfo;
 
        if ((ct->c_encoding != CE_7BIT) && (ct->c_encoding != CE_8BIT)) {
-               admonish (NULL,
-                       "\"%s/%s\" type in message %s should be encoded in 7bit or 8bit",
-                       ci->ci_type, ci->ci_subtype, ct->c_file);
+               admonish(NULL, "\"%s/%s\" type in message %s should be encoded in 7bit or 8bit", ci->ci_type, ci->ci_subtype, ct->c_file);
                return NOTOK;
        }
 
        /* check for missing subtype */
        if (!*ci->ci_subtype)
-               ci->ci_subtype = add ("rfc822", ci->ci_subtype);
+               ci->ci_subtype = add("rfc822", ci->ci_subtype);
 
        /* match subtype */
        for (kv = SubMessage; kv->kv_key; kv++)
-               if (!mh_strcasecmp (ci->ci_subtype, kv->kv_key))
+               if (!mh_strcasecmp(ci->ci_subtype, kv->kv_key))
                        break;
        ct->c_subtype = kv->kv_value;
 
        switch (ct->c_subtype) {
-               case MESSAGE_RFC822:
-                       break;
+       case MESSAGE_RFC822:
+               break;
 
-               case MESSAGE_PARTIAL:
-                       {
-                               char **ap, **ep;
-                               struct partial *p;
+       case MESSAGE_PARTIAL:
+               {
+               char **ap, **ep;
+               struct partial *p;
 
-                               if ((p = (struct partial *) calloc (1, sizeof(*p))) == NULL)
-                                       adios (NULL, "out of memory");
-                               ct->c_ctparams = (void *) p;
+               if ((p = (struct partial *) calloc(1, sizeof(*p))) == NULL)
+                       adios(NULL, "out of memory");
+               ct->c_ctparams = (void *) p;
 
-                               /*
-                               ** scan for parameters "id", "number",
-                               ** and "total"
-                               */
-                               for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++) {
-                                       if (!mh_strcasecmp (*ap, "id")) {
-                                               p->pm_partid = add (*ep, NULL);
-                                               continue;
-                                       }
-                                       if (!mh_strcasecmp (*ap, "number")) {
-                                               if (sscanf (*ep, "%d", &p->pm_partno) != 1
-                                                       || p->pm_partno < 1) {
+               /*
+               ** scan for parameters "id", "number",
+               ** and "total"
+               */
+               for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++) {
+                       if (!mh_strcasecmp(*ap, "id")) {
+                               p->pm_partid = getcpy(*ep);
+                               continue;
+                       }
+                       if (!mh_strcasecmp(*ap, "number")) {
+                               if (sscanf(*ep, "%d", &p->pm_partno) != 1 || p->pm_partno < 1) {
 invalid_param:
-                                                       advise (NULL,
-                                                               "invalid %s parameter for \"%s/%s\" type in message %s's %s field",
-                                                               *ap, ci->ci_type, ci->ci_subtype,
-                                                               ct->c_file, TYPE_FIELD);
-                                                       return NOTOK;
-                                               }
-                                               continue;
-                                       }
-                                       if (!mh_strcasecmp (*ap, "total")) {
-                                               if (sscanf (*ep, "%d", &p->pm_maxno) != 1
-                                                       || p->pm_maxno < 1)
-                                                       goto invalid_param;
-                                               continue;
-                                       }
-                               }
-
-                               if (!p->pm_partid || !p->pm_partno
-                                       || (p->pm_maxno && p->pm_partno > p->pm_maxno)) {
-                                       advise (NULL,
-                                               "invalid parameters for \"%s/%s\" type in message %s's %s field",
-                                               ci->ci_type, ci->ci_subtype,
-                                               ct->c_file, TYPE_FIELD);
+                                       advise(NULL, "invalid %s parameter for \"%s/%s\" type in message %s's %s field", *ap, ci->ci_type, ci->ci_subtype, ct->c_file, TYPE_FIELD);
                                        return NOTOK;
                                }
+                               continue;
                        }
-                       break;
-
-               case MESSAGE_EXTERNAL:
-                       {
-                               int exresult;
-                               struct exbody *e;
-                               CT p;
-                               FILE *fp;
+                       if (!mh_strcasecmp(*ap, "total")) {
+                               if (sscanf(*ep, "%d", &p->pm_maxno) != 1 ||
+                                               p->pm_maxno < 1)
+                                       goto invalid_param;
+                               continue;
+                       }
+               }
 
-                               if ((e = (struct exbody *) calloc (1, sizeof(*e))) == NULL)
-                                       adios (NULL, "out of memory");
-                               ct->c_ctparams = (void *) e;
+               if (!p->pm_partid || !p->pm_partno
+                       || (p->pm_maxno && p->pm_partno > p->pm_maxno)) {
+                       advise(NULL, "invalid parameters for \"%s/%s\" type in message %s's %s field", ci->ci_type, ci->ci_subtype, ct->c_file, TYPE_FIELD);
+                       return NOTOK;
+               }
+               }
+               break;
 
-                               if (!ct->c_fp && (ct->c_fp = fopen (ct->c_file, "r")) == NULL) {
-                                       advise (ct->c_file, "unable to open for reading");
-                                       return NOTOK;
-                               }
+       case MESSAGE_EXTERNAL:
+               {
+               int exresult;
+               struct exbody *e;
+               CT p;
+               FILE *fp;
 
-                               fseek (fp = ct->c_fp, ct->c_begin, SEEK_SET);
+               if ((e = (struct exbody *) calloc(1, sizeof(*e))) == NULL)
+                       adios(NULL, "out of memory");
+               ct->c_ctparams = (void *) e;
 
-                               if (!(p = get_content (fp, ct->c_file, 0))) {
-                                       ct->c_fp = NULL;
-                                       return NOTOK;
-                               }
+               if (!ct->c_fp && (ct->c_fp = fopen(ct->c_file, "r")) == NULL) {
+                       advise(ct->c_file, "unable to open for reading");
+                       return NOTOK;
+               }
 
-                               e->eb_parent = ct;
-                               e->eb_content = p;
-                               p->c_ctexbody = e;
-                               if ((exresult = params_external (ct, 0)) != NOTOK
-                                       && p->c_ceopenfnx == openMail) {
-                                       int cc, size;
-                                       char *bp;
-
-                                       if ((size = ct->c_end - p->c_begin) <= 0) {
-                                               if (!e->eb_subject)
-                                                       content_error (NULL, ct, "empty body for access-type=mail-server");
-                                               goto no_body;
-                                       }
+               fseek(fp = ct->c_fp, ct->c_begin, SEEK_SET);
 
-                                       e->eb_body = bp = mh_xmalloc ((unsigned) size);
-                                       fseek (p->c_fp, p->c_begin, SEEK_SET);
-                                       while (size > 0)
-                                               switch (cc = fread (bp, sizeof(*bp), size, p->c_fp)) {
-                                                       case NOTOK:
-                                                               adios ("failed", "fread");
+               if (!(p = get_content(fp, ct->c_file, 0))) {
+                       ct->c_fp = NULL;
+                       return NOTOK;
+               }
 
-                                                       case OK:
-                                                               adios (NULL, "unexpected EOF from fread");
+               e->eb_parent = ct;
+               e->eb_content = p;
+               p->c_ctexbody = e;
+               if ((exresult = params_external(ct, 0)) != NOTOK &&
+                               p->c_ceopenfnx == openMail) {
+                       int cc, size;
+                       char *bp;
+
+                       if ((size = ct->c_end - p->c_begin) <= 0) {
+                               if (!e->eb_subject)
+                                       content_error(NULL, ct, "empty body for access-type=mail-server");
+                               goto no_body;
+                       }
 
-                                                       default:
-                                                               bp += cc, size -= cc;
-                                                               break;
-                                               }
-                                       *bp = 0;
+                       e->eb_body = bp = mh_xmalloc((unsigned) size);
+                       fseek(p->c_fp, p->c_begin, SEEK_SET);
+                       while (size > 0)
+                               switch (cc = fread(bp, sizeof(*bp), size, p->c_fp)) {
+                               case NOTOK:
+                                       adios("failed", "fread");
+                               case OK:
+                                       adios(NULL, "unexpected EOF from fread");
+                               default:
+                                       bp += cc, size -= cc;
+                                       break;
                                }
+                       *bp = 0;
+               }
 no_body:
-                               p->c_fp = NULL;
-                               p->c_end = p->c_begin;
+               p->c_fp = NULL;
+               p->c_end = p->c_begin;
 
-                               fclose (ct->c_fp);
-                               ct->c_fp = NULL;
-
-                               if (exresult == NOTOK)
-                                       return NOTOK;
-                               if (e->eb_flags == NOTOK)
-                                       return OK;
+               fclose(ct->c_fp);
+               ct->c_fp = NULL;
 
-                               switch (p->c_type) {
-                                       case CT_MULTIPART:
-                                               break;
+               if (exresult == NOTOK)
+                       return NOTOK;
+               if (e->eb_flags == NOTOK)
+                       return OK;
 
-                                       case CT_MESSAGE:
-                                               if (p->c_subtype != MESSAGE_RFC822)
-                                                       break;
-                                               /* else fall... */
-                                       default:
-                                               e->eb_partno = ct->c_partno;
-                                               if (p->c_ctinitfnx)
-                                                       (*p->c_ctinitfnx) (p);
-                                               break;
-                               }
-                       }
+               switch (p->c_type) {
+               case CT_MULTIPART:
                        break;
 
+               case CT_MESSAGE:
+                       if (p->c_subtype != MESSAGE_RFC822)
+                               break;
+                               /* else fall... */
                default:
+                       e->eb_partno = ct->c_partno;
+                       if (p->c_ctinitfnx)
+                               (*p->c_ctinitfnx) (p);
                        break;
+               }
+               }
+               break;
+
+       default:
+               break;
        }
 
        return OK;
@@ -1526,19 +1495,19 @@ no_body:
 
 
 int
-params_external (CT ct, int composing)
+params_external(CT ct, int composing)
 {
        char **ap, **ep;
        struct exbody *e = (struct exbody *) ct->c_ctparams;
        CI ci = &ct->c_ctinfo;
 
        for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++) {
-               if (!mh_strcasecmp (*ap, "access-type")) {
+               if (!mh_strcasecmp(*ap, "access-type")) {
                        struct str2init *s2i;
                        CT p = e->eb_content;
 
                        for (s2i = str2methods; s2i->si_key; s2i++)
-                               if (!mh_strcasecmp (*ep, s2i->si_key))
+                               if (!mh_strcasecmp(*ep, s2i->si_key))
                                        break;
                        if (!s2i->si_key) {
                                e->eb_access = *ep;
@@ -1555,48 +1524,46 @@ params_external (CT ct, int composing)
                                return NOTOK;
                        continue;
                }
-               if (!mh_strcasecmp (*ap, "name")) {
+               if (!mh_strcasecmp(*ap, "name")) {
                        e->eb_name = *ep;
                        continue;
                }
-               if (!mh_strcasecmp (*ap, "permission")) {
+               if (!mh_strcasecmp(*ap, "permission")) {
                        e->eb_permission = *ep;
                        continue;
                }
-               if (!mh_strcasecmp (*ap, "site")) {
+               if (!mh_strcasecmp(*ap, "site")) {
                        e->eb_site = *ep;
                        continue;
                }
-               if (!mh_strcasecmp (*ap, "directory")) {
+               if (!mh_strcasecmp(*ap, "directory")) {
                        e->eb_dir = *ep;
                        continue;
                }
-               if (!mh_strcasecmp (*ap, "mode")) {
+               if (!mh_strcasecmp(*ap, "mode")) {
                        e->eb_mode = *ep;
                        continue;
                }
-               if (!mh_strcasecmp (*ap, "size")) {
-                       sscanf (*ep, "%lu", &e->eb_size);
+               if (!mh_strcasecmp(*ap, "size")) {
+                       sscanf(*ep, "%lu", &e->eb_size);
                        continue;
                }
-               if (!mh_strcasecmp (*ap, "server")) {
+               if (!mh_strcasecmp(*ap, "server")) {
                        e->eb_server = *ep;
                        continue;
                }
-               if (!mh_strcasecmp (*ap, "subject")) {
+               if (!mh_strcasecmp(*ap, "subject")) {
                        e->eb_subject = *ep;
                        continue;
                }
-               if (composing && !mh_strcasecmp (*ap, "body")) {
-                       e->eb_body = getcpy (*ep);
+               if (composing && !mh_strcasecmp(*ap, "body")) {
+                       e->eb_body = getcpy(*ep);
                        continue;
                }
        }
 
        if (!e->eb_access) {
-               advise (NULL,
-                       "invalid parameters for \"%s/%s\" type in message %s's %s field",
-                       ci->ci_type, ci->ci_subtype, ct->c_file, TYPE_FIELD);
+               advise(NULL, "invalid parameters for \"%s/%s\" type in message %s's %s field", ci->ci_type, ci->ci_subtype, ct->c_file, TYPE_FIELD);
                return NOTOK;
        }
 
@@ -1609,14 +1576,14 @@ params_external (CT ct, int composing)
 */
 
 static int
-InitApplication (CT ct)
+InitApplication(CT ct)
 {
        struct k2v *kv;
        CI ci = &ct->c_ctinfo;
 
        /* match subtype */
        for (kv = SubApplication; kv->kv_key; kv++)
-               if (!mh_strcasecmp (ci->ci_subtype, kv->kv_key))
+               if (!mh_strcasecmp(ci->ci_subtype, kv->kv_key))
                        break;
        ct->c_subtype = kv->kv_value;
 
@@ -1629,12 +1596,12 @@ InitApplication (CT ct)
 */
 
 static int
-init_encoding (CT ct, OpenCEFunc openfnx)
+init_encoding(CT ct, OpenCEFunc openfnx)
 {
        CE ce;
 
-       if ((ce = (CE) calloc (1, sizeof(*ce))) == NULL)
-               adios (NULL, "out of memory");
+       if ((ce = (CE) calloc(1, sizeof(*ce))) == NULL)
+               adios(NULL, "out of memory");
 
        ct->c_cefile     = ce;
        ct->c_ceopenfnx  = openfnx;
@@ -1646,7 +1613,7 @@ init_encoding (CT ct, OpenCEFunc openfnx)
 
 
 void
-close_encoding (CT ct)
+close_encoding(CT ct)
 {
        CE ce;
 
@@ -1654,14 +1621,14 @@ close_encoding (CT ct)
                return;
 
        if (ce->ce_fp) {
-               fclose (ce->ce_fp);
+               fclose(ce->ce_fp);
                ce->ce_fp = NULL;
        }
 }
 
 
 static unsigned long
-size_encoding (CT ct)
+size_encoding(CT ct)
 {
        int fd;
        unsigned long size;
@@ -1672,11 +1639,11 @@ size_encoding (CT ct)
        if (!(ce = ct->c_cefile))
                return (ct->c_end - ct->c_begin);
 
-       if (ce->ce_fp && fstat (fileno (ce->ce_fp), &st) != NOTOK)
+       if (ce->ce_fp && fstat(fileno(ce->ce_fp), &st) != NOTOK)
                return (long) st.st_size;
 
        if (ce->ce_file) {
-               if (stat (ce->ce_file, &st) != NOTOK)
+               if (stat(ce->ce_file, &st) != NOTOK)
                        return (long) st.st_size;
                else
                        return 0L;
@@ -1689,7 +1656,7 @@ size_encoding (CT ct)
        if ((fd = (*ct->c_ceopenfnx) (ct, &file)) == NOTOK)
                return (ct->c_end - ct->c_begin);
 
-       if (fstat (fd, &st) != NOTOK)
+       if (fstat(fd, &st) != NOTOK)
                size = (long) st.st_size;
        else
                size = 0L;
@@ -1724,14 +1691,14 @@ static unsigned char b642nib[0x80] = {
 
 
 static int
-InitBase64 (CT ct)
+InitBase64(CT ct)
 {
-       return init_encoding (ct, openBase64);
+       return init_encoding(ct, openBase64);
 }
 
 
 static int
-openBase64 (CT ct, char **file)
+openBase64(CT ct, char **file)
 {
        int bitno, cc, digested;
        int fd, len, skip;
@@ -1751,35 +1718,36 @@ openBase64 (CT ct, char **file)
 
        ce = ct->c_cefile;
        if (ce->ce_fp) {
-               fseek (ce->ce_fp, 0L, SEEK_SET);
+               fseek(ce->ce_fp, 0L, SEEK_SET);
                goto ready_to_go;
        }
 
        if (ce->ce_file) {
-               if ((ce->ce_fp = fopen (ce->ce_file, "r")) == NULL) {
-                       content_error (ce->ce_file, ct, "unable to fopen for reading");
+               if ((ce->ce_fp = fopen(ce->ce_file, "r")) == NULL) {
+                       content_error(ce->ce_file, ct,
+                                       "unable to fopen for reading");
                        return NOTOK;
                }
                goto ready_to_go;
        }
 
        if (*file == NULL) {
-               ce->ce_file = add (m_mktemp(tmp, NULL, NULL), NULL);
+               ce->ce_file = getcpy(m_mktemp(tmp, NULL, NULL));
                ce->ce_unlink = 1;
        } else {
-               ce->ce_file = add (*file, NULL);
+               ce->ce_file = getcpy(*file);
                ce->ce_unlink = 0;
        }
 
        /* sbeck@cise.ufl.edu -- handle suffixes */
        ci = &ct->c_ctinfo;
-       snprintf (buffer, sizeof(buffer), "%s-suffix-%s/%s",
-                 invo_name, ci->ci_type, ci->ci_subtype);
-       cp = context_find (buffer);
+       snprintf(buffer, sizeof(buffer), "%s-suffix-%s/%s",
+                       invo_name, ci->ci_type, ci->ci_subtype);
+       cp = context_find(buffer);
        if (cp == NULL || *cp == '\0') {
-               snprintf (buffer, sizeof(buffer), "%s-suffix-%s", invo_name,
+               snprintf(buffer, sizeof(buffer), "%s-suffix-%s", invo_name,
                          ci->ci_type);
-               cp = context_find (buffer);
+               cp = context_find(buffer);
        }
        if (cp != NULL && *cp != '\0') {
                if (ce->ce_unlink) {
@@ -1788,46 +1756,48 @@ openBase64 (CT ct, char **file)
                        ** version with extension.
                        */
                        char *file_org = strdup(ce->ce_file);
-                       ce->ce_file = add (cp, ce->ce_file);
+                       ce->ce_file = add(cp, ce->ce_file);
                        if (rename(file_org, ce->ce_file)) {
-                               adios (ce->ce_file, "unable to rename %s to ", file_org);
+                               adios(ce->ce_file, "unable to rename %s to ",
+                                               file_org);
                        }
                        free(file_org);
 
                } else {
-                       ce->ce_file = add (cp, ce->ce_file);
+                       ce->ce_file = add(cp, ce->ce_file);
                }
        }
 
-       if ((ce->ce_fp = fopen (ce->ce_file, "w+")) == NULL) {
-               content_error (ce->ce_file, ct, "unable to fopen for reading/writing");
+       if ((ce->ce_fp = fopen(ce->ce_file, "w+")) == NULL) {
+               content_error(ce->ce_file, ct,
+                               "unable to fopen for reading/writing");
                return NOTOK;
        }
 
        if ((len = ct->c_end - ct->c_begin) < 0)
-               adios (NULL, "internal error(1)");
+               adios(NULL, "internal error(1)");
 
-       if (!ct->c_fp && (ct->c_fp = fopen (ct->c_file, "r")) == NULL) {
-               content_error (ct->c_file, ct, "unable to open for reading");
+       if (!ct->c_fp && (ct->c_fp = fopen(ct->c_file, "r")) == NULL) {
+               content_error(ct->c_file, ct, "unable to open for reading");
                return NOTOK;
        }
 
        if ((digested = ct->c_digested))
-               MD5Init (&mdContext);
+               MD5Init(&mdContext);
 
        bitno = 18;
        bits = 0L;
        skip = 0;
 
-       lseek (fd = fileno (ct->c_fp), (off_t) ct->c_begin, SEEK_SET);
+       lseek(fd = fileno(ct->c_fp), (off_t) ct->c_begin, SEEK_SET);
        while (len > 0) {
-               switch (cc = read (fd, buffer, sizeof(buffer) - 1)) {
+               switch (cc = read(fd, buffer, sizeof(buffer) - 1)) {
                case NOTOK:
-                       content_error (ct->c_file, ct, "error reading from");
+                       content_error(ct->c_file, ct, "error reading from");
                        goto clean_up;
 
                case OK:
-                       content_error (NULL, ct, "premature eof");
+                       content_error(NULL, ct, "premature eof");
                        goto clean_up;
 
                default:
@@ -1838,37 +1808,35 @@ openBase64 (CT ct, char **file)
                        for (ep = (cp = buffer) + cc; cp < ep; cp++) {
                                switch (*cp) {
                                default:
-                                       if (isspace (*cp))
+                                       if (isspace(*cp))
                                                break;
-                                       if (skip || (*cp & 0x80)
-                                               || (value = b642nib[*cp & 0x7f]) > 0x3f) {
+                                       if (skip || (*cp & 0x80) || (value = b642nib[*cp & 0x7f]) > 0x3f) {
                                                if (debugsw) {
-                                                       fprintf (stderr, "*cp=0x%x pos=%ld skip=%d\n", *cp, (long) (lseek (fd, (off_t) 0, SEEK_CUR) - (ep - cp)), skip);
+                                                       fprintf(stderr, "*cp=0x%x pos=%ld skip=%d\n", *cp, (long) (lseek(fd, (off_t) 0, SEEK_CUR) - (ep - cp)), skip);
                                                }
-                                               content_error (NULL, ct,
-                                                                          "invalid BASE64 encoding -- continuing");
+                                               content_error(NULL, ct, "invalid BASE64 encoding -- continuing");
                                                continue;
                                        }
 
                                        bits |= value << bitno;
 test_end:
                                        if ((bitno -= 6) < 0) {
-                                               putc ((char) *b1, ce->ce_fp);
+                                               putc((char) *b1, ce->ce_fp);
                                                if (digested)
-                                                       MD5Update (&mdContext, b1, 1);
+                                                       MD5Update(&mdContext, b1, 1);
                                                if (skip < 2) {
-                                                       putc ((char) *b2, ce->ce_fp);
+                                                       putc((char) *b2, ce->ce_fp);
                                                        if (digested)
-                                                               MD5Update (&mdContext, b2, 1);
+                                                               MD5Update(&mdContext, b2, 1);
                                                        if (skip < 1) {
-                                                               putc ((char) *b3, ce->ce_fp);
+                                                               putc((char) *b3, ce->ce_fp);
                                                                if (digested)
-                                                                       MD5Update (&mdContext, b3, 1);
+                                                                       MD5Update(&mdContext, b3, 1);
                                                        }
                                                }
 
-                                               if (ferror (ce->ce_fp)) {
-                                                       content_error (ce->ce_file, ct,
+                                               if (ferror(ce->ce_fp)) {
+                                                       content_error(ce->ce_file, ct,
                                                                                   "error writing to");
                                                        goto clean_up;
                                                }
@@ -1887,39 +1855,40 @@ test_end:
 
        if (bitno != 18) {
                if (debugsw)
-                       fprintf (stderr, "premature ending (bitno %d)\n", bitno);
+                       fprintf(stderr, "premature ending (bitno %d)\n",
+                                       bitno);
 
-               content_error (NULL, ct, "invalid BASE64 encoding");
+               content_error(NULL, ct, "invalid BASE64 encoding");
                goto clean_up;
        }
 
 self_delimiting:
-       fseek (ct->c_fp, 0L, SEEK_SET);
+       fseek(ct->c_fp, 0L, SEEK_SET);
 
-       if (fflush (ce->ce_fp)) {
-               content_error (ce->ce_file, ct, "error writing to");
+       if (fflush(ce->ce_fp)) {
+               content_error(ce->ce_file, ct, "error writing to");
                goto clean_up;
        }
 
        if (digested) {
                unsigned char digest[16];
 
-               MD5Final (digest, &mdContext);
+               MD5Final(digest, &mdContext);
                if (memcmp((char *) digest, (char *) ct->c_digest,
                        sizeof(digest) / sizeof(digest[0])))
-                       content_error (NULL, ct, "content integrity suspect (digest mismatch) -- continuing");
+                       content_error(NULL, ct, "content integrity suspect (digest mismatch) -- continuing");
                else if (debugsw)
-                       fprintf (stderr, "content integrity confirmed\n");
+                       fprintf(stderr, "content integrity confirmed\n");
        }
 
-       fseek (ce->ce_fp, 0L, SEEK_SET);
+       fseek(ce->ce_fp, 0L, SEEK_SET);
 
 ready_to_go:
        *file = ce->ce_file;
-       return fileno (ce->ce_fp);
+       return fileno(ce->ce_fp);
 
 clean_up:
-       free_encoding (ct, 0);
+       free_encoding(ct, 0);
        return NOTOK;
 }
 
@@ -1949,14 +1918,14 @@ static char hex2nib[0x80] = {
 
 
 static int
-InitQuoted (CT ct)
+InitQuoted(CT ct)
 {
-       return init_encoding (ct, openQuoted);
+       return init_encoding(ct, openQuoted);
 }
 
 
 static int
-openQuoted (CT ct, char **file)
+openQuoted(CT ct, char **file)
 {
        int cc, digested, len, quoted;
        unsigned char *cp, *ep;
@@ -1969,13 +1938,13 @@ openQuoted (CT ct, char **file)
 
        ce = ct->c_cefile;
        if (ce->ce_fp) {
-               fseek (ce->ce_fp, 0L, SEEK_SET);
+               fseek(ce->ce_fp, 0L, SEEK_SET);
                goto ready_to_go;
        }
 
        if (ce->ce_file) {
-               if ((ce->ce_fp = fopen (ce->ce_file, "r")) == NULL) {
-                       content_error (ce->ce_file, ct,
+               if ((ce->ce_fp = fopen(ce->ce_file, "r")) == NULL) {
+                       content_error(ce->ce_file, ct,
                                        "unable to fopen for reading");
                        return NOTOK;
                }
@@ -1983,81 +1952,81 @@ openQuoted (CT ct, char **file)
        }
 
        if (*file == NULL) {
-               ce->ce_file = add (m_mktemp(tmp, NULL, NULL), NULL);
+               ce->ce_file = getcpy(m_mktemp(tmp, NULL, NULL));
                ce->ce_unlink = 1;
        } else {
-               ce->ce_file = add (*file, NULL);
+               ce->ce_file = getcpy(*file);
                ce->ce_unlink = 0;
        }
 
        /* sbeck@cise.ufl.edu -- handle suffixes */
        ci = &ct->c_ctinfo;
-       snprintf (buffer, sizeof(buffer), "%s-suffix-%s/%s",
-                         invo_name, ci->ci_type, ci->ci_subtype);
-       cp = context_find (buffer);
+       snprintf(buffer, sizeof(buffer), "%s-suffix-%s/%s",
+                       invo_name, ci->ci_type, ci->ci_subtype);
+       cp = context_find(buffer);
        if (cp == NULL || *cp == '\0') {
-               snprintf (buffer, sizeof(buffer), "%s-suffix-%s", invo_name,
-                                 ci->ci_type);
-               cp = context_find (buffer);
+               snprintf(buffer, sizeof(buffer), "%s-suffix-%s", invo_name,
+                               ci->ci_type);
+               cp = context_find(buffer);
        }
        if (cp != NULL && *cp != '\0') {
                if (ce->ce_unlink) {
                        // Temporary file already exists, so we rename to
                        // version with extension.
                        char *file_org = strdup(ce->ce_file);
-                       ce->ce_file = add (cp, ce->ce_file);
+                       ce->ce_file = add(cp, ce->ce_file);
                        if (rename(file_org, ce->ce_file)) {
-                               adios (ce->ce_file, "unable to rename %s to ",
+                               adios(ce->ce_file, "unable to rename %s to ",
                                                file_org);
                        }
                        free(file_org);
 
                } else {
-                       ce->ce_file = add (cp, ce->ce_file);
+                       ce->ce_file = add(cp, ce->ce_file);
                }
        }
 
-       if ((ce->ce_fp = fopen (ce->ce_file, "w+")) == NULL) {
-               content_error (ce->ce_file, ct,
+       if ((ce->ce_fp = fopen(ce->ce_file, "w+")) == NULL) {
+               content_error(ce->ce_file, ct,
                                "unable to fopen for reading/writing");
                return NOTOK;
        }
 
-       if ((ce->ce_fp = fopen (ce->ce_file, "w+")) == NULL) {
-               content_error (ce->ce_file, ct,
+       if ((ce->ce_fp = fopen(ce->ce_file, "w+")) == NULL) {
+               content_error(ce->ce_file, ct,
                                "unable to fopen for reading/writing");
                return NOTOK;
        }
 
        if ((len = ct->c_end - ct->c_begin) < 0)
-               adios (NULL, "internal error(2)");
+               adios(NULL, "internal error(2)");
 
-       if (!ct->c_fp && (ct->c_fp = fopen (ct->c_file, "r")) == NULL) {
-               content_error (ct->c_file, ct, "unable to open for reading");
+       if (!ct->c_fp && (ct->c_fp = fopen(ct->c_file, "r")) == NULL) {
+               content_error(ct->c_file, ct, "unable to open for reading");
                return NOTOK;
        }
 
        if ((digested = ct->c_digested))
-               MD5Init (&mdContext);
+               MD5Init(&mdContext);
 
        quoted = 0;
 #ifdef lint
        mask = 0;
 #endif
 
-       fseek (ct->c_fp, ct->c_begin, SEEK_SET);
+       fseek(ct->c_fp, ct->c_begin, SEEK_SET);
        while (len > 0) {
-               if (fgets (buffer, sizeof(buffer) - 1, ct->c_fp) == NULL) {
-                       content_error (NULL, ct, "premature eof");
+               if (fgets(buffer, sizeof(buffer) - 1, ct->c_fp) == NULL) {
+                       content_error(NULL, ct, "premature eof");
                        goto clean_up;
                }
 
-               if ((cc = strlen (buffer)) > len)
+               if ((cc = strlen(buffer)) > len)
                        cc = len;
                len -= cc;
 
                for (ep = (cp = buffer) + cc - 1; cp <= ep; ep--)
-                       if (!isspace (*ep))
+                       if (!isspace(*ep))
                                break;
                *++ep = '\n', ep++;
 
@@ -2073,11 +2042,11 @@ openQuoted (CT ct, char **file)
                                        /* at byte 2 of an escape sequence */
                                        mask <<= 4;
                                        mask |= hex2nib[*cp & 0x7f];
-                                       putc (mask, ce->ce_fp);
+                                       putc(mask, ce->ce_fp);
                                        if (digested)
-                                               MD5Update (&mdContext, &mask, 1);
-                                       if (ferror (ce->ce_fp)) {
-                                               content_error (ce->ce_file, ct, "error writing to");
+                                               MD5Update(&mdContext, &mask, 1);
+                                       if (ferror(ce->ce_fp)) {
+                                               content_error(ce->ce_file, ct, "error writing to");
                                                goto clean_up;
                                        }
                                        /*
@@ -2108,7 +2077,7 @@ openQuoted (CT ct, char **file)
                                        ** sequence; just show the raw bytes
                                        ** (below).
                                        */
-                               } else if (isxdigit (cp[1]) && isxdigit (cp[2])) {
+                               } else if (isxdigit(cp[1]) && isxdigit(cp[2])) {
                                        /*
                                        ** Next 2 bytes are hex digits,
                                        ** making this a valid escape
@@ -2127,53 +2096,52 @@ openQuoted (CT ct, char **file)
                        }
 
                        /* Just show the raw byte. */
-                       putc (*cp, ce->ce_fp);
+                       putc(*cp, ce->ce_fp);
                        if (digested) {
                                if (*cp == '\n') {
-                                       MD5Update (&mdContext, (unsigned char *) "\r\n",2);
+                                       MD5Update(&mdContext, (unsigned char *) "\r\n",2);
                                } else {
-                                       MD5Update (&mdContext, (unsigned char *) cp, 1);
+                                       MD5Update(&mdContext, (unsigned char *) cp, 1);
                                }
                        }
-                       if (ferror (ce->ce_fp)) {
-                               content_error (ce->ce_file, ct, "error writing to");
+                       if (ferror(ce->ce_fp)) {
+                               content_error(ce->ce_file, ct,
+                                               "error writing to");
                                goto clean_up;
                        }
                }
        }
        if (quoted) {
-               content_error (NULL, ct,
-                       "invalid QUOTED-PRINTABLE encoding -- end-of-content while still quoting");
+               content_error(NULL, ct, "invalid QUOTED-PRINTABLE encoding -- end-of-content while still quoting");
                goto clean_up;
        }
 
-       fseek (ct->c_fp, 0L, SEEK_SET);
+       fseek(ct->c_fp, 0L, SEEK_SET);
 
-       if (fflush (ce->ce_fp)) {
-               content_error (ce->ce_file, ct, "error writing to");
+       if (fflush(ce->ce_fp)) {
+               content_error(ce->ce_file, ct, "error writing to");
                goto clean_up;
        }
 
        if (digested) {
                unsigned char digest[16];
 
-               MD5Final (digest, &mdContext);
+               MD5Final(digest, &mdContext);
                if (memcmp((char *) digest, (char *) ct->c_digest,
                        sizeof(digest) / sizeof(digest[0])))
-                       content_error (NULL, ct,
-                               "content integrity suspect (digest mismatch) -- continuing");
+                       content_error(NULL, ct, "content integrity suspect (digest mismatch) -- continuing");
                else if (debugsw)
-                       fprintf (stderr, "content integrity confirmed\n");
+                       fprintf(stderr, "content integrity confirmed\n");
        }
 
-       fseek (ce->ce_fp, 0L, SEEK_SET);
+       fseek(ce->ce_fp, 0L, SEEK_SET);
 
 ready_to_go:
        *file = ce->ce_file;
-       return fileno (ce->ce_fp);
+       return fileno(ce->ce_fp);
 
 clean_up:
-       free_encoding (ct, 0);
+       free_encoding(ct, 0);
        return NOTOK;
 }
 
@@ -2183,9 +2151,9 @@ clean_up:
 */
 
 static int
-Init7Bit (CT ct)
+Init7Bit(CT ct)
 {
-       if (init_encoding (ct, open7Bit) == NOTOK)
+       if (init_encoding(ct, open7Bit) == NOTOK)
                return NOTOK;
 
        ct->c_cesizefnx = NULL;  /* no need to decode for real size */
@@ -2194,7 +2162,7 @@ Init7Bit (CT ct)
 
 
 int
-open7Bit (CT ct, char **file)
+open7Bit(CT ct, char **file)
 {
        int cc, fd, len;
        char buffer[BUFSIZ];
@@ -2205,13 +2173,13 @@ open7Bit (CT ct, char **file)
 
        ce = ct->c_cefile;
        if (ce->ce_fp) {
-               fseek (ce->ce_fp, 0L, SEEK_SET);
+               fseek(ce->ce_fp, 0L, SEEK_SET);
                goto ready_to_go;
        }
 
        if (ce->ce_file) {
-               if ((ce->ce_fp = fopen (ce->ce_file, "r")) == NULL) {
-                       content_error (ce->ce_file, ct,
+               if ((ce->ce_fp = fopen(ce->ce_file, "r")) == NULL) {
+                       content_error(ce->ce_file, ct,
                                        "unable to fopen for reading");
                        return NOTOK;
                }
@@ -2219,22 +2187,22 @@ open7Bit (CT ct, char **file)
        }
 
        if (*file == NULL) {
-               ce->ce_file = add (m_mktemp(tmp, NULL, NULL), NULL);
+               ce->ce_file = getcpy(m_mktemp(tmp, NULL, NULL));
                ce->ce_unlink = 1;
        } else {
-               ce->ce_file = add (*file, NULL);
+               ce->ce_file = getcpy(*file);
                ce->ce_unlink = 0;
        }
 
        /* sbeck@cise.ufl.edu -- handle suffixes */
        ci = &ct->c_ctinfo;
-       snprintf (buffer, sizeof(buffer), "%s-suffix-%s/%s",
-               invo_name, ci->ci_type, ci->ci_subtype);
-       cp = context_find (buffer);
+       snprintf(buffer, sizeof(buffer), "%s-suffix-%s/%s",
+                       invo_name, ci->ci_type, ci->ci_subtype);
+       cp = context_find(buffer);
        if (cp == NULL || *cp == '\0') {
-               snprintf (buffer, sizeof(buffer), "%s-suffix-%s", invo_name,
+               snprintf(buffer, sizeof(buffer), "%s-suffix-%s", invo_name,
                        ci->ci_type);
-               cp = context_find (buffer);
+               cp = context_find(buffer);
        }
        if (cp != NULL && *cp != '\0') {
                if (ce->ce_unlink) {
@@ -2243,20 +2211,20 @@ open7Bit (CT ct, char **file)
                        ** version with extension.
                        */
                        char *file_org = strdup(ce->ce_file);
-                       ce->ce_file = add (cp, ce->ce_file);
+                       ce->ce_file = add(cp, ce->ce_file);
                        if (rename(file_org, ce->ce_file)) {
-                               adios (ce->ce_file, "unable to rename %s to ",
+                               adios(ce->ce_file, "unable to rename %s to ",
                                                file_org);
                        }
                        free(file_org);
 
                } else {
-                       ce->ce_file = add (cp, ce->ce_file);
+                       ce->ce_file = add(cp, ce->ce_file);
                }
        }
 
-       if ((ce->ce_fp = fopen (ce->ce_file, "w+")) == NULL) {
-               content_error (ce->ce_file, ct,
+       if ((ce->ce_fp = fopen(ce->ce_file, "w+")) == NULL) {
+               content_error(ce->ce_file, ct,
                                "unable to fopen for reading/writing");
                return NOTOK;
        }
@@ -2266,67 +2234,67 @@ open7Bit (CT ct, char **file)
                CI ci = &ct->c_ctinfo;
 
                len = 0;
-               fprintf (ce->ce_fp, "%s: %s/%s", TYPE_FIELD, ci->ci_type,
+               fprintf(ce->ce_fp, "%s: %s/%s", TYPE_FIELD, ci->ci_type,
                                ci->ci_subtype);
-               len += strlen (TYPE_FIELD) + 2 + strlen (ci->ci_type)
-                       + 1 + strlen (ci->ci_subtype);
+               len += strlen(TYPE_FIELD) + 2 + strlen(ci->ci_type) + 1 +
+                               strlen(ci->ci_subtype);
                for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++) {
-                       putc (';', ce->ce_fp);
+                       putc(';', ce->ce_fp);
                        len++;
 
-                       snprintf (buffer, sizeof(buffer), "%s=\"%s\"",
+                       snprintf(buffer, sizeof(buffer), "%s=\"%s\"",
                                        *ap, *ep);
 
-                       if (len + 1 + (cc = strlen (buffer)) >= CPERLIN) {
-                               fputs ("\n\t", ce->ce_fp);
+                       if (len + 1 + (cc = strlen(buffer)) >= CPERLIN) {
+                               fputs("\n\t", ce->ce_fp);
                                len = 8;
                        } else {
-                               putc (' ', ce->ce_fp);
+                               putc(' ', ce->ce_fp);
                                len++;
                        }
-                       fprintf (ce->ce_fp, "%s", buffer);
+                       fprintf(ce->ce_fp, "%s", buffer);
                        len += cc;
                }
 
                if (ci->ci_comment) {
-                       if (len + 1 + (cc = 2 + strlen (ci->ci_comment)) >= CPERLIN) {
-                               fputs ("\n\t", ce->ce_fp);
+                       if (len + 1 + (cc = 2 + strlen(ci->ci_comment))
+                                               >= CPERLIN) {
+                               fputs("\n\t", ce->ce_fp);
                                len = 8;
-                       }
-                       else {
-                               putc (' ', ce->ce_fp);
+                       } else {
+                               putc(' ', ce->ce_fp);
                                len++;
                        }
-                       fprintf (ce->ce_fp, "(%s)", ci->ci_comment);
+                       fprintf(ce->ce_fp, "(%s)", ci->ci_comment);
                        len += cc;
                }
-               fprintf (ce->ce_fp, "\n");
+               fprintf(ce->ce_fp, "\n");
                if (ct->c_id)
-                       fprintf (ce->ce_fp, "%s:%s", ID_FIELD, ct->c_id);
+                       fprintf(ce->ce_fp, "%s:%s", ID_FIELD, ct->c_id);
                if (ct->c_descr)
-                       fprintf (ce->ce_fp, "%s:%s", DESCR_FIELD, ct->c_descr);
+                       fprintf(ce->ce_fp, "%s:%s", DESCR_FIELD, ct->c_descr);
                if (ct->c_dispo)
-                       fprintf (ce->ce_fp, "%s:%s", DISPO_FIELD, ct->c_dispo);
-               fprintf (ce->ce_fp, "\n");
+                       fprintf(ce->ce_fp, "%s:%s", DISPO_FIELD, ct->c_dispo);
+               fprintf(ce->ce_fp, "\n");
        }
 
        if ((len = ct->c_end - ct->c_begin) < 0)
-               adios (NULL, "internal error(3)");
+               adios(NULL, "internal error(3)");
 
-       if (!ct->c_fp && (ct->c_fp = fopen (ct->c_file, "r")) == NULL) {
-               content_error (ct->c_file, ct, "unable to open for reading");
+       if (!ct->c_fp && (ct->c_fp = fopen(ct->c_file, "r")) == NULL) {
+               content_error(ct->c_file, ct, "unable to open for reading");
                return NOTOK;
        }
 
-       lseek (fd = fileno (ct->c_fp), (off_t) ct->c_begin, SEEK_SET);
+       lseek(fd = fileno(ct->c_fp), (off_t) ct->c_begin, SEEK_SET);
        while (len > 0)
-               switch (cc = read (fd, buffer, sizeof(buffer) - 1)) {
+               switch (cc = read(fd, buffer, sizeof(buffer) - 1)) {
                case NOTOK:
-                       content_error (ct->c_file, ct, "error reading from");
+                       content_error(ct->c_file, ct, "error reading from");
                        goto clean_up;
 
                case OK:
-                       content_error (NULL, ct, "premature eof");
+                       content_error(NULL, ct, "premature eof");
                        goto clean_up;
 
                default:
@@ -2334,28 +2302,29 @@ open7Bit (CT ct, char **file)
                                cc = len;
                        len -= cc;
 
-                       fwrite (buffer, sizeof(*buffer), cc, ce->ce_fp);
-                       if (ferror (ce->ce_fp)) {
-                               content_error (ce->ce_file, ct, "error writing to");
+                       fwrite(buffer, sizeof(*buffer), cc, ce->ce_fp);
+                       if (ferror(ce->ce_fp)) {
+                               content_error(ce->ce_file, ct,
+                                               "error writing to");
                                goto clean_up;
                        }
                }
 
-       fseek (ct->c_fp, 0L, SEEK_SET);
+       fseek(ct->c_fp, 0L, SEEK_SET);
 
-       if (fflush (ce->ce_fp)) {
-               content_error (ce->ce_file, ct, "error writing to");
+       if (fflush(ce->ce_fp)) {
+               content_error(ce->ce_file, ct, "error writing to");
                goto clean_up;
        }
 
-       fseek (ce->ce_fp, 0L, SEEK_SET);
+       fseek(ce->ce_fp, 0L, SEEK_SET);
 
 ready_to_go:
        *file = ce->ce_file;
-       return fileno (ce->ce_fp);
+       return fileno(ce->ce_fp);
 
 clean_up:
-       free_encoding (ct, 0);
+       free_encoding(ct, 0);
        return NOTOK;
 }
 
@@ -2365,32 +2334,32 @@ clean_up:
 */
 
 static int
-openExternal (CT ct, CT cb, CE ce, char **file, int *fd)
+openExternal(CT ct, CT cb, CE ce, char **file, int *fd)
 {
        char cachefile[BUFSIZ];
 
        if (ce->ce_fp) {
-               fseek (ce->ce_fp, 0L, SEEK_SET);
+               fseek(ce->ce_fp, 0L, SEEK_SET);
                goto ready_already;
        }
 
        if (ce->ce_file) {
-               if ((ce->ce_fp = fopen (ce->ce_file, "r")) == NULL) {
-                       content_error (ce->ce_file, ct,
+               if ((ce->ce_fp = fopen(ce->ce_file, "r")) == NULL) {
+                       content_error(ce->ce_file, ct,
                                        "unable to fopen for reading");
                        return NOTOK;
                }
                goto ready_already;
        }
 
-       if (find_cache (ct, rcachesw, (int *) 0, cb->c_id,
-                               cachefile, sizeof(cachefile)) != NOTOK) {
-               if ((ce->ce_fp = fopen (cachefile, "r"))) {
-                       ce->ce_file = getcpy (cachefile);
+       if (find_cache(ct, rcachesw, (int *) 0, cb->c_id,
+                       cachefile, sizeof(cachefile)) != NOTOK) {
+               if ((ce->ce_fp = fopen(cachefile, "r"))) {
+                       ce->ce_file = getcpy(cachefile);
                        ce->ce_unlink = 0;
                        goto ready_already;
                } else {
-                       admonish (cachefile, "unable to fopen for reading");
+                       admonish(cachefile, "unable to fopen for reading");
                }
        }
 
@@ -2398,7 +2367,7 @@ openExternal (CT ct, CT cb, CE ce, char **file, int *fd)
 
 ready_already:
        *file = ce->ce_file;
-       *fd = fileno (ce->ce_fp);
+       *fd = fileno(ce->ce_fp);
        return DONE;
 }
 
@@ -2407,77 +2376,78 @@ ready_already:
 */
 
 static int
-InitFile (CT ct)
+InitFile(CT ct)
 {
-       return init_encoding (ct, openFile);
+       return init_encoding(ct, openFile);
 }
 
 
 static int
-openFile (CT ct, char **file)
+openFile(CT ct, char **file)
 {
        int fd, cachetype;
        char cachefile[BUFSIZ];
        struct exbody *e = ct->c_ctexbody;
        CE ce = ct->c_cefile;
 
-       switch (openExternal (e->eb_parent, e->eb_content, ce, file, &fd)) {
-               case NOTOK:
-                       return NOTOK;
-
-               case OK:
-                       break;
-
-               case DONE:
-                       return fd;
+       switch (openExternal(e->eb_parent, e->eb_content, ce, file, &fd)) {
+       case NOTOK:
+               return NOTOK;
+       case OK:
+               break;
+       case DONE:
+               return fd;
        }
 
        if (!e->eb_name) {
-               content_error (NULL, ct, "missing name parameter");
+               content_error(NULL, ct, "missing name parameter");
                return NOTOK;
        }
 
-       ce->ce_file = getcpy (e->eb_name);
+       ce->ce_file = getcpy(e->eb_name);
        ce->ce_unlink = 0;
 
-       if ((ce->ce_fp = fopen (ce->ce_file, "r")) == NULL) {
-               content_error (ce->ce_file, ct, "unable to fopen for reading");
+       if ((ce->ce_fp = fopen(ce->ce_file, "r")) == NULL) {
+               content_error(ce->ce_file, ct, "unable to fopen for reading");
                return NOTOK;
        }
 
-       if ((!e->eb_permission || mh_strcasecmp (e->eb_permission, "read-write"))
-               && find_cache (NULL, wcachesw, &cachetype, e->eb_content->c_id,
-                       cachefile, sizeof(cachefile)) != NOTOK) {
+       if ((!e->eb_permission ||
+                       mh_strcasecmp(e->eb_permission, "read-write")) &&
+                       find_cache(NULL, wcachesw, &cachetype,
+                       e->eb_content->c_id, cachefile, sizeof(cachefile))
+                       != NOTOK) {
                int mask;
                FILE *fp;
 
-               mask = umask (cachetype ? ~m_gmprot () : 0222);
-               if ((fp = fopen (cachefile, "w"))) {
+               mask = umask(cachetype ? ~m_gmprot() : 0222);
+               if ((fp = fopen(cachefile, "w"))) {
                        int cc;
                        char buffer[BUFSIZ];
                        FILE *gp = ce->ce_fp;
 
-                       fseek (gp, 0L, SEEK_SET);
+                       fseek(gp, 0L, SEEK_SET);
 
-                       while ((cc = fread (buffer, sizeof(*buffer), sizeof(buffer), gp)) > 0)
-                               fwrite (buffer, sizeof(*buffer), cc, fp);
-                       fflush (fp);
+                       while ((cc = fread(buffer, sizeof(*buffer),
+                                       sizeof(buffer), gp)) > 0)
+                               fwrite(buffer, sizeof(*buffer), cc, fp);
+                       fflush(fp);
 
-                       if (ferror (gp)) {
-                               admonish (ce->ce_file, "error reading");
-                               unlink (cachefile);
-                       } else if (ferror (fp)) {
-                               admonish (cachefile, "error writing");
-                               unlink (cachefile);
+                       if (ferror(gp)) {
+                               admonish(ce->ce_file, "error reading");
+                               unlink(cachefile);
+                       } else if (ferror(fp)) {
+                               admonish(cachefile, "error writing");
+                               unlink(cachefile);
                        }
-                       fclose (fp);
+                       fclose(fp);
                }
-               umask (mask);
+               umask(mask);
        }
 
-       fseek (ce->ce_fp, 0L, SEEK_SET);
+       fseek(ce->ce_fp, 0L, SEEK_SET);
        *file = ce->ce_file;
-       return fileno (ce->ce_fp);
+       return fileno(ce->ce_fp);
 }
 
 /*
@@ -2485,14 +2455,14 @@ openFile (CT ct, char **file)
 */
 
 static int
-InitFTP (CT ct)
+InitFTP(CT ct)
 {
-       return init_encoding (ct, openFTP);
+       return init_encoding(ct, openFTP);
 }
 
 
 static int
-openFTP (CT ct, char **file)
+openFTP(CT ct, char **file)
 {
        int cachetype, caching, fd;
        int len, buflen;
@@ -2502,39 +2472,37 @@ openFTP (CT ct, char **file)
        CE ce;
        static char *username = NULL;
        static char *password = NULL;
+       int child_id, vecp;
+       char *vec[9];
 
        e  = ct->c_ctexbody;
        ce = ct->c_cefile;
 
-       if ((ftp = context_find (nmhaccessftp)) && !*ftp)
+       if ((ftp = context_find(nmhaccessftp)) && !*ftp)
                ftp = NULL;
 
-#ifndef BUILTIN_FTP
        if (!ftp)
                return NOTOK;
-#endif
-
-       switch (openExternal (e->eb_parent, e->eb_content, ce, file, &fd)) {
-               case NOTOK:
-                       return NOTOK;
-
-               case OK:
-                       break;
 
-               case DONE:
-                       return fd;
+       switch (openExternal(e->eb_parent, e->eb_content, ce, file, &fd)) {
+       case NOTOK:
+               return NOTOK;
+       case OK:
+               break;
+       case DONE:
+               return fd;
        }
 
        if (!e->eb_name || !e->eb_site) {
-               content_error (NULL, ct, "missing %s parameter",
-                       e->eb_name ? "site": "name");
+               content_error(NULL, ct, "missing %s parameter",
+                               e->eb_name ? "site": "name");
                return NOTOK;
        }
 
        if (xpid) {
                if (xpid < 0)
                        xpid = -xpid;
-               pidcheck (pidwait (xpid, NOTOK));
+               pidcheck(pidwait(xpid, NOTOK));
                xpid = 0;
        }
 
@@ -2545,45 +2513,45 @@ openFTP (CT ct, char **file)
        /*
        ** Construct the query message for user
        */
-       snprintf (bp, buflen, "Retrieve %s", e->eb_name);
-       len = strlen (bp);
+       snprintf(bp, buflen, "Retrieve %s", e->eb_name);
+       len = strlen(bp);
        bp += len;
        buflen -= len;
 
        if (e->eb_partno) {
-               snprintf (bp, buflen, " (content %s)", e->eb_partno);
-               len = strlen (bp);
+               snprintf(bp, buflen, " (content %s)", e->eb_partno);
+               len = strlen(bp);
                bp += len;
                buflen -= len;
        }
 
-       snprintf (bp, buflen, "\n    using %sFTP from site %s",
-               e->eb_flags ? "anonymous " : "", e->eb_site);
-       len = strlen (bp);
+       snprintf(bp, buflen, "\n    using %sFTP from site %s",
+                       e->eb_flags ? "anonymous " : "", e->eb_site);
+       len = strlen(bp);
        bp += len;
        buflen -= len;
 
        if (e->eb_size > 0) {
-               snprintf (bp, buflen, " (%lu octets)", e->eb_size);
-               len = strlen (bp);
+               snprintf(bp, buflen, " (%lu octets)", e->eb_size);
+               len = strlen(bp);
                bp += len;
                buflen -= len;
        }
-       snprintf (bp, buflen, "? ");
+       snprintf(bp, buflen, "? ");
 
        /*
        ** Now, check the answer
        */
-       if (!getanswer (buffer))
+       if (!getanswer(buffer))
                return NOTOK;
 
        if (e->eb_flags) {
                user = "anonymous";
-               snprintf (buffer, sizeof(buffer), "%s@%s", getusername (),
-                               LocalName ());
+               snprintf(buffer, sizeof(buffer), "%s@%s", getusername(),
+                               LocalName());
                pass = buffer;
        } else {
-               ruserpass (e->eb_site, &username, &password);
+               ruserpass(e->eb_site, &username, &password);
                user = username;
                pass = password;
        }
@@ -2591,9 +2559,11 @@ openFTP (CT ct, char **file)
        ce->ce_unlink = (*file == NULL);
        caching = 0;
        cachefile[0] = '\0';
-       if ((!e->eb_permission || mh_strcasecmp (e->eb_permission, "read-write"))
-                       && find_cache (NULL, wcachesw, &cachetype, e->eb_content->c_id,
-                       cachefile, sizeof(cachefile)) != NOTOK) {
+       if ((!e->eb_permission ||
+                       mh_strcasecmp(e->eb_permission, "read-write")) &&
+                       find_cache(NULL, wcachesw, &cachetype,
+                       e->eb_content->c_id, cachefile, sizeof(cachefile))
+                       != NOTOK) {
                if (*file == NULL) {
                        ce->ce_unlink = 0;
                        caching = 1;
@@ -2601,106 +2571,90 @@ openFTP (CT ct, char **file)
        }
 
        if (*file)
-               ce->ce_file = add (*file, NULL);
+               ce->ce_file = getcpy(*file);
        else if (caching)
-               ce->ce_file = add (cachefile, NULL);
+               ce->ce_file = getcpy(cachefile);
        else
-               ce->ce_file = add (m_mktemp(tmp, NULL, NULL), NULL);
+               ce->ce_file = getcpy(m_mktemp(tmp, NULL, NULL));
 
-       if ((ce->ce_fp = fopen (ce->ce_file, "w+")) == NULL) {
-               content_error (ce->ce_file, ct, "unable to fopen for reading/writing");
+       if ((ce->ce_fp = fopen(ce->ce_file, "w+")) == NULL) {
+               content_error (ce->ce_file, ct,
+                               "unable to fopen for reading/writing");
                return NOTOK;
        }
 
-#ifdef BUILTIN_FTP
-       if (ftp)
-#endif
-       {
-               int child_id, i, vecp;
-               char *vec[9];
-
-               vecp = 0;
-               vec[vecp++] = r1bindex (ftp, '/');
-               vec[vecp++] = e->eb_site;
-               vec[vecp++] = user;
-               vec[vecp++] = pass;
-               vec[vecp++] = e->eb_dir;
-               vec[vecp++] = e->eb_name;
-               vec[vecp++] = ce->ce_file,
-               vec[vecp++] = e->eb_mode && !mh_strcasecmp (e->eb_mode, "ascii")
-                       ? "ascii" : "binary";
-               vec[vecp] = NULL;
-
-               fflush (stdout);
-
-               for (i = 0; (child_id = vfork ()) == NOTOK && i < 5; i++)
-                       sleep (5);
-               switch (child_id) {
-                       case NOTOK:
-                               adios ("fork", "unable to");
-                               /* NOTREACHED */
-
-                       case OK:
-                               close (fileno (ce->ce_fp));
-                               execvp (ftp, vec);
-                               fprintf (stderr, "unable to exec ");
-                               perror (ftp);
-                               _exit (-1);
-                               /* NOTREACHED */
-
-                       default:
-                               if (pidXwait (child_id, NULL)) {
-#ifdef BUILTIN_FTP
-losing_ftp:
-#endif
-                                       username = password = NULL;
-                                       ce->ce_unlink = 1;
-                                       return NOTOK;
-                               }
-                               break;
+       vecp = 0;
+       vec[vecp++] = mhbasename(ftp);
+       vec[vecp++] = e->eb_site;
+       vec[vecp++] = user;
+       vec[vecp++] = pass;
+       vec[vecp++] = e->eb_dir;
+       vec[vecp++] = e->eb_name;
+       vec[vecp++] = ce->ce_file,
+       vec[vecp++] = e->eb_mode &&
+                       !mh_strcasecmp(e->eb_mode, "ascii") ?
+                       "ascii" : "binary";
+       vec[vecp] = NULL;
+
+       fflush(stdout);
+
+       switch (child_id = fork()) {
+       case NOTOK:
+               adios("fork", "unable to");
+               /* NOTREACHED */
+
+       case OK:
+               close(fileno(ce->ce_fp));
+               execvp(ftp, vec);
+               fprintf(stderr, "unable to exec ");
+               perror(ftp);
+               _exit(-1);
+               /* NOTREACHED */
+
+       default:
+               if (pidXwait(child_id, NULL)) {
+                       username = password = NULL;
+                       ce->ce_unlink = 1;
+                       return NOTOK;
                }
+               break;
        }
-#ifdef BUILTIN_FTP
-       else if (ftp_get (e->eb_site, user, pass, e->eb_dir, e->eb_name,
-                       ce->ce_file, e->eb_mode && !mh_strcasecmp (e->eb_mode, "ascii"), 0)
-                       == NOTOK)
-                       goto losing_ftp;
-#endif
 
        if (cachefile[0]) {
                if (caching)
-                       chmod (cachefile, cachetype ? m_gmprot () : 0444);
+                       chmod(cachefile, cachetype ? m_gmprot() : 0444);
                else {
                        int mask;
                        FILE *fp;
 
-                       mask = umask (cachetype ? ~m_gmprot () : 0222);
-                       if ((fp = fopen (cachefile, "w"))) {
+                       mask = umask(cachetype ? ~m_gmprot() : 0222);
+                       if ((fp = fopen(cachefile, "w"))) {
                                int cc;
                                FILE *gp = ce->ce_fp;
 
-                               fseek (gp, 0L, SEEK_SET);
+                               fseek(gp, 0L, SEEK_SET);
 
-                               while ((cc= fread (buffer, sizeof(*buffer), sizeof(buffer), gp)) > 0)
-                                       fwrite (buffer, sizeof(*buffer), cc, fp);
-                               fflush (fp);
+                               while ((cc= fread(buffer, sizeof(*buffer),
+                                               sizeof(buffer), gp)) > 0)
+                                       fwrite(buffer, sizeof(*buffer), cc, fp);
+                               fflush(fp);
 
-                               if (ferror (gp)) {
-                                       admonish (ce->ce_file, "error reading");
-                                       unlink (cachefile);
-                               } else if (ferror (fp)) {
-                                       admonish (cachefile, "error writing");
-                                       unlink (cachefile);
+                               if (ferror(gp)) {
+                                       admonish(ce->ce_file, "error reading");
+                                       unlink(cachefile);
+                               } else if (ferror(fp)) {
+                                       admonish(cachefile, "error writing");
+                                       unlink(cachefile);
                                }
-                               fclose (fp);
+                               fclose(fp);
                        }
-                       umask (mask);
+                       umask(mask);
                }
        }
 
-       fseek (ce->ce_fp, 0L, SEEK_SET);
+       fseek(ce->ce_fp, 0L, SEEK_SET);
        *file = ce->ce_file;
-       return fileno (ce->ce_fp);
+       return fileno(ce->ce_fp);
 }
 
 
@@ -2709,41 +2663,39 @@ losing_ftp:
 */
 
 static int
-InitMail (CT ct)
+InitMail(CT ct)
 {
-       return init_encoding (ct, openMail);
+       return init_encoding(ct, openMail);
 }
 
 
 static int
-openMail (CT ct, char **file)
+openMail(CT ct, char **file)
 {
-       int child_id, fd, i, vecp;
+       int child_id, fd, vecp;
        int len, buflen;
        char *bp, buffer[BUFSIZ], *vec[7];
        struct exbody *e = ct->c_ctexbody;
        CE ce = ct->c_cefile;
 
-       switch (openExternal (e->eb_parent, e->eb_content, ce, file, &fd)) {
-               case NOTOK:
-                       return NOTOK;
-
-               case OK:
-                       break;
-
-               case DONE:
-                       return fd;
+       switch (openExternal(e->eb_parent, e->eb_content, ce, file, &fd)) {
+       case NOTOK:
+               return NOTOK;
+       case OK:
+               break;
+       case DONE:
+               return fd;
        }
 
        if (!e->eb_server) {
-               content_error (NULL, ct, "missing server parameter");
+               content_error(NULL, ct, "missing server parameter");
                return NOTOK;
        }
 
        if (xpid) {
                if (xpid < 0)
                        xpid = -xpid;
-               pidcheck (pidwait (xpid, NOTOK));
+               pidcheck(pidwait(xpid, NOTOK));
                xpid = 0;
        }
 
@@ -2752,27 +2704,27 @@ openMail (CT ct, char **file)
        buflen = sizeof(buffer);
 
        /* Now, construct query message */
-       snprintf (bp, buflen, "Retrieve content");
-       len = strlen (bp);
+       snprintf(bp, buflen, "Retrieve content");
+       len = strlen(bp);
        bp += len;
        buflen -= len;
 
        if (e->eb_partno) {
-               snprintf (bp, buflen, " %s", e->eb_partno);
-               len = strlen (bp);
+               snprintf(bp, buflen, " %s", e->eb_partno);
+               len = strlen(bp);
                bp += len;
                buflen -= len;
        }
 
-       snprintf (bp, buflen, " by asking %s\n\n%s\n? ", e->eb_server,
-               e->eb_subject ? e->eb_subject : e->eb_body);
+       snprintf(bp, buflen, " by asking %s\n\n%s\n? ", e->eb_server,
+                       e->eb_subject ? e->eb_subject : e->eb_body);
 
        /* Now, check answer */
-       if (!getanswer (buffer))
+       if (!getanswer(buffer))
                return NOTOK;
 
        vecp = 0;
-       vec[vecp++] = r1bindex (mailproc, '/');
+       vec[vecp++] = "mhmail";
        vec[vecp++] = e->eb_server;
        vec[vecp++] = "-subject";
        vec[vecp++] = e->eb_subject ? e->eb_subject : "mail-server request";
@@ -2780,36 +2732,35 @@ openMail (CT ct, char **file)
        vec[vecp++] = e->eb_body;
        vec[vecp] = NULL;
 
-       for (i = 0; (child_id = vfork ()) == NOTOK && i < 5; i++)
-               sleep (5);
-       switch (child_id) {
-               case NOTOK:
-                       advise ("fork", "unable to");
-                       return NOTOK;
+       switch (child_id = fork()) {
+       case NOTOK:
+               advise("fork", "unable to");
+               return NOTOK;
 
-               case OK:
-                       execvp (mailproc, vec);
-                       fprintf (stderr, "unable to exec ");
-                       perror (mailproc);
-                       _exit (-1);
-                       /* NOTREACHED */
+       case OK:
+               execvp(*vec, vec);
+               fprintf(stderr, "unable to exec ");
+               perror(*vec);
+               _exit(-1);
+               /* NOTREACHED */
 
-               default:
-                       if (pidXwait (child_id, NULL) == OK)
-                               advise (NULL, "request sent");
-                       break;
+       default:
+               if (pidXwait(child_id, NULL) == OK)
+                       advise(NULL, "request sent");
+               break;
        }
 
        if (*file == NULL) {
-               ce->ce_file = add (m_mktemp(tmp, NULL, NULL), NULL);
+               ce->ce_file = getcpy(m_mktemp(tmp, NULL, NULL));
                ce->ce_unlink = 1;
        } else {
-               ce->ce_file = add (*file, NULL);
+               ce->ce_file = getcpy(*file);
                ce->ce_unlink = 0;
        }
 
-       if ((ce->ce_fp = fopen (ce->ce_file, "w+")) == NULL) {
-               content_error (ce->ce_file, ct, "unable to fopen for reading/writing");
+       if ((ce->ce_fp = fopen(ce->ce_file, "w+")) == NULL) {
+               content_error(ce->ce_file, ct,
+                               "unable to fopen for reading/writing");
                return NOTOK;
        }
 
@@ -2818,17 +2769,17 @@ openMail (CT ct, char **file)
        ** prints it, too.
        */
        if (ct->c_showproc)
-               free (ct->c_showproc);
-       ct->c_showproc = add ("true", NULL);
+               free(ct->c_showproc);
+       ct->c_showproc = getcpy("true");
 
-       fseek (ce->ce_fp, 0L, SEEK_SET);
+       fseek(ce->ce_fp, 0L, SEEK_SET);
        *file = ce->ce_file;
-       return fileno (ce->ce_fp);
+       return fileno(ce->ce_fp);
 }
 
 
 static int
-readDigest (CT ct, char *cp)
+readDigest(CT ct, char *cp)
 {
        int bitno, skip;
        unsigned long bits;
@@ -2847,39 +2798,40 @@ readDigest (CT ct, char *cp)
        for (ep = (dp = ct->c_digest)
                + sizeof(ct->c_digest) / sizeof(ct->c_digest[0]); *cp; cp++)
                switch (*cp) {
-                       default:
-                               if (skip || (*cp & 0x80)
-                                       || (value = b642nib[*cp & 0x7f]) > 0x3f) {
-                                       if (debugsw)
-                                               fprintf (stderr, "invalid BASE64 encoding\n");
-                                       return NOTOK;
-                               }
+               default:
+                       if (skip || (*cp & 0x80) ||
+                                       (value = b642nib[*cp & 0x7f])
+                                       > 0x3f) {
+                               if (debugsw)
+                                       fprintf(stderr, "invalid BASE64 encoding\n");
+                               return NOTOK;
+                       }
 
-                               bits |= value << bitno;
+                       bits |= value << bitno;
 test_end:
-                               if ((bitno -= 6) < 0) {
-                                       if (dp + (3 - skip) > ep)
-                                               goto invalid_digest;
-                                       *dp++ = *b1;
-                                       if (skip < 2) {
-                                               *dp++ = *b2;
-                                               if (skip < 1)
-                                                       *dp++ = *b3;
-                                       }
-                                       bitno = 18;
-                                       bits = 0L;
-                                       skip = 0;
+                       if ((bitno -= 6) < 0) {
+                               if (dp + (3 - skip) > ep)
+                                       goto invalid_digest;
+                               *dp++ = *b1;
+                               if (skip < 2) {
+                                       *dp++ = *b2;
+                                       if (skip < 1)
+                                               *dp++ = *b3;
                                }
-                               break;
+                               bitno = 18;
+                               bits = 0L;
+                               skip = 0;
+                       }
+                       break;
 
-                       case '=':
-                               if (++skip > 3)
-                                       goto self_delimiting;
-                               goto test_end;
+               case '=':
+                       if (++skip > 3)
+                               goto self_delimiting;
+                       goto test_end;
                }
        if (bitno != 18) {
                if (debugsw)
-                       fprintf (stderr, "premature ending (bitno %d)\n",
+                       fprintf(stderr, "premature ending (bitno %d)\n",
                                        bitno);
 
                return NOTOK;
@@ -2890,7 +2842,7 @@ invalid_digest:
                if (debugsw) {
                        while (*cp)
                                cp++;
-                       fprintf (stderr, "invalid MD5 digest (got %d octets)\n",
+                       fprintf(stderr, "invalid MD5 digest (got %d octets)\n",
                                (int)(cp - bp));
                }
 
@@ -2898,10 +2850,10 @@ invalid_digest:
        }
 
        if (debugsw) {
-               fprintf (stderr, "MD5 digest=");
+               fprintf(stderr, "MD5 digest=");
                for (dp = ct->c_digest; dp < ep; dp++)
-                       fprintf (stderr, "%02x", *dp & 0xff);
-               fprintf (stderr, "\n");
+                       fprintf(stderr, "%02x", *dp & 0xff);
+               fprintf(stderr, "\n");
        }
 
        return OK;