From 83236f7a8b638454ea74743e1103faa0545e9a62 Mon Sep 17 00:00:00 2001 From: Serhiy Storchaka Date: Sun, 26 Jul 2015 09:01:22 +0300 Subject: Issue #24683: Fixed crashes in _json functions called with arguments of inappropriate type. --- Lib/test/test_json/test_separators.py | 6 ++++++ Misc/NEWS | 3 +++ Modules/_json.c | 12 ++++++++++-- 3 files changed, 19 insertions(+), 2 deletions(-) diff --git a/Lib/test/test_json/test_separators.py b/Lib/test/test_json/test_separators.py index 84a2be9..8ca5174 100644 --- a/Lib/test/test_json/test_separators.py +++ b/Lib/test/test_json/test_separators.py @@ -39,6 +39,12 @@ class TestSeparators: self.assertEqual(h2, h) self.assertEqual(d2, expect) + def test_illegal_separators(self): + h = {1: 2, 3: 4} + self.assertRaises(TypeError, self.dumps, h, separators=(b', ', ': ')) + self.assertRaises(TypeError, self.dumps, h, separators=(', ', b': ')) + self.assertRaises(TypeError, self.dumps, h, separators=(b', ', b': ')) + class TestPySeparators(TestSeparators, PyTest): pass class TestCSeparators(TestSeparators, CTest): pass diff --git a/Misc/NEWS b/Misc/NEWS index a347b94..b84e38d 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -66,6 +66,9 @@ Core and Builtins Library ------- +- Issue #24683: Fixed crashes in _json functions called with arguments of + inappropriate type. + - Issue #21697: shutil.copytree() now correctly handles symbolic links that point to directories. Patch by Eduardo Seabra and Thomas Kluyver. diff --git a/Modules/_json.c b/Modules/_json.c index 2f42c34..dded2c9 100644 --- a/Modules/_json.c +++ b/Modules/_json.c @@ -1223,11 +1223,19 @@ encoder_init(PyObject *self, PyObject *args, PyObject *kwds) assert(PyEncoder_Check(self)); s = (PyEncoderObject *)self; - if (!PyArg_ParseTupleAndKeywords(args, kwds, "OOOOOOOOp:make_encoder", kwlist, - &markers, &defaultfn, &encoder, &indent, &key_separator, &item_separator, + if (!PyArg_ParseTupleAndKeywords(args, kwds, "OOOOUUOOp:make_encoder", kwlist, + &markers, &defaultfn, &encoder, &indent, + &key_separator, &item_separator, &sort_keys, &skipkeys, &allow_nan)) return -1; + if (markers != Py_None && !PyDict_Check(markers)) { + PyErr_Format(PyExc_TypeError, + "make_encoder() argument 1 must be dict or None, " + "not %.200s", Py_TYPE(markers)->tp_name); + return -1; + } + s->markers = markers; s->defaultfn = defaultfn; s->encoder = encoder; -- cgit v0.12