Raise on missing closing quotes in matplotlibrc

timhoffm · timhoffm · commit aa365cadf03d · 2022-03-18T23:54:11.000+01:00
diff --git a/lib/matplotlib/cbook/__init__.py b/lib/matplotlib/cbook/__init__.py
@@ -412,7 +412,10 @@ def _strip_comment(s):
         elif 0 <= hash_pos < quote_pos:
             return s[:hash_pos].strip()
         else:
-            pos = s.find('"', quote_pos + 1) + 1  # behind closing quote
+            closing_quote_pos = s.find('"', quote_pos + 1)
+            if closing_quote_pos < 0:
+                raise ValueError(f"Missing closing quote in: {s}")
+            pos = closing_quote_pos + 1  # behind closing quote
 
 
 def is_writable_file_like(obj):
diff --git a/lib/matplotlib/tests/test_cbook.py b/lib/matplotlib/tests/test_cbook.py
@@ -424,6 +424,13 @@ def test_strip_comment(line, result):
     assert cbook._strip_comment(line) == result
 
 
+def test_strip_comment_invalid():
+    with pytest.raises(ValueError, match="Missing closing quote"):
+        cbook._strip_comment('grid.color: "aa')
+    with pytest.raises(ValueError, match="Missing closing quote"):
+        cbook._strip_comment('grid.color: "aa')
+
+
 def test_sanitize_sequence():
     d = {'a': 1, 'b': 2, 'c': 3}
     k = ['a', 'b', 'c']
