ProcessConfirmCmd(): Confirmation of an subscription for an address

  can bomb out with MMAlreadyAMember if the address has already been
  subscribed by some other mechanism after the pending confirmation
  was registered.  Catch this and reply politely.

1 files changed, 4 insertions, 1 deletions

diff --git a/Mailman/MailCommandHandler.py b/Mailman/MailCommandHandler.py
index 5b8d9424f..cc9bc410a 100644
--- a/Mailman/MailCommandHandler.py
+++ b/Mailman/MailCommandHandler.py
@@ -609,7 +609,10 @@ background and instructions for subscribing to and using it, visit:
         except Errors.MMNeedApproval, admin_addr:
             self.AddToResponse("Your request has been forwarded to the list "
                                "administrator for approval")
-            
+        except Errors.MMAlreadyAMember:
+            # Some other subscription request for this address has
+            # already succeeded.
+            self.AddError("You are already subscribed!")
         else:
             #
             # if the list sends a welcome message, we don't need a response