Any chance of someone commiting the patch in bin/131861 ?

Ulrich Spörlein uqs at spoerlein.net
Wed Apr 14 16:33:54 UTC 2010


On Wed, 14.04.2010 at 15:38:21 +0100, Pete French wrote:
> > Sorry Pete, but the patch still seems incomplete. You merely catch the
> > case when a comma is followed by space or quotation marks, but the email
> > header might look like this:
> > To: foo at domain.com,bar at otherdomain.com
> 
> I think the original code handles cases like that fine, my patch merely
> looks for an extra possibility. I've just tested it with the
> above, and it works.
> 
> Can you give me an example of one which doesn't work for you ? Either
> in the original /usr/bin/mail or in my patched version ?

Well, the following header didn't work:
Cc: <a at a.a>,<b at b.b>

Postfix will re-write this as part of sanitization, so I had to revert
to creating mbox files by hand. Anyway, could you please test the
following patch with a wider variety of mails?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mail.diff
Type: text/x-diff
Size: 981 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-stable/attachments/20100414/68c522a0/mail.bin


More information about the freebsd-stable mailing list