diff options
| author | Facundo Batista <facundobatista@gmail.com> | 2008-06-22 13:36:20 +0000 |
|---|---|---|
| committer | Facundo Batista <facundobatista@gmail.com> | 2008-06-22 13:36:20 +0000 |
| commit | f9a8f28f6aba61bccef4d16326ae5b2e3796a059 (patch) | |
| tree | 34724511d0042df8f530b6f22c970ffffac9ed0a /Modules/posixmodule.c | |
| parent | f843dfde6e7f03706cd502d100fb02f49f324473 (diff) | |
| download | cpython-f9a8f28f6aba61bccef4d16326ae5b2e3796a059.tar.gz | |
Issue #2722. Now the char buffer to support the path string has
not fixed length, it mallocs memory if needed. As a result, we
don't have a maximum for the getcwd() method.
Diffstat (limited to 'Modules/posixmodule.c')
| -rw-r--r-- | Modules/posixmodule.c | 29 |
1 files changed, 24 insertions, 5 deletions
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c index a4bbac556a..8f32fd4349 100644 --- a/Modules/posixmodule.c +++ b/Modules/posixmodule.c @@ -1956,19 +1956,38 @@ Return a string representing the current working directory."); static PyObject * posix_getcwd(PyObject *self, PyObject *noargs) { - char buf[1026]; - char *res; + int bufsize_incr = 1024; + int bufsize = 0; + char *tmpbuf = NULL; + char *res = NULL; + PyObject *dynamic_return; Py_BEGIN_ALLOW_THREADS + do { + bufsize = bufsize + bufsize_incr; + tmpbuf = malloc(bufsize); + if (tmpbuf == NULL) { + break; + } #if defined(PYOS_OS2) && defined(PYCC_GCC) - res = _getcwd2(buf, sizeof buf); + res = _getcwd2(tmpbuf, bufsize); #else - res = getcwd(buf, sizeof buf); + res = getcwd(tmpbuf, bufsize); #endif + + if (res == NULL) { + free(tmpbuf); + } + } while ((res == NULL) && (errno == ERANGE)); Py_END_ALLOW_THREADS + if (res == NULL) return posix_error(); - return PyString_FromString(buf); + + dynamic_return = PyString_FromString(tmpbuf); + free(tmpbuf); + + return dynamic_return; } #ifdef Py_USING_UNICODE |
