Fix handling of empty strings in .get_dtext, .get_qp_ctext and .get_qcontent

sergey-miryanov · sergey-miryanov · commit 5c3fa1140ef5 · 2025-05-18T23:39:21.000+05:00
diff --git a/Lib/email/_header_value_parser.py b/Lib/email/_header_value_parser.py
@@ -1024,6 +1024,7 @@ def _get_ptext_to_endchars(value, endchars):
     vchars = []
     escape = False
     had_qp = False
+    pos = 0
     for pos in range(len(fragment)):
         if fragment[pos] == '\\':
             if escape:
diff --git a/Lib/test/test_email/test__header_value_parser.py b/Lib/test/test_email/test__header_value_parser.py
@@ -463,6 +463,9 @@ def test_get_qp_ctext_non_printables(self):
                                 [errors.NonPrintableDefect], ')')
         self.assertEqual(ptext.defects[0].non_printables[0], '\x00')
 
+    def test_get_qp_ctext_empty(self):
+        self._test_get_x(parser.get_qp_ctext, '', '', ' ', [], '')
+
     # get_qcontent
 
     def test_get_qcontent_only(self):
@@ -503,6 +506,9 @@ def test_get_qcontent_non_printables(self):
                                 [errors.NonPrintableDefect], '"')
         self.assertEqual(ptext.defects[0].non_printables[0], '\x00')
 
+    def test_get_qcontent_empty(self):
+        self._test_get_x(parser.get_qcontent, '', '', '', [], '')
+
     # get_atext
 
     def test_get_atext_only(self):
@@ -1283,6 +1289,9 @@ def test_get_dtext_open_bracket_mid_word(self):
         self._test_get_x(parser.get_dtext,
                         'foo[bar', 'foo', 'foo', [], '[bar')
 
+    def test_get_dtext_empty(self):
+        self._test_get_x(parser.get_dtext, '', '', '', [], '')
+
     # get_domain_literal
 
     def test_get_domain_literal_only(self):
