From: Neal Norwitz <nnorwitz@gmail.com>
Date: Sun, 17 Oct 2004 16:27:18 +0000 (+0000)
Subject: Invalid patterns to substitute and safe_substitute would crash since pattern
X-Git-Tag: v2.4b2~102
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6627a9670541f33ae55f6ba8df6b6ce7264c3f34;p=python

Invalid patterns to substitute and safe_substitute would crash since pattern
is not a local variable.  Add a test case.
---

diff --git a/Lib/string.py b/Lib/string.py
index 7371c915a1..e10087eb2a 100644
--- a/Lib/string.py
+++ b/Lib/string.py
@@ -167,7 +167,8 @@ class Template:
                 return self.delimiter
             if mo.group('invalid') is not None:
                 self._invalid(mo)
-            raise ValueError('Unrecognized named group in pattern', pattern)
+            raise ValueError('Unrecognized named group in pattern',
+                             self.pattern)
         return self.pattern.sub(convert, self.template)
 
     def safe_substitute(self, *args, **kws):
@@ -199,7 +200,8 @@ class Template:
                 return self.delimiter
             if mo.group('invalid') is not None:
                 self._invalid(mo)
-            raise ValueError('Unrecognized named group in pattern', pattern)
+            raise ValueError('Unrecognized named group in pattern',
+                             self.pattern)
         return self.pattern.sub(convert, self.template)
 
 
diff --git a/Lib/test/test_pep292.py b/Lib/test/test_pep292.py
index f955774828..19952e4239 100644
--- a/Lib/test/test_pep292.py
+++ b/Lib/test/test_pep292.py
@@ -113,6 +113,18 @@ class TestTemplate(unittest.TestCase):
         s = MyPattern('@bag.foo.who likes to eat a bag of @bag.what')
         self.assertEqual(s.substitute(m), 'tim likes to eat a bag of ham')
 
+        class BadPattern(Template):
+            pattern = r"""
+            (?P<badname>.*)                     |
+            (?P<escaped>@{2})                   |
+            @(?P<named>[_a-z][._a-z0-9]*)       |
+            @{(?P<braced>[_a-z][._a-z0-9]*)}    |
+            (?P<invalid>@)                      |
+            """
+        s = BadPattern('@bag.foo.who likes to eat a bag of @bag.what')
+        self.assertRaises(ValueError, s.substitute, {})
+        self.assertRaises(ValueError, s.safe_substitute, {})
+
     def test_unicode_values(self):
         s = Template('$who likes $what')
         d = dict(who=u't\xffm', what=u'f\xfe\fed')