summaryrefslogtreecommitdiffstats
path: root/Modules/posixmodule.c
diff options
context:
space:
mode:
authorTim Peters <tim.peters@gmail.com>2006-03-01 04:35:45 (GMT)
committerTim Peters <tim.peters@gmail.com>2006-03-01 04:35:45 (GMT)
commit67d70eb957faf61e0cba548fcad374ec361429c4 (patch)
treec1ef1c0e17e60e27958ff7bd5ff1ba0fb4c1c215 /Modules/posixmodule.c
parentacde7347a547ce515b42805cb18789591c03c016 (diff)
downloadcpython-67d70eb957faf61e0cba548fcad374ec361429c4.zip
cpython-67d70eb957faf61e0cba548fcad374ec361429c4.tar.gz
cpython-67d70eb957faf61e0cba548fcad374ec361429c4.tar.bz2
Repair mangled code in the Windows flavor of
posix__getfullpathname(). In partial answer to the now-deleted XXX comment: /* XXX(twouters) Why use 'et#' here at all? insize isn't used */ `insize` is an input parameter too, and it was left uninitialized, leading to seemingly random failures.
Diffstat (limited to 'Modules/posixmodule.c')
-rw-r--r--Modules/posixmodule.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index efe5074..6949c90 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -1901,7 +1901,7 @@ posix__getfullpathname(PyObject *self, PyObject *args)
/* assume encoded strings wont more than double no of chars */
char inbuf[MAX_PATH*2];
char *inbufp = inbuf;
- Py_ssize_t insize;
+ Py_ssize_t insize = sizeof(inbuf);
char outbuf[MAX_PATH*2];
char *temp;
#ifdef Py_WIN_WIDE_FILENAMES
@@ -1921,7 +1921,6 @@ posix__getfullpathname(PyObject *self, PyObject *args)
PyErr_Clear();
}
#endif
- /* XXX(twouters) Why use 'et#' here at all? insize isn't used */
if (!PyArg_ParseTuple (args, "et#:_getfullpathname",
Py_FileSystemDefaultEncoding, &inbufp,
&insize))