Fix problems reported by valgrind:

 * Fix memory leak in posix.access()
 * Fix use of uninitialized value in forkpty()
   - from the manpage it isn't clear if there are conditions where master_fd
     are uninitialized, but it's safer to initialize

diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index a116c41d5e10c183b3a74ec1c19c3446cfe38547..0649854f5524fc0d77909cd1a8c4bcf765e48729 100644 (file)
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -1119,6 +1119,7 @@ posix_access(PyObject *self, PyObject *args)
        Py_BEGIN_ALLOW_THREADS
        res = access(path, mode);
        Py_END_ALLOW_THREADS
+       PyMem_Free(path);
        return(PyBool_FromLong(res == 0));
 }
 
@@ -2950,7 +2951,7 @@ To both, return fd of newly opened pseudo-terminal.\n");
 static PyObject *
 posix_forkpty(PyObject *self, PyObject *noargs)
 {
-       int master_fd, pid;
+       int master_fd = -1, pid;
 
        pid = forkpty(&master_fd, NULL, NULL, NULL);
        if (pid == -1)