[ mailman-Bugs-820322 ] Crash when global subcription change causes collision

[SourceForge.net]

Bugs item #820322, was opened at 2003-10-08 22:47
Message generated for change (Comment added) made by simonha
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=820322&group_id=103

Category: (un)subscribing
Group: 2.1 (stable)
Status: Open
Resolution: None
Priority: 5
Submitted By: Rich Pasco (pasco)
Assigned to: Nobody/Anonymous (nobody)
Summary: Crash when global subcription change causes collision

Initial Comment:
I ran into this bug in MailMan version 2.1.2 today:

If you change your e-mail address "globally" (on all
lists to which you subscribe), and one of those lists
(other than the one whose options page you're working
from) already has a subscription under your new address
(as well as your old address), then MailMan will crash
(and display an exception traceback).

For example, I was working from the web-team page
http://domain.net/mailman/options/listname
as I tried to change my address globally.  This would
affect about ten lists to which my old address
subscribed, including a list called "Team" among
others.  But "Team" already had my new address among
its subscribers, so it could not accept another one. 
Rather than giving a graceful error message when it
discovered this, MailMan just crashed.

I didn't capture the exception traceback, but it should
be easy to reproduce.

     - Rich

----------------------------------------------------------------------

Comment By: Simon Harrison (simonha)
Date: 2003-10-30 11:39

Message:
Logged In: YES 
user_id=898088

I have experience the same problem, here is our traceback:

Traceback:

Traceback (most recent call last):
  File "/usr/local/mailman-2.1.2/scripts/driver", line 87, in 
run_main
    main()
  File "/usr/local/mailman-2.1.2/Mailman/Cgi/confirm.py", line 
131, in main
    addrchange_confirm(mlist, doc, cookie)
  File "/usr/local/mailman-2.1.2/Mailman/Cgi/confirm.py", line 
492, in addrchange_confirm
    op, oldaddr, newaddr = mlist.ProcessConfirmation(cookie)
  File "/usr/local/mailman-2.1.2/Mailman/MailList.py", line 1113, 
in ProcessConfirmation
    self.ApprovedChangeMemberAddress(oldaddr, newaddr, 
globally)
  File "/usr/local/mailman-2.1.2/Mailman/MailList.py", line 1048, 
in ApprovedChangeMemberAddress
    mlist.changeMemberAddress(oldaddr, newaddr)
  File "/usr/local/mailman-2.1.2/Mailman/OldStyleMemberships.
py", line 251, in changeMemberAddress
    password=xxxxxxxx, language=lang)
  File "/usr/local/mailman-2.1.2/Mailman/OldStyleMemberships.
py", line 175, in addNewMember
    raise Errors.MMAlreadyAMember, member
MMAlreadyAMember: oneofourusers at ourorganisation.org.uk

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=820322&group_id=103