mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-09-26 18:29:57 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

Fixed possible leaks in ElementTree parser.

Browse source
This commit is contained in:
Serhiy Storchaka 2015-12-09 19:44:30 +02:00
parent d6a69d8ccb
commit a29eb08fb9
1 changed files with 8 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

10
Modules/_elementtree.c
View file

@ -2935,8 +2935,10 @@ expat_start_handler(XMLParserObject* self, const XML_Char* tag_in,
/* attributes */ /* attributes */
if (attrib_in[0]) { if (attrib_in[0]) {
attrib = PyDict_New(); attrib = PyDict_New();
if (!attrib) if (!attrib) {
Py_DECREF(tag);
return; return;
}
while (attrib_in[0] && attrib_in[1]) { while (attrib_in[0] && attrib_in[1]) {
PyObject* key = makeuniversal(self, attrib_in[0]); PyObject* key = makeuniversal(self, attrib_in[0]);
PyObject* value = PyUnicode_DecodeUTF8(attrib_in[1], strlen(attrib_in[1]), "strict"); PyObject* value = PyUnicode_DecodeUTF8(attrib_in[1], strlen(attrib_in[1]), "strict");
@ -2944,6 +2946,7 @@ expat_start_handler(XMLParserObject* self, const XML_Char* tag_in,
Py_XDECREF(value); Py_XDECREF(value);
Py_XDECREF(key); Py_XDECREF(key);
Py_DECREF(attrib); Py_DECREF(attrib);
Py_DECREF(tag);
return; return;
} }
ok = PyDict_SetItem(attrib, key, value); ok = PyDict_SetItem(attrib, key, value);
@ -2951,6 +2954,7 @@ expat_start_handler(XMLParserObject* self, const XML_Char* tag_in,
Py_DECREF(key); Py_DECREF(key);
if (ok < 0) { if (ok < 0) {
Py_DECREF(attrib); Py_DECREF(attrib);
Py_DECREF(tag);
return; return;
} }
attrib_in += 2; attrib_in += 2;
@ -2958,8 +2962,10 @@ expat_start_handler(XMLParserObject* self, const XML_Char* tag_in,
} else { } else {
/* Pass an empty dictionary on */ /* Pass an empty dictionary on */
attrib = PyDict_New(); attrib = PyDict_New();
if (!attrib) if (!attrib) {
Py_DECREF(tag);
return; return;
}
} }
if (TreeBuilder_CheckExact(self->target)) { if (TreeBuilder_CheckExact(self->target)) {