From: Christian Heimes Date: Fri, 18 Jan 2008 08:47:59 +0000 (+0000) Subject: Coverity issue CID #169 X-Git-Tag: v2.6a1~563 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3e8c897d80db38613f26b2f4c120f9e15a8ae9d2;p=python Coverity issue CID #169 local_ptr_assign_local: Assigning address of stack variable "namebuf" to pointer "filename" out_of_scope: Variable "namebuf" goes out of scope use_invalid: Used "filename" pointing to out-of-scope variable "namebuf" --- diff --git a/Python/traceback.c b/Python/traceback.c index d6f68ab169..877ca5a475 100644 --- a/Python/traceback.c +++ b/Python/traceback.c @@ -129,6 +129,8 @@ tb_displayline(PyObject *f, char *filename, int lineno, char *name) FILE *xfp; char linebuf[2000]; int i; + char namebuf[MAXPATHLEN+1]; + if (filename == NULL || name == NULL) return -1; /* This is needed by Emacs' compile command */ @@ -147,7 +149,6 @@ tb_displayline(PyObject *f, char *filename, int lineno, char *name) Py_ssize_t _npath = PyList_Size(path); int npath = Py_SAFE_DOWNCAST(_npath, Py_ssize_t, int); size_t taillen = strlen(tail); - char namebuf[MAXPATHLEN+1]; for (i = 0; i < npath; i++) { PyObject *v = PyList_GetItem(path, i); if (v == NULL) {