diff options
author | Guido van Rossum <guido@python.org> | 2001-08-24 10:17:36 (GMT) |
---|---|---|
committer | Guido van Rossum <guido@python.org> | 2001-08-24 10:17:36 (GMT) |
commit | 845fc48bf739fc04ed3e082aec772c2dc72bf1b6 (patch) | |
tree | 47ddf8f1d4961dd495ee9cd8630f0113c67b3bf9 | |
parent | 2c25239215964746e030d9f7e47232d4dca5746d (diff) | |
download | cpython-845fc48bf739fc04ed3e082aec772c2dc72bf1b6.zip cpython-845fc48bf739fc04ed3e082aec772c2dc72bf1b6.tar.gz cpython-845fc48bf739fc04ed3e082aec772c2dc72bf1b6.tar.bz2 |
getset_descr_set(): guard against deletion (indicated by a set call
with a NULL value), in a somewhat lame way: call the set() function
with one argument. Should I add a 3rd function, 'del', instead?
-rw-r--r-- | Objects/descrobject.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/Objects/descrobject.c b/Objects/descrobject.c index a9ee2a4..c29dd83 100644 --- a/Objects/descrobject.c +++ b/Objects/descrobject.c @@ -906,7 +906,10 @@ getset_descr_set(PyObject *self, PyObject *obj, PyObject *value) PyErr_SetString(PyExc_AttributeError, "unsettable attribute"); return -1; } - res = PyObject_CallFunction(gs->set, "(OO)", obj, value); + if (value == NULL) + res = PyObject_CallFunction(gs->set, "(O)", obj); + else + res = PyObject_CallFunction(gs->set, "(OO)", obj, value); if (res == NULL) return -1; Py_DECREF(res); |