diff options
author | Leah Neukirchen <leah@vuxu.org> | 2021-08-30 17:49:32 +0200 |
---|---|---|
committer | Leah Neukirchen <leah@vuxu.org> | 2021-08-30 18:12:30 +0200 |
commit | 41bd429452f45cb43110ec4699871a2cbd9c0a5e (patch) | |
tree | 49fa1e50cb888dceb7341f88d3d465caa7d42823 /t | |
parent | 4be6e0ce91c0690d3cbf58bd7b76144e01fe2be7 (diff) | |
download | mblaze-41bd429452f45cb43110ec4699871a2cbd9c0a5e.tar.gz mblaze-41bd429452f45cb43110ec4699871a2cbd9c0a5e.tar.xz mblaze-41bd429452f45cb43110ec4699871a2cbd9c0a5e.zip |
blaze822: blaze822/blaze822_mem: detect line ending before scanning end of header
A mail using CRLF which contained (for some reason) a LFLF pair would be misparsed as the header was read until the LFLF. Instead, scan for the first LF, check if it's preceded by CR, and then search for the proper header terminator only. Closes #212.
Diffstat (limited to 't')
-rwxr-xr-x | t/1701-mshow-regress.t | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/t/1701-mshow-regress.t b/t/1701-mshow-regress.t new file mode 100755 index 0000000..ffb676d --- /dev/null +++ b/t/1701-mshow-regress.t @@ -0,0 +1,29 @@ +#!/bin/sh -e +cd ${0%/*} +. ./lib.sh +plan 2 + +# Mail with \n\n and \r\n\r\n +cr=$(printf '\r') +cat <<EOF >tmp +Content-Type: multipart/form-data; boundary=------------------------55a586f81559face$cr +$cr +--------------------------55a586f81559face$cr +Content-Disposition: form-data; name="a"; filename="foo"$cr +Content-Type: application/octet-stream$cr +$cr +foo$cr + +previously there are two NL$cr +$cr +--------------------------55a586f81559face$cr +Content-Disposition: form-data; name="a"; filename="bar"$cr +Content-Type: application/octet-stream$cr +$cr +bar$cr +$cr +--------------------------55a586f81559face--$cr +EOF + +check 'mail has 3 attachments' 'mshow -t ./tmp | wc -l | grep 4' +check 'mail attachment foo has size 35' 'mshow -t ./tmp | grep size=35.*name=\"foo\"' |