diff options
| author | T. Wouters <thomas@python.org> | 2021-07-14 00:56:45 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-07-13 15:56:45 -0700 |
| commit | 0093876328afa330224c9d887c18dee0b3117852 (patch) | |
| tree | 9bd8a30bb6577ab1a973fa68b078ee5e6627f873 /Lib/test/test_csv.py | |
| parent | 054e9c84ac7c394941bba3ea1829d14dce1243fc (diff) | |
| download | cpython-0093876328afa330224c9d887c18dee0b3117852.tar.gz cpython-0093876328afa330224c9d887c18dee0b3117852.zip | |
bpo-44630: Fix assertion errors in csv module (GH-27127)
Fix incorrect handling of exceptions when interpreting dialect objects in
the csv module. Not clearing exceptions between calls to
PyObject_GetAttrString() causes assertion failures in pydebug mode (or with
assertions enabled).
Add a minimal test that would've caught this (passing None as dialect, or
any object that isn't a csv.Dialect subclass, which the csv module allows
and caters to, even though it is not documented.) In pydebug mode, the test
triggers the assertion failure in the old code.
Contributed-By: T. Wouters [Google]
Diffstat (limited to 'Lib/test/test_csv.py')
| -rw-r--r-- | Lib/test/test_csv.py | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/Lib/test/test_csv.py b/Lib/test/test_csv.py index 225f5c72890..18b86aa71a5 100644 --- a/Lib/test/test_csv.py +++ b/Lib/test/test_csv.py @@ -458,9 +458,15 @@ class TestDialectRegistry(unittest.TestCase): class testUni(csv.excel): delimiter = "\u039B" + class unspecified(): + # A class to pass as dialect but with no dialect attributes. + pass + csv.register_dialect('testC', testC) try: self.compare_dialect_123("1,2,3\r\n") + self.compare_dialect_123("1,2,3\r\n", dialect=None) + self.compare_dialect_123("1,2,3\r\n", dialect=unspecified) self.compare_dialect_123("1\t2\t3\r\n", testA) self.compare_dialect_123("1:2:3\r\n", dialect=testB()) self.compare_dialect_123("1|2|3\r\n", dialect='testC') |