[dovecot-cvs] dovecot/src/imap imap-fetch-body-section.c,1.24,1.25

[cras at procontrol.fi]

Update of /home/cvs/dovecot/src/imap
In directory talvi:/tmp/cvs-serv909

Modified Files:
	imap-fetch-body-section.c 
Log Message:
Add CRLF to end of FETCH BODY[HEADER.FIELDS (...)] always.



Index: imap-fetch-body-section.c
===================================================================
RCS file: /home/cvs/dovecot/src/imap/imap-fetch-body-section.c,v
retrieving revision 1.24
retrieving revision 1.25
diff -u -d -r1.24 -r1.25
--- imap-fetch-body-section.c	27 Apr 2004 20:25:52 -0000	1.24
+++ imap-fetch-body-section.c	2 May 2004 13:05:53 -0000	1.25
@@ -293,13 +293,11 @@
 
 	hdr_ctx = message_parse_header_init(input, NULL);
 	while ((hdr = message_parse_header_next(hdr_ctx)) != NULL) {
-		/* see if we want this field.
-		   we always want the end-of-headers line */
-		if (!hdr->eoh &&
-		    !ctx->match_func(ctx->fields, hdr->name, hdr->name_len))
+		/* see if we want this field. */
+		if (!ctx->match_func(ctx->fields, hdr->name, hdr->name_len))
 			continue;
 
-		if (!hdr->continued && !hdr->eoh) {
+		if (!hdr->continued) {
 			if (!fetch_header_append(ctx, hdr->name, hdr->name_len))
 				break;
 			if (!fetch_header_append(ctx, ": ", 2))
@@ -312,6 +310,16 @@
 				break;
 		}
 	}
+
+	/* FIXME: We'll just always add the end of headers mark now.
+	   mail-storage should rather include it in the header stream..
+	   however, not much of a problem since all non-broken mails have it.
+
+	   Also, Netscape 4.x seems to require this or it won't show the
+	   mail.. So if we do make this as RFC says, we'll need to add
+	   netscape-workaround. */
+	(void)fetch_header_append(ctx, "\r\n", 2);
+
 	message_parse_header_deinit(hdr_ctx);
 
 	i_assert(ctx->dest_size <= ctx->max_size);