commit b399ab2c5c5d63d6e8b7570921b5e3956dcc9925
author Benjamin Peterson <benjamin@python.org> Fri May 04 01:31:13 2012 -0400
committer Benjamin Peterson <benjamin@python.org> Fri May 04 01:31:13 2012 -0400
tree c36067483149d2d487aa5d0a438852a05f331540
parent 3e2e368f3b493399f3eda04a048863713e29d160

clean up converted path on error

Modules/posixmodule.c

diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index 100d539..73f62f6 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -3695,7 +3695,7 @@
                      "%s: you may specify either 'times'"
                      " or 'ns' but not both",
                      ua->function_name);
-        return 0;
+        goto fail;
     }
 
     if (times && (times != Py_None)) {
@@ -3704,13 +3704,15 @@
                          "%s: 'time' must be either"
                          " a valid tuple of two ints or None",
                          ua->function_name);
-            return 0;
+            goto fail;
         }
         ua->now = 0;
-        return (_PyTime_ObjectToTimespec(PyTuple_GET_ITEM(times, 0),
-                    &(ua->atime_s), &(ua->atime_ns)) != -1)
-            && (_PyTime_ObjectToTimespec(PyTuple_GET_ITEM(times, 1),
-                    &(ua->mtime_s), &(ua->mtime_ns)) != -1);
+        if (_PyTime_ObjectToTimespec(PyTuple_GET_ITEM(times, 0),
+                                     &ua->atime_s, &ua->atime_ns) == -1 ||
+            _PyTime_ObjectToTimespec(PyTuple_GET_ITEM(times, 1),
+                                     &ua->mtime_s, &ua->mtime_ns) == -1)
+            goto fail;
+        return 1;
     }
 
     if (ns) {
@@ -3718,18 +3720,25 @@
             PyErr_Format(PyExc_TypeError,
                          "%s: 'ns' must be a valid tuple of two ints",
                          ua->function_name);
-            return 0;
+            goto fail;
         }
         ua->now = 0;
-        return (split_py_long_to_s_and_ns(PyTuple_GET_ITEM(ns, 0),
-                    &(ua->atime_s), &(ua->atime_ns)))
-            && (split_py_long_to_s_and_ns(PyTuple_GET_ITEM(ns, 1),
-                    &(ua->mtime_s), &(ua->mtime_ns)));
+        if (!split_py_long_to_s_and_ns(PyTuple_GET_ITEM(ns, 0),
+                                      &ua->atime_s, &ua->atime_ns) ||
+            !split_py_long_to_s_and_ns(PyTuple_GET_ITEM(ns, 1),
+                                       &ua->mtime_s, &ua->mtime_ns))
+            goto fail;
+        return 1;
     }
 
     /* either times=None, or neither times nor ns was specified. use "now". */
     ua->now = 1;
     return 1;
+
+  fail:
+    if (ua->converter)
+        Py_DECREF(ua->path);
+    return 0;
 }