Fixed a bug in bin/rmlist that would throw an exception or just fail to

remove held message files for a list with regexp special characters in
its name.

2 files changed, 6 insertions, 2 deletions

diff --git a/NEWS b/NEWS
index b14f50c6..061cae15 100755
--- a/NEWS
+++ b/NEWS
@@ -1,6 +1,6 @@
 -*- coding: iso-8859-1 -*-
 Mailman - The GNU Mailing List Management System
-Copyright (C) 1998-2014 by the Free Software Foundation, Inc.
+Copyright (C) 1998-2015 by the Free Software Foundation, Inc.
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
 
 Here is a history of user visible changes to Mailman.
@@ -124,6 +124,10 @@ Here is a history of user visible changes to Mailman.
 
   Bug fixes and other patches
 
+    - Fixed a bug in bin/rmlist that would throw an exception or just fail to
+      remove held message files for a list with regexp special characters in
+      its name.  (LP:#1414864)
+
     - When applying DMARC mitigations, CookHeaders now adds the original From:
       to Cc: rather than Reply-To: in some cases to make MUA 'reply' and
       'reply all' more consistent with the non-DMARC cases.  (LP: #1407098)
diff --git a/bin/rmlist b/bin/rmlist
index f61b41d4..b0980104 100755
--- a/bin/rmlist
+++ b/bin/rmlist
@@ -134,7 +134,7 @@ def main():
 
     # Remove any held messages for this list
     for filename in os.listdir(mm_cfg.DATA_DIR):
-        cre = re.compile('^heldmsg-%s-\d+\.(pck|txt)$' % listname,
+        cre = re.compile('^heldmsg-%s-\d+\.(pck|txt)$' % re.escape(listname),
                          re.IGNORECASE)
         if cre.match(filename):
             REMOVABLES.append((os.path.join(mm_cfg.DATA_DIR, filename),