From cb71e15ef27c145eede8dc3a858c401c70d9cd19 Mon Sep 17 00:00:00 2001
From: bwarsaw
Date: Fri, 2 May 2003 03:45:31 +0000
Subject: main(): It's possible for old databases to have password-less members
 in them.  In that case, log it and continue.

---
 cron/mailpasswds | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/cron/mailpasswds b/cron/mailpasswds
index 348ca3363..88bb624d9 100755
--- a/cron/mailpasswds
+++ b/cron/mailpasswds
@@ -142,7 +142,15 @@ def main():
                     continue
                 # Group by the lower-cased address, since Mailman always
                 # treates person@dom.ain the same as PERSON@dom.ain.
-                password = mlist.getMemberPassword(member)
+                try:
+                    password = mlist.getMemberPassword(member)
+                except Errors.NotAMemberError:
+                    # Here's a member with no passwords, which I think was
+                    # possible in older versions of Mailman.  Log this and
+                    # move on.
+                    syslog('error', 'password-less member %s for list %s',
+                           member, mlist.internal_name())
+                    continue
                 optionsurl = mlist.GetOptionsURL(member)
                 lang = mlist.getMemberLanguage(member)
                 info = (listaddr, password, optionsurl, lang)
-- 
cgit v1.3.1