From 1af415f539730e67a268dc472e89b18d32e157c7 Mon Sep 17 00:00:00 2001
From: bwarsaw
Date: Tue, 4 Feb 2003 15:32:51 +0000
Subject: verpdeliver(): If we find a non-fully qualified recipient address
 when calculating the verp header, we log a message and skip this recipient,
 because we can't possibly deliver to this person via verp.

---
 Mailman/Handlers/SMTPDirect.py | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/Mailman/Handlers/SMTPDirect.py b/Mailman/Handlers/SMTPDirect.py
index fd64f6f1a..e1f832ea9 100644
--- a/Mailman/Handlers/SMTPDirect.py
+++ b/Mailman/Handlers/SMTPDirect.py
@@ -274,6 +274,14 @@ def verpdeliver(mlist, msg, msgdata, envsender, failures, conn):
         if msgdata.get('verp'):
             bmailbox, bdomain = Utils.ParseEmail(envsender)
             rmailbox, rdomain = Utils.ParseEmail(recip)
+            if rdomain is None:
+                # The recipient address is not fully-qualified.  We can't
+                # deliver it to this person, nor can we craft a valid verp
+                # header.  I don't think there's much we can do except ignore
+                # this recipient.
+                syslog('smtp', 'Skipping VERP delivery to unqual recip: %s',
+                       recip)
+                continue
             d = {'bounces': bmailbox,
                  'mailbox': rmailbox,
                  'host'   : DOT.join(rdomain),
-- 
cgit v1.3.1