X-Git-Url: https://diplodocus.org/git/nmh/blobdiff_plain/4ea717b986c4b0ce9ccd01687c7b50bd2ab227ca..1e2eede170a7cc4ac86aa78f688b246c407664f9:/uip/mhstoresbr.c

diff --git a/uip/mhstoresbr.c b/uip/mhstoresbr.c
index fab16800..156411b7 100644
--- a/uip/mhstoresbr.c
+++ b/uip/mhstoresbr.c
@@ -135,13 +135,11 @@ store_switch (CT ct)
     switch (ct->c_type) {
 	case CT_MULTIPART:
 	    return store_multi (ct);
-	    break;
 
 	case CT_MESSAGE:
 	    switch (ct->c_subtype) {
 		case MESSAGE_PARTIAL:
 		    return store_partial (ct);
-		    break;
 
 		case MESSAGE_EXTERNAL:
 		    return store_external (ct);
@@ -149,24 +147,19 @@ store_switch (CT ct)
 		case MESSAGE_RFC822:
 		default:
 		    return store_generic (ct);
-		    break;
 	    }
-	    break;
 
 	case CT_APPLICATION:
 	    return store_application (ct);
-	    break;
 
 	case CT_TEXT:
 	case CT_AUDIO:
 	case CT_IMAGE:
 	case CT_VIDEO:
 	    return store_generic (ct);
-	    break;
 
 	default:
 	    adios (NULL, "unknown content type %d", ct->c_type);
-	    break;
     }
 
     return OK;	/* NOT REACHED */
