diff options
author | Tim Golden <mail@timgolden.me.uk> | 2013-11-12 12:36:54 (GMT) |
---|---|---|
committer | Tim Golden <mail@timgolden.me.uk> | 2013-11-12 12:36:54 (GMT) |
commit | 6e51b8ff0f1a2210548b3ce18e6ba840e0a3c8ae (patch) | |
tree | c3614c7ce4a2dafe20a429928f151adc341edafa | |
parent | c12f09ed24db31f0fc0b58010cb3da9e3107e06c (diff) | |
download | cpython-6e51b8ff0f1a2210548b3ce18e6ba840e0a3c8ae.zip cpython-6e51b8ff0f1a2210548b3ce18e6ba840e0a3c8ae.tar.gz cpython-6e51b8ff0f1a2210548b3ce18e6ba840e0a3c8ae.tar.bz2 |
Issue13674 Correct crash with strftime %y format under Windows
-rw-r--r-- | Lib/test/test_strftime.py | 26 | ||||
-rw-r--r-- | Modules/timemodule.c | 7 |
2 files changed, 32 insertions, 1 deletions
diff --git a/Lib/test/test_strftime.py b/Lib/test/test_strftime.py index 14057eb..78c9c5b 100644 --- a/Lib/test/test_strftime.py +++ b/Lib/test/test_strftime.py @@ -176,8 +176,32 @@ class StrftimeTest(unittest.TestCase): (e[0], e[2])) print(" Expected %s, but got %s" % (e[1], result)) + +class Y1900Tests(unittest.TestCase): + """A limitation of the MS C runtime library is that it crashes if + a date before 1900 is passed with a format string containing "%y" + """ + + @unittest.skipUnless(sys.platform == "win32", "Only applies to Windows") + def test_y_before_1900_win(self): + with self.assertRaises(ValueError): + time.strftime("%y", (1899, 1, 1, 0, 0, 0, 0, 0, 0)) + + @unittest.skipIf(sys.platform == "win32", "Doesn't apply on Windows") + def test_y_before_1900_nonwin(self): + self.assertEquals( + time.strftime("%y", (1899, 1, 1, 0, 0, 0, 0, 0, 0)), "99") + + def test_y_1900(self): + self.assertEquals( + time.strftime("%y", (1900, 1, 1, 0, 0, 0, 0, 0, 0)), "00") + + def test_y_after_1900(self): + self.assertEquals( + time.strftime("%y", (2013, 1, 1, 0, 0, 0, 0, 0, 0)), "13") + def test_main(): - support.run_unittest(StrftimeTest) + support.run_unittest(StrftimeTest, Y1900Tests) if __name__ == '__main__': test_main() diff --git a/Modules/timemodule.c b/Modules/timemodule.c index 03476d9..42ec366 100644 --- a/Modules/timemodule.c +++ b/Modules/timemodule.c @@ -632,6 +632,13 @@ time_strftime(PyObject *self, PyObject *args) Py_DECREF(format); return NULL; } + if ((outbuf[1] == 'y') && buf.tm_year < 0) + { + PyErr_SetString(PyExc_ValueError, + "format %y requires year >= 1900 on Windows"); + Py_DECREF(format); + return NULL; + } } #endif |