Mirror
/
cpython
mirror of https://github.com/python/cpython
4
1
Fork 0
Code Issues Releases Wiki Activity

Issue #17989: element_setattro returned incorrect error value. 

This caused an exception to be raised later than expected.
This commit is contained in:
Eli Bendersky 2013-05-18 07:53:47 -07:00
parent 163e9821b4 ef9683b73f
commit 1a8c3e247a
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
Modules/_elementtree.c
View File

@ -1768,17 +1768,16 @@ element_getattro(ElementObject* self, PyObject* nameobj)
return res;
}
static PyObject*
static int
element_setattro(ElementObject* self, PyObject* nameobj, PyObject* value)
{
char *name = "";
if (PyUnicode_Check(nameobj))
name = _PyUnicode_AsString(nameobj);
if (name == NULL)
return NULL;
if (strcmp(name, "tag") == 0) {
if (name == NULL) {
return -1;
} else if (strcmp(name, "tag") == 0) {
Py_DECREF(self->tag);
self->tag = value;
Py_INCREF(self->tag);
@ -1797,11 +1796,12 @@ element_setattro(ElementObject* self, PyObject* nameobj, PyObject* value)
self->extra->attrib = value;
Py_INCREF(self->extra->attrib);
} else {
PyErr_SetString(PyExc_AttributeError, name);
return NULL;
PyErr_SetString(PyExc_AttributeError,
"Can't set arbitraty attributes on Element");
return -1;
}
return NULL;
return 0;
}
static PySequenceMethods element_as_sequence = {