Ignore empty Content-Transfer-Encoding headers

Author: jkbzh

Changelog

@@ -7,8 +7,8 @@ HYPERMAIL VERSION 2.4.0:
 2019-12-19 Jose Kahan
 	* src/parse.c
 	parsemail(): a Content-Transfer-Encoding header with a missing value
-	would output a random string when generating unsupported encoding
-	body message.
+	would result in an unitialized variable being used to output an unknown
+	encoding warning message. Hypermail now skips this header if it's empty.
 
 2019-11-22 Jose Kahan
 	* src/print.c

src/parse.c

@@ -2467,18 +2467,23 @@ int parsemail(char *mbox,	/* file name */
 			    char code[64];
 
 			    /* is there any value for content-encoding or is it missing? */
-			    if (*ptr) {
-			      sscanf(ptr, "%63s", code);
+			    if (sscanf(ptr, "%63s", code) != EOF) {
+			      
+			      snprintf(line, sizeof(line_buf) - set_ietf_mbox,
+				       " ('%s' %s)\n", code, 
+				       lang[MSG_ENCODING_IS_NOT_SUPPORTED]);
+
+			      bp = addbody(bp, &lp, line,
+					   BODY_HTMLIZED | bodyflags);
+
+#if DEBUG_PARSE
+			      printf("Ignoring unknown Content-Transfer-Encoding: %s\n", code);
+#endif
 			    } else {
-			      code[0] = '\0';
+#if DEBUG_PARSE
+			      printf("Missing Content-Transfer-Encoding value\n");
+#endif
 			    }
-			    
-                            snprintf(line, sizeof(line_buf) - set_ietf_mbox,
-				     " ('%s' %s)\n", code, 
-                                     lang[MSG_ENCODING_IS_NOT_SUPPORTED]);
-
-			    bp = addbody(bp, &lp, line,
-					BODY_HTMLIZED | bodyflags);
 			}
 #if DEBUG_PARSE
 			printf("DECODE set to %d\n", decode);