------------------------------------------------------------
revno: 1072
committer: Mark Sapiro <[email protected]>
branch nick: 2.2
timestamp: Thu 2009-09-03 11:07:58 -0700
message:
  Inadvertently setting a null site or list password allowed access
  to a lists web admin interface without authentication.  Fixed by
  not accepting null passwords.
modified:
  Mailman/SecurityManager.py
  NEWS


--
lp:mailman/2.2
https://code.launchpad.net/~mailman-coders/mailman/2.2

Your team Mailman Checkins is subscribed to branch lp:mailman/2.2.
To unsubscribe from this branch go to 
https://code.launchpad.net/~mailman-coders/mailman/2.2/+edit-subscription.
=== modified file 'Mailman/SecurityManager.py'
--- Mailman/SecurityManager.py	2008-12-03 01:11:33 +0000
+++ Mailman/SecurityManager.py	2009-09-03 18:07:58 +0000
@@ -137,6 +137,9 @@
         #
         # Return the authcontext from the argument sequence that matches the
         # response, or UnAuthorized.
+        if not response:
+            # Don't authenticate null passwords
+            return mm_cfg.UnAuthorized
         for ac in authcontexts:
             if ac == mm_cfg.AuthCreator:
                 ok = Utils.check_global_password(response, siteadmin=0)

=== modified file 'NEWS'
--- NEWS	2009-09-03 17:25:03 +0000
+++ NEWS	2009-09-03 18:07:58 +0000
@@ -72,6 +72,10 @@
 
   Bug Fixes and other patches
 
+    - Inadvertently setting a null site or list password allowed access
+      to a lists web admin interface without authentication.  Fixed by
+      not accepting null passwords.
+
     - Changed VERP_CONFIRM_REGEXP  in Defaults.py to work if the replying
       MUA folds the To: header and in cases where the list name includes '+'.
 

_______________________________________________
Mailman-checkins mailing list
[email protected]
Unsubscribe: 
http://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org

Reply via email to