diff options
| author | Serhiy Storchaka <storchaka@gmail.com> | 2015-09-07 19:37:02 (GMT) |
|---|---|---|
| committer | Serhiy Storchaka <storchaka@gmail.com> | 2015-09-07 19:37:02 (GMT) |
| commit | 931331a328d522bb014f9e9c13884d7566108268 (patch) | |
| tree | 23432881b1f5933381fb5a100c8def49799ad8d6 /Modules/pyexpat.c | |
| parent | eba8feedfa0d62b042ad0ecabf6cab733528dbd6 (diff) | |
| download | cpython-931331a328d522bb014f9e9c13884d7566108268.zip cpython-931331a328d522bb014f9e9c13884d7566108268.tar.gz cpython-931331a328d522bb014f9e9c13884d7566108268.tar.bz2 | |
Issue #25019: Fixed a crash caused by setting non-string key of expat parser.
Added additional tests for expat parser attributes.
Based on patch by John Leitch.
Diffstat (limited to 'Modules/pyexpat.c')
| -rw-r--r-- | Modules/pyexpat.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/Modules/pyexpat.c b/Modules/pyexpat.c index 19be0c7..c4fdaaf 100644 --- a/Modules/pyexpat.c +++ b/Modules/pyexpat.c @@ -1341,11 +1341,16 @@ static int xmlparse_setattro(xmlparseobject *self, PyObject *name, PyObject *v) { /* Set attribute 'name' to value 'v'. v==NULL means delete */ + if (!PyUnicode_Check(name)) { + PyErr_Format(PyExc_TypeError, + "attribute name must be string, not '%.200s'", + name->ob_type->tp_name); + return -1; + } if (v == NULL) { PyErr_SetString(PyExc_RuntimeError, "Cannot delete attribute"); return -1; } - assert(PyUnicode_Check(name)); if (PyUnicode_CompareWithASCIIString(name, "buffer_text") == 0) { int b = PyObject_IsTrue(v); if (b < 0) |