From: Art Cancro <ajc@citadel.org>
Date: Wed, 14 May 2008 14:59:22 +0000 (+0000)
Subject: Minor cleanup of comments
X-Git-Tag: v7.86~2259
X-Git-Url: https://code.citadel.org/?a=commitdiff_plain;ds=sidebyside;h=336cc142792b95a1dab71277cb93ad722d4835ab;p=citadel.git

Minor cleanup of comments
---

diff --git a/libcitadel/lib/mime_parser.c b/libcitadel/lib/mime_parser.c
index f53294c81..45a6fc700 100644
--- a/libcitadel/lib/mime_parser.c
+++ b/libcitadel/lib/mime_parser.c
@@ -493,22 +493,15 @@ void the_mime_parser(char *partnum,
 		}
 		part_end = content_end;
 
-		/******
-		 * I thought there was an off-by-one error here, but there isn't.
-		 * This probably means that there's an off-by-one error somewhere
-		 * else ... or maybe only in certain messages?
-		--part_end;
-		--length;
-		******/
-		
-		/* Truncate if the header told us to */
-                /* Content length headers tend to be wrong since mails
-		 * gain/loose linefeeds/QP all over their way
-		 * so, we ignore the size for now and trust our own calculation.
-		if ( (content_length > 0) && (length > content_length) ) {
-			length = content_length;
-		}
-                */
+
+		/* The following code will truncate the MIME part to the size
+		 * specified by the Content-length: header.   We have commented it
+		 * out because these headers have a tendency to be wrong.
+		 *
+		 *	if ( (content_length > 0) && (length > content_length) ) {
+		 *		length = content_length;
+		 *	}
+                 */
 
 		/* Sometimes the "name" field is tacked on to Content-type,
 		 * and sometimes it's tacked on to Content-disposition.  Use