summaryrefslogtreecommitdiff
path: root/Mailman/MailList.py
diff options
context:
space:
mode:
authorhmeland1999-06-13 07:19:20 +0000
committerhmeland1999-06-13 07:19:20 +0000
commit76f3ec2988151798f16ddbc01d293b872c0aa79c (patch)
tree7c248f404ff2b1e9d522aecdca732b23ce098120 /Mailman/MailList.py
parent5551c37235c9c603bd7ae608093198c4c4d929f5 (diff)
downloadmailman-76f3ec2988151798f16ddbc01d293b872c0aa79c.tar.gz
mailman-76f3ec2988151798f16ddbc01d293b872c0aa79c.tar.zst
mailman-76f3ec2988151798f16ddbc01d293b872c0aa79c.zip
Create(): Do ValidateEmail(admin) before trying to actually create
anything, to disallow bin/newlist creating lists with bogus admin addresses.
Diffstat (limited to 'Mailman/MailList.py')
-rw-r--r--Mailman/MailList.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/Mailman/MailList.py b/Mailman/MailList.py
index 250740a20..181101469 100644
--- a/Mailman/MailList.py
+++ b/Mailman/MailList.py
@@ -714,8 +714,8 @@ class MailList(MailCommandHandler, HTMLFormatter, Deliverer, ListAdmin,
def Create(self, name, admin, crypted_password):
if name in Utils.list_names():
raise ValueError, 'List %s already exists.' % name
- else:
- Utils.MakeDirTree(os.path.join(mm_cfg.LIST_DATA_DIR, name))
+ Utils.ValidateEmail(admin)
+ Utils.MakeDirTree(os.path.join(mm_cfg.LIST_DATA_DIR, name))
self._full_path = os.path.join(mm_cfg.LIST_DATA_DIR, name)
self._internal_name = name
self.Lock()