From c68a1fcd75f103d08f8c2dabf9b3a1db4239f59c Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 3 Oct 2015 10:02:33 +0000 Subject: drop Message-IDs longer than 244 bytes Xapian has this limit for terms, and there are likely no legitimate Message-IDs (or single header lines) this long; so there's no need to workaround this limit. --- lib/PublicInbox/MDA.pm | 2 ++ 1 file changed, 2 insertions(+) (limited to 'lib/PublicInbox/MDA.pm') diff --git a/lib/PublicInbox/MDA.pm b/lib/PublicInbox/MDA.pm index 25d6ae55..7025fb3c 100644 --- a/lib/PublicInbox/MDA.pm +++ b/lib/PublicInbox/MDA.pm @@ -6,6 +6,7 @@ use warnings; use Email::Address; use Date::Parse qw(strptime); use constant MAX_SIZE => 1024 * 500; # same as spamc default, should be tunable +use constant MAX_MID_SIZE => 244; # max term size - 1 in Xapian use constant cmd => qw/ssoma-mda -1/; # drop plus addressing for matching @@ -20,6 +21,7 @@ sub precheck { my ($klass, $filter, $address) = @_; my $simple = $filter->simple; my $mid = $simple->header("Message-ID"); + return 0 if (length($mid) > MAX_MID_SIZE); return 0 unless usable_str(length(''), $mid) && $mid =~ /\@/; return 0 unless usable_str(length('u@h'), $filter->from); return 0 unless usable_str(length(':o'), $simple->header("Subject")); -- cgit v1.2.3-24-ge0c7