From 4ccf2f08c1aa8b15f31ac469edebe6c4710d74f1 Mon Sep 17 00:00:00 2001 From: Leah Neukirchen Date: Sat, 7 Aug 2021 19:33:08 +0200 Subject: mmime: ensure no empty parts are generated after an include. Reported by lhynes. --- mmime.c | 13 +++++++++---- t/1000-mmime.t | 11 ++++++++++- 2 files changed, 19 insertions(+), 5 deletions(-) diff --git a/mmime.c b/mmime.c index 7d47a20..f082cf0 100644 --- a/mmime.c +++ b/mmime.c @@ -423,6 +423,7 @@ gen_build() size_t linelen = 0; int inheader = 1; int intext = 0; + int emptybody = 1; int ret = 0; char *contenttype = 0; char *contenttransferenc = 0; @@ -430,12 +431,13 @@ gen_build() while (1) { ssize_t read = getdelim(&line, &linelen, '\n', stdin); if (read == -1) { - if (!intext && !rflag) + if (feof(stdin)) { + if (!emptybody) + break; line = strdup(inheader ? "\n" : ""); - else if (feof(stdin)) - break; - else + } else { // errored exit(1); + } } if (inheader) { if (line[0] == '\n') { @@ -476,6 +478,7 @@ gen_build() if (gen_file(f+1, line+1) != 0) ret = 1; intext = 0; + emptybody = 0; continue; } *f = of; @@ -496,6 +499,8 @@ gen_build() printf("%s", line); else gen_qp((uint8_t *)line, strlen(line), 78, 0); + + emptybody = 0; } if (!rflag && !inheader) printf("\n--%s--\n", sep); diff --git a/t/1000-mmime.t b/t/1000-mmime.t index 0408d32..6d4c40c 100755 --- a/t/1000-mmime.t +++ b/t/1000-mmime.t @@ -1,7 +1,8 @@ #!/bin/sh -e cd ${0%/*} . ./lib.sh -plan 16 + +plan 17 cat <tmp References: @@ -115,3 +116,11 @@ Body. EOF check 'non-encoded quoted-strings are kept correctly' 'mmime tmp2 +Subject: inclusion without further content + +#message/rfc822#inline $PWD/tmp +EOF + +check 'no empty parts are generated after inclusion lines' '! mmime