From: Antoine Pitrou <solipsis@pitrou.net>
Date: Thu, 6 Jan 2011 18:29:05 +0000 (+0000)
Subject: Merged revisions 87802 via svnmerge from
X-Git-Tag: v3.2.1b1~347^2~91
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=ff17385fbde2b4fcd5d309e14904a9a27a742b8c;p=thirdparty%2FPython%2Fcpython.git

Merged revisions 87802 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k

........
  r87802 | antoine.pitrou | 2011-01-06 19:25:55 +0100 (jeu., 06 janv. 2011) | 6 lines

  Issue #7858: Raise an error properly when os.utime() fails under Windows
  on an existing file.

  (this does not seem to be easily testable)
........
---

diff --git a/Misc/NEWS b/Misc/NEWS
index c3268558143e..4f01ca2c3fdc 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -27,6 +27,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #7858: Raise an error properly when os.utime() fails under Windows
+  on an existing file.
+
 - Issue #3839: wsgiref should not override a Content-Length header set by
   the application.  Initial patch by Clovis Fabricio.
 
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index cb9489802508..870c0ea85eaa 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -3031,6 +3031,7 @@ posix_utime(PyObject *self, PyObject *args)
            something is wrong with the file, when it also
            could be the time stamp that gives a problem. */
         win32_error("utime", NULL);
+        goto done;
     }
     Py_INCREF(Py_None);
     result = Py_None;