X-Git-Url: http://git.marmaro.de/?p=mmh;a=blobdiff_plain;f=uip%2Fspost.c;h=39d8ada53dfb99f8b35888ac62d290e53d7ed978;hp=4a24fc4b74989228e0b6c3236eae866e87387701;hb=6e9577f324bef90765a5edc02044eb111ec48072;hpb=27f39f1e62375fdd22f4cecab4883b0ad764b361

diff --git a/uip/spost.c b/uip/spost.c
index 4a24fc4..39d8ada 100644
--- a/uip/spost.c
+++ b/uip/spost.c
@@ -39,6 +39,7 @@ static struct swit switches[] = {
 	{ NULL, 0 }
 };
 
+char *version=VERSION;
 
 /* flags for headers->flags */
 #define HNOP  0x0000  /* just used to keep .set around */
@@ -144,9 +145,11 @@ static size_t do_aliasing(struct mailname *, struct mailname **);
 int
 main(int argc, char **argv)
 {
-	int state, compnum;
+	enum state state;
+	struct field f = {{0}};
+	int compnum;
 	char *cp, *msg = NULL, **argp, **arguments;
-	char **sargv, buf[BUFSIZ], name[NAMESZ];
+	char **sargv, buf[BUFSIZ];
 	FILE *in;
 
 	setlocale(LC_ALL, "");
@@ -211,7 +214,7 @@ main(int argc, char **argv)
 		verbose++;
 		out = stdout;
 	} else {
-		tmpfil = getcpy(m_mktemp2("/tmp/", invo_name, NULL, &out));
+		tmpfil = mh_xstrdup(m_mktemp2("/tmp/", invo_name, NULL, &out));
 	}
 
 	/* check for "Aliasfile:" profile entry */
@@ -219,7 +222,7 @@ main(int argc, char **argv)
 		char *dp, **ap; 
 
 		aliasflg = 1;
-		for (ap=brkstring(dp=getcpy(cp), " ", "\n"); ap && *ap;
+		for (ap=brkstring(dp=mh_xstrdup(cp), " ", "\n"); ap && *ap;
 				ap++) {
 			if ((state = alias(etcpath(*ap))) != AK_OK) {
 				adios(EX_IOERR, NULL, "aliasing error in file %s: %s",
@@ -231,37 +234,31 @@ main(int argc, char **argv)
 
 	hdrtab = (msgstate == normal) ? NHeaders : RHeaders;
 
-	for (compnum = 1, state = FLD;;) {
-		switch (state = m_getfld(state, name, buf, sizeof(buf), in)) {
-		case FLD:
-		case FLDPLUS:
+	for (compnum = 1, state = FLD2;;) {
+		switch (state = m_getfld2(state, &f, in)) {
+		case FLD2:
 			compnum++;
-			cp = getcpy(buf);
-			while (state == FLDPLUS) {
-				state = m_getfld(state, name, buf,
-						sizeof(buf), in);
-				cp = add(buf, cp);
-			}
-			putfmt(name, cp, out);
-			free(cp);
+			putfmt(f.name, f.value, out);
 			continue;
 
-		case BODY:
+		case BODY2:
 			finish_headers(out);
-			fprintf(out, "\n%s", buf);
-			while (state == BODY) {
-				state = m_getfld(state, name, buf,
-						sizeof(buf), in);
-				fputs(buf, out);
+			fprintf(out, "\n%s", f.value);
+			while ((state = m_getfld2(state, &f, in)) == BODY2) {
+				if (f.valuelen >= NAMESZ) {
+					adios(EX_DATAERR, NULL, "Body contains a to long line");
+				}
+				fputs(f.value, out);
 			}
 			break;
 
-		case FILEEOF:
+		case FILEEOF2:
 			finish_headers(out);
 			break;
 
-		case LENERR:
-		case FMTERR:
+		case LENERR2:
+		case FMTERR2:
+		case IOERR2:
 			adios(EX_DATAERR, NULL, "message format error in component #%d",
 					compnum);
 
@@ -272,6 +269,10 @@ main(int argc, char **argv)
 	}
 	fclose(in);
 
+	if (state != FILEEOF2) {
+		adios(EX_IOERR, "m_getfld2", "Error while reading body");
+	}
+
 	if (debug) {
 		struct mailname *i = recipients;
 		/* stop here */
@@ -319,7 +320,7 @@ main(int argc, char **argv)
 		adios(EX_DATAERR, NULL, "message has no recipients");
 	}
 
-	sargv = mh_xmalloc(sizeof(char **) * (recipientsc + 4));
+	sargv = mh_xcalloc(recipientsc + 4, sizeof(char **));
 
 	argp = sargv;
 	*argp++ = "send-mail";
@@ -329,7 +330,7 @@ main(int argc, char **argv)
 	}
 
 	while (recipients != NULL) {
-		cp = getcpy(recipients->m_mbox);
+		cp = mh_xstrdup(recipients->m_mbox);
 		if (recipients->m_host) {
 			cp = add("@", cp);
 			cp = add(recipients->m_host, cp);
@@ -405,7 +406,7 @@ putfmt(char *name, char *str, FILE *out)
 	}
 
 	if (hdr->flags & HSUB) {
-		subject = getcpy(str);
+		subject = mh_xstrdup(str);
 	}
 
 	if (!(hdr->flags & HADR)) {
@@ -551,10 +552,10 @@ putadr(char *name, struct mailname *nl)
 		}
 		if (mp->m_ingrp) {
 			if (mp->m_gname != NULL) {
-				cp = getcpy(mp->m_gname);
+				cp = mh_xstrdup(mp->m_gname);
 				cp = add(";", cp);
 				linepos = putone(cp, linepos, namelen);
-				free(cp);
+				mh_free0(&cp);
 				cp = NULL;
 			}
 		} else {
@@ -650,7 +651,7 @@ fcc(char *file, char *folders)
 		fprintf(stderr, "Skipped %sFcc %s: unable to system().\n",
 				msgstate == resent ? "Resent-" : "", folders);
 	} else if (status != 0) {
-		fprintf(stderr, "%sFcc %s: Problems occured.\n",
+		fprintf(stderr, "%sFcc %s: Problems occurred.\n",
 				msgstate == resent ? "Resent-" : "", folders);
 	}
 }
@@ -666,7 +667,7 @@ process_bccs(char *origmsg)
 	FILE *out = NULL;
 
 	for (mp=bccs; mp; mp=mp->m_next) {
-		bccdraft = getcpy(m_mktemp2("/tmp/", invo_name, NULL, &out));
+		bccdraft = mh_xstrdup(m_mktemp2("/tmp/", invo_name, NULL, &out));
 		fprintf(out, "To: %s\n", mp->m_text);
 		if (from) {
 			fprintf(out, "From: %s\n", from->m_text);