commit 015f4d87ab0f7d0888a9f20672feaec373f8ef36
author Victor Stinner <victor.stinner@haypocalc.com> Thu Oct 07 22:29:53 2010 +0000
committer Victor Stinner <victor.stinner@haypocalc.com> Thu Oct 07 22:29:53 2010 +0000
tree 4b3444f0d329c5b4283a6f9b77e77026552d093c
parent a4a759515e2e0114d41acdada3642265c01270ac

_Py_wrealpath() requires the size of the output buffer

Include/fileutils.h

diff --git a/Include/fileutils.h b/Include/fileutils.h
index 1d8df15..a232460 100644
--- a/Include/fileutils.h
+++ b/Include/fileutils.h
@@ -41,7 +41,8 @@
 #ifdef HAVE_REALPATH
 PyAPI_FUNC(wchar_t*) _Py_wrealpath(
     const wchar_t *path,
-    wchar_t *resolved_path);
+    wchar_t *resolved_path,
+    size_t resolved_path_size);
 #endif
 
 PyAPI_FUNC(wchar_t*) _Py_wgetcwd(

Python/fileutils.c

diff --git a/Python/fileutils.c b/Python/fileutils.c
index bd6ab5d..502868f 100644
--- a/Python/fileutils.c
+++ b/Python/fileutils.c
@@ -321,7 +321,8 @@
 
 #ifdef HAVE_REALPATH
 wchar_t*
-_Py_wrealpath(const wchar_t *path, wchar_t *resolved_path)
+_Py_wrealpath(const wchar_t *path,
+              wchar_t *resolved_path, size_t resolved_path_size)
 {
     char *cpath;
     char cresolved_path[PATH_MAX];
@@ -336,7 +337,7 @@
     PyMem_Free(cpath);
     if (res == NULL)
         return NULL;
-    r = mbstowcs(resolved_path, cresolved_path, PATH_MAX);
+    r = mbstowcs(resolved_path, cresolved_path, resolved_path_size);
     if (r == (size_t)-1 || r >= PATH_MAX) {
         errno = EINVAL;
         return NULL;

Python/sysmodule.c

diff --git a/Python/sysmodule.c b/Python/sysmodule.c
index 1eba28e..d02ee5b 100644
--- a/Python/sysmodule.c
+++ b/Python/sysmodule.c
@@ -1742,7 +1742,7 @@
 #else /* All other filename syntaxes */
     if (_HAVE_SCRIPT_ARGUMENT(argc, argv)) {
 #if defined(HAVE_REALPATH)
-        if (_Py_wrealpath(argv0, fullpath)) {
+        if (_Py_wrealpath(argv0, fullpath, PATH_MAX)) {
             argv0 = fullpath;
         }
 #endif