@@ -200,7 +193,6 @@ store_generic (CT ct)
 static int
 store_application (CT ct)
 {
-    char **ap, **ep;
     CI ci = &ct->c_ctinfo;
 
     /* Check if the content specifies a filename */
@@ -214,28 +206,23 @@ store_application (CT ct)
      */
     if (!ct->c_storeproc && ct->c_subtype == APPLICATION_OCTETS) {
 	int tarP = 0, zP = 0, gzP = 0;
+	char *cp;
 
-	for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++) {
-	    /* check for "type=tar" attribute */
-	    if (!strcasecmp (*ap, "type")) {
-		if (strcasecmp (*ep, "tar"))
-		    break;
-
+	if ((cp = get_param(ci->ci_first_pm, "type", ' ', 1))) {
+	    if (strcasecmp (cp, "tar") == 0)
 		tarP = 1;
-		continue;
-	    }
+	}
 
-	    /* check for "conversions=compress" attribute */
-	    if ((!strcasecmp (*ap, "conversions") || !strcasecmp (*ap, "x-conversions"))
-		&& (!strcasecmp (*ep, "compress") || !strcasecmp (*ep, "x-compress"))) {
+	/* check for "conversions=compress" attribute */
+	if ((cp = get_param(ci->ci_first_pm, "conversions", ' ', 1)) ||
+	    (cp = get_param(ci->ci_first_pm, "x-conversions", ' ', 1))) {
+	    if (strcasecmp (cp, "compress") == 0 ||
+		    strcasecmp (cp, "x-compress") == 0) {
 		zP = 1;
-		continue;
 	    }
-	    /* check for "conversions=gzip" attribute */
-	    if ((!strcasecmp (*ap, "conversions") || !strcasecmp (*ap, "x-conversions"))
-		&& (!strcasecmp (*ep, "gzip") || !strcasecmp (*ep, "x-gzip"))) {
+	    if (strcasecmp (cp, "gzip") == 0 ||
+		    strcasecmp (cp, "x-gzip") == 0) {
 		gzP = 1;
-		continue;
 	    }
 	}
 
@@ -569,7 +556,10 @@ store_content (CT ct, CT p)
 	    char *tmpfilenam, *folder;
 
 	    /* Store content in temporary file for now */
-	    tmpfilenam = m_mktemp(invo_name, NULL, NULL);
+	    if ((tmpfilenam = m_mktemp(invo_name, NULL, NULL)) == NULL) {
+		adios(NULL, "unable to create temporary file in %s",
+		      get_temp_dir());
+	    }
 	    ct->c_storage = add (tmpfilenam, NULL);
 
 	    /* Get the folder name */
@@ -607,7 +597,13 @@ store_content (CT ct, CT p)
 	if ((ct->c_storage = clobber_check (add (buffer, NULL))) == NULL) {
 	    return NOTOK;
 	}
-    } /* else output filename was explicitly specified, so use it */
+    } else {
+        /* The output filename was explicitly specified, so use it. */
+	if ((ct->c_storage = clobber_check (add (ct->c_storage, NULL))) ==
+            NULL) {
+	    return NOTOK;
+	}
+    }
 
 got_filename:
     /* flush the output stream */
@@ -624,7 +620,7 @@ got_filename:
      */
     if (ct->c_folder && (!is_partial || last_partial)) {
 	msgnum = output_content_folder (ct->c_folder, ct->c_storage);
-	unlink (ct->c_storage);
+	(void) m_unlink (ct->c_storage);
 	if (msgnum == NOTOK)
 	    return NOTOK;
     }
@@ -959,12 +955,12 @@ parse_format_string (CT ct, char *cp, char *buffer, int buflen, char *dir)
 			buflen--;
 			continue;
 		    } else {
-			char **ap, **ep;
+			PM pm;
 			char *s = "";
 
-			for (ap = ci->ci_attrs, ep = ci->ci_values;
-			         *ap; ap++, ep++) {
-			    snprintf (bp, buflen, "%s%s=\"%s\"", s, *ap, *ep);
+			for (pm = ci->ci_first_pm; pm; pm = pm->pm_next) {
+			    snprintf (bp, buflen, "%s%s=\"%s\"", s,
+				      pm->pm_name, get_param_value(pm, '?'));
 			    len = strlen (bp);
 			    bp += len;
 			    buflen -= len;
@@ -1036,7 +1032,7 @@ raw:
 static void
 get_storeproc (CT ct)
 {
-    char **ap, **ep, *cp;
+    char *cp;
     CI ci;
 
     /*
@@ -1052,29 +1048,18 @@ get_storeproc (CT ct)
      * use that (RFC-2183).
      */
     if (ct->c_dispo) {
-	char *cp = strchr (ct->c_dispo, ';');
-	CI ci = calloc (1, sizeof *ci);
-	int status;
-	int found_filename = 0;
-
-	if (cp  &&  parse_header_attrs (ct->c_file, strlen (invo_name) + 2, &cp,
-                                        ci, &status) == OK) {
-	    for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++) {
-		if (! strcasecmp (*ap, "filename")
-		    && *(cp = *ep) != '/'
-		    && *cp != '.'
-		    && *cp != '|'
-		    && *cp != '!'
-		    && !strchr (cp, '%')) {
-		    ct->c_storeproc = add (cp, NULL);
-		    found_filename = 1;
-		}
-		free (*ap);
-	    }
+	if ((cp = get_param(ct->c_dispo_first, "filename", '_', 0))
+		&& *cp != '/'
+		&& *cp != '.'
+		&& *cp != '|'
+		&& *cp != '!'
+		&& !strchr (cp, '%')) {
+		ct->c_storeproc = add (cp, NULL);
+		free(cp);
+		return;
 	}
-
-	free (ci);
-	if (found_filename) return;
+	if (cp)
+	    free(cp);
     }
 
     /*
@@ -1083,17 +1068,17 @@ get_storeproc (CT ct)
      * the storeproc.
      */
     ci = &ct->c_ctinfo;
-    for (ap = ci->ci_attrs, ep = ci->ci_values; *ap; ap++, ep++) {
-	if (! strcasecmp (*ap, "name")
-	    && *(cp = *ep) != '/'
-	    && *cp != '.'
-	    && *cp != '|'
-	    && *cp != '!'
-	    && !strchr (cp, '%')) {
+    if ((cp = get_param(ci->ci_first_pm, "name", '_', 0))
+	  && *cp != '/'
+	  && *cp != '.'
+	  && *cp != '|'
+	  && *cp != '!'
+	  && !strchr (cp, '%')) {
 	    ct->c_storeproc = add (cp, NULL);
-	    return;
-	}
+
     }
+    if (cp)
+	free(cp);
 }