[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