diff options
| author |   Christian Heimes <christian@cheimes.de> | 2012-09-10 02:45:31 +0200 |
|---|---|---|
| committer | Christian Heimes <christian@cheimes.de> | 2012-09-10 02:45:31 +0200 |
| commit | 949f3317312c64425efae21bda86b98423aac9cf (patch) | |
| tree | e8adc528ca62b6ce58d53b995b6366df018519dd /Objects/classobject.c | |
| parent | Closes #15676: mmap: add empty file check prior to offset check (diff) | |
| download | cpython-949f3317312c64425efae21bda86b98423aac9cf.tar.gz cpython-949f3317312c64425efae21bda86b98423aac9cf.tar.bz2 cpython-949f3317312c64425efae21bda86b98423aac9cf.zip | |
Py_TYPE() has already dereferenced self before the NULL check. Moved Py_TYPE() after the check for self == NULL
Diffstat (limited to 'Objects/classobject.c')
| -rw-r--r-- | Objects/classobject.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Objects/classobject.c b/Objects/classobject.c index b7d35ef8852..f9568527b36 100644 --- a/Objects/classobject.c +++ b/Objects/classobject.c @@ -218,7 +218,7 @@ method_repr(PyMethodObject *a) { PyObject *self = a->im_self; PyObject *func = a->im_func; - PyObject *klass = (PyObject*)Py_TYPE(self); + PyObject *klass; PyObject *funcname = NULL ,*klassname = NULL, *result = NULL; char *defname = "?"; @@ -226,6 +226,7 @@ method_repr(PyMethodObject *a) PyErr_BadInternalCall(); return NULL; } + klass = (PyObject*)Py_TYPE(self); funcname = PyObject_GetAttrString(func, "__name__"); if (funcname == NULL) { |