Bill Cole:
> On 9 Apr 2016, at 9:00, Wietse Venema wrote:
>
> > Unfortunately, I don't have time to decode this discussion. Can
> > someone post a tested diff, someone maybe post a revised version,
> > and when there is agreement, then I can adopt it.
>
>
> Simplest fix: prevent *that* class of false positives by narrowing the
> check to a single attribute, rather than including all attributes in the
> header following one which includes 'name' in its name:
>
> - /^Content-(Disposition|Type).*name\s*=\s*"?(.*(\.|=2E)(
> + /^Content-(Disposition|Type).*name\s*=\s*"?([^;]*(\.|=2E)(
Thanks, got it.
> As Viktor noted: regular expressions are the wrong toolkit for MIME
> parsing. A proper mature MIME parser is available for Postfix in the
> MIMEDefang milter, which also is a fine tool for hooking in SpamAssassin
> and AV tools.
Absolutely. But this class of false positives is easy to remove.
Wietse