[Bro-Dev] [Bro-Commits] [git/bro] fastpath: Fixed a bug with the MIME analyzer not removing whitespace on wrapped headers. (89cb103)
Robin Sommer
robin at icir.org
Wed Jun 6 12:20:59 PDT 2012
I had applied this, but I'm not sure it's actually correct. RFC 822
states:
Unfolding is accomplished by regarding CRLF immediately
followed by a LWSP-char as equivalent to the LWSP-char.
Which seems to say the white-space should be left in except for the
CRLF.
Also, by removing it, the two parts now flow togehter. Here's an
example I found with the test-suite:
Old: ([xx.xx.xx.xx])\x09by xxx.xxx.gov
New: ([xx.xx.xx.xx])by xxx.xxx.gov
So I'm reverting this for now until we've decided what's right.
Robin
--
Robin Sommer * Phone +1 (510) 722-6541 * robin at icir.org
ICSI/LBNL * Fax +1 (510) 666-2956 * www.icir.org
More information about the bro-dev
mailing list