X-Git-Url: http://git.marmaro.de/?a=blobdiff_plain;f=sbr%2Fm_getfld2.c;h=a2b8ae9a4049fa50f7df81311fff78a28a40b54e;hb=d0d56900365f8088ac1fcfea53183ba2f34c705f;hp=6d96bdd42c38c5985f1e93ca46c3753c1c80ce82;hpb=c4de1023b3141b9a89038012b246fa74205752cd;p=mmh

diff --git a/sbr/m_getfld2.c b/sbr/m_getfld2.c
index 6d96bdd..a2b8ae9 100644
--- a/sbr/m_getfld2.c
+++ b/sbr/m_getfld2.c
@@ -10,6 +10,7 @@
 #include <h/utils.h>
 #include <stdio.h>
 #include <ctype.h>
+#include <stdbool.h>
 
 
 enum threestate {
@@ -23,6 +24,7 @@ enum threestate {
 */
 static enum threestate is_falted(FILE *);
 static size_t copyname(char *, char *);
+static bool is_separator(char *);
 
 
 /*
@@ -47,29 +49,29 @@ m_getfld2(enum state s, struct field *f, FILE *msg)
 	ssize_t nchars;
 	enum threestate falted;
 
-	nchars = getline(&tmpline, &len, msg);
-	if (nchars == -1) {
-		if (feof(msg)) {
-			return FILEEOF2;
-		} else {
-			return IOERR2;
+	switch (s) {
+	case FLD2:
+		nchars = getline(&tmpline, &len, msg);
+		if (nchars < 1) {
+			if (feof(msg)) {
+				return FILEEOF2;
+			} else {
+				return IOERR2;
+			}
 		}
-	}
 
-	*f->name = '\0';
-	f->namelen = 0;
+		*f->name = '\0';
+		f->namelen = 0;
 
-	if (nchars >= NAMESZ) {
-		if (f->value) {
-			free(f->value);
+		if (nchars >= NAMESZ) {
+			if (f->value) {
+				free(f->value);
+			}
+			f->value = tmpline;
+			f->valuelen = nchars;
+			return LENERR2;
 		}
-		f->value = tmpline;
-		f->valuelen = nchars;
-		return LENERR2;
-	}
 
-	switch (s) {
-	case FLD2:
 		if (*(tmpline + nchars - 1) != '\n') {
 			if (f->value) {
 				free(f->value);
@@ -80,7 +82,7 @@ m_getfld2(enum state s, struct field *f, FILE *msg)
 			return FMTERR2;
 		}
 
-		if (nchars > 0 && (*tmpline == '\n' || *tmpline == '-')) {
+		if (is_separator(tmpline)) {
 			/* header/body separator found */
 			free(tmpline);
 			return m_getfld2(BODY2, f, msg);
@@ -150,6 +152,26 @@ m_getfld2(enum state s, struct field *f, FILE *msg)
 		return FLD2;
 
 	case BODY2:
+		nchars = getline(&tmpline, &len, msg);
+		if (nchars < 1) {
+			if (feof(msg)) {
+				return FILEEOF2;
+			} else {
+				return IOERR2;
+			}
+		}
+
+		*f->name = '\0';
+		f->namelen = 0;
+
+		if (nchars >= NAMESZ) {
+			if (f->value) {
+				free(f->value);
+			}
+			f->value = tmpline;
+			f->valuelen = nchars;
+			return LENERR2;
+		}
 		if (f->value) {
 			free(f->value);
 		}
@@ -215,3 +237,19 @@ copyname(char *dst, char *src)
 
 	return strlen(dst);
 }
+
+static bool
+is_separator(char *line)
+{
+	/*
+	** In MH, lines that are consists of dashes only are
+	** separators as well ... not so in RFC 822.
+	*/
+	while (*line == '-') {
+		line++;
+	}
+	if (strcmp("\n", line) == 0 || strcmp("\r\n", line) == 0 ) {
+		return true;
+	}
+	return false;
+}