diff options
| author | Neal Norwitz <nnorwitz@gmail.com> | 2006-08-12 02:06:34 (GMT) |
|---|---|---|
| committer | Neal Norwitz <nnorwitz@gmail.com> | 2006-08-12 02:06:34 (GMT) |
| commit | 6b4953fd3d3d1df06f692af67f593d3d0a7aef26 (patch) | |
| tree | 6b659e9036a0e6961855205920251a5b0bd6d647 /Objects | |
| parent | b45f351832b00c80bf9881e92b12c330324e3116 (diff) | |
| download | cpython-6b4953fd3d3d1df06f692af67f593d3d0a7aef26.zip cpython-6b4953fd3d3d1df06f692af67f593d3d0a7aef26.tar.gz cpython-6b4953fd3d3d1df06f692af67f593d3d0a7aef26.tar.bz2 | |
Check returned pointer is valid.
Klocwork #233
Diffstat (limited to 'Objects')
| -rw-r--r-- | Objects/funcobject.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/Objects/funcobject.c b/Objects/funcobject.c index 1ba74c5..b972e08 100644 --- a/Objects/funcobject.c +++ b/Objects/funcobject.c @@ -486,9 +486,10 @@ function_call(PyObject *func, PyObject *arg, PyObject *kw) Py_ssize_t nk, nd; argdefs = PyFunction_GET_DEFAULTS(func); + /* XXX(nnorwitz): don't we know argdefs is either NULL or a tuple? */ if (argdefs != NULL && PyTuple_Check(argdefs)) { d = &PyTuple_GET_ITEM((PyTupleObject *)argdefs, 0); - nd = PyTuple_Size(argdefs); + nd = PyTuple_GET_SIZE(argdefs); } else { d = NULL; @@ -517,7 +518,7 @@ function_call(PyObject *func, PyObject *arg, PyObject *kw) result = PyEval_EvalCodeEx( (PyCodeObject *)PyFunction_GET_CODE(func), PyFunction_GET_GLOBALS(func), (PyObject *)NULL, - &PyTuple_GET_ITEM(arg, 0), PyTuple_Size(arg), + &PyTuple_GET_ITEM(arg, 0), PyTuple_GET_SIZE(arg), k, nk, d, nd, PyFunction_GET_CLOSURE(func)); |