From 21ad7e7ac75ce46bbbfc9283509cba81a42c5511 Mon Sep 17 00:00:00 2001
From: bwarsaw <>
Date: Sun, 29 Feb 2004 16:45:27 +0000
Subject: main(): The list lock must be held in order to pend unsubscription
 requests.

---
 Mailman/Cgi/options.py | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

(limited to 'Mailman')

diff --git a/Mailman/Cgi/options.py b/Mailman/Cgi/options.py
index 447dc38c..aa5b1da1 100644
--- a/Mailman/Cgi/options.py
+++ b/Mailman/Cgi/options.py
@@ -163,22 +163,23 @@ def main():
         # Because they can't supply a password for unsubscribing, we'll need
         # to do the confirmation dance.
         if mlist.isMember(user):
-            # If unsubs require admin approval, then this request has to be
-            # held.  Otherwise, send a confirmation.
-            if mlist.unsubscribe_policy:
-                try:
-                    mlist.Lock()
+            # We must acquire the list lock in order to pend a request.
+            try:
+                mlist.Lock()
+                # If unsubs require admin approval, then this request has to
+                # be held.  Otherwise, send a confirmation.
+                if mlist.unsubscribe_policy:
                     mlist.HoldUnsubscription(user)
                     doc.addError(_("""Your unsubscription request has been
                     forwarded to the list administrator for approval."""),
                                  tag='')
-                    mlist.Save()
-                finally:
-                    mlist.Unlock()
-            else:
-                mlist.ConfirmUnsubscription(user, userlang)
-                doc.addError(_('The confirmation email has been sent.'),
-                             tag='')
+                else:
+                    mlist.ConfirmUnsubscription(user, userlang)
+                    doc.addError(_('The confirmation email has been sent.'),
+                                 tag='')
+                mlist.Save()
+            finally:
+                mlist.Unlock()
         else:
             # Not a member
             if mlist.private_roster == 0:
-- 
cgit v1.2.3