summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRichard Oudkerk <shibturn@gmail.com>2013-04-17 20:16:52 (GMT)
committerRichard Oudkerk <shibturn@gmail.com>2013-04-17 20:16:52 (GMT)
commit0471ac3d80cb9acdfc82b053145477f5f1e7c6f1 (patch)
tree13f4d83ad674abf729dc4c01e7183c2b92aac1a2
parent409c31390f839debb2f05cef691e8b59e52bd524 (diff)
parentcf8a1e51ece7cad7096963927993ddfa738627e6 (diff)
downloadcpython-0471ac3d80cb9acdfc82b053145477f5f1e7c6f1.zip
cpython-0471ac3d80cb9acdfc82b053145477f5f1e7c6f1.tar.gz
cpython-0471ac3d80cb9acdfc82b053145477f5f1e7c6f1.tar.bz2
Merge.
-rw-r--r--Misc/NEWS3
-rw-r--r--Modules/posixmodule.c10
-rw-r--r--Modules/signalmodule.c6
-rw-r--r--Modules/timemodule.c6
4 files changed, 21 insertions, 4 deletions
diff --git a/Misc/NEWS b/Misc/NEWS
index 5a8c0b9..de1bd88 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -12,6 +12,9 @@ What's New in Python 3.3.2?
Core and Builtins
-----------------
+- Issue #17782: Fix undefined behaviour on platforms where
+ ``struct timespec``'s "tv_nsec" member is not a C long.
+
- Issue #17715: Fix segmentation fault from raising an exception in a __trunc__
method.
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c
index 91352d4..551af2f 100644
--- a/Modules/posixmodule.c
+++ b/Modules/posixmodule.c
@@ -4775,6 +4775,8 @@ posix_utime(PyObject *self, PyObject *args, PyObject *kwargs)
}
if (times && (times != Py_None)) {
+ time_t a_sec, m_sec;
+ long a_nsec, m_nsec;
if (!PyTuple_CheckExact(times) || (PyTuple_Size(times) != 2)) {
PyErr_SetString(PyExc_TypeError,
"utime: 'times' must be either"
@@ -4783,11 +4785,15 @@ posix_utime(PyObject *self, PyObject *args, PyObject *kwargs)
}
utime.now = 0;
if (_PyTime_ObjectToTimespec(PyTuple_GET_ITEM(times, 0),
- &utime.atime_s, &utime.atime_ns) == -1 ||
+ &a_sec, &a_nsec) == -1 ||
_PyTime_ObjectToTimespec(PyTuple_GET_ITEM(times, 1),
- &utime.mtime_s, &utime.mtime_ns) == -1) {
+ &m_sec, &m_nsec) == -1) {
goto exit;
}
+ utime.atime_s = a_sec;
+ utime.atime_ns = a_nsec;
+ utime.mtime_s = m_sec;
+ utime.mtime_ns = m_nsec;
}
else if (ns) {
if (!PyTuple_CheckExact(ns) || (PyTuple_Size(ns) != 2)) {
diff --git a/Modules/signalmodule.c b/Modules/signalmodule.c
index 41acbfa..0cc7237 100644
--- a/Modules/signalmodule.c
+++ b/Modules/signalmodule.c
@@ -784,14 +784,18 @@ signal_sigtimedwait(PyObject *self, PyObject *args)
struct timespec buf;
sigset_t set;
siginfo_t si;
+ time_t tv_sec;
+ long tv_nsec;
int err;
if (!PyArg_ParseTuple(args, "OO:sigtimedwait",
&signals, &timeout))
return NULL;
- if (_PyTime_ObjectToTimespec(timeout, &buf.tv_sec, &buf.tv_nsec) == -1)
+ if (_PyTime_ObjectToTimespec(timeout, &tv_sec, &tv_nsec) == -1)
return NULL;
+ buf.tv_sec = tv_sec;
+ buf.tv_nsec = tv_nsec;
if (buf.tv_sec < 0 || buf.tv_nsec < 0) {
PyErr_SetString(PyExc_ValueError, "timeout must be non-negative");
diff --git a/Modules/timemodule.c b/Modules/timemodule.c
index 161407d..5a0f378 100644
--- a/Modules/timemodule.c
+++ b/Modules/timemodule.c
@@ -195,14 +195,18 @@ time_clock_settime(PyObject *self, PyObject *args)
{
int clk_id;
PyObject *obj;
+ time_t tv_sec;
+ long tv_nsec;
struct timespec tp;
int ret;
if (!PyArg_ParseTuple(args, "iO:clock_settime", &clk_id, &obj))
return NULL;
- if (_PyTime_ObjectToTimespec(obj, &tp.tv_sec, &tp.tv_nsec) == -1)
+ if (_PyTime_ObjectToTimespec(obj, &tv_sec, &tv_nsec) == -1)
return NULL;
+ tp.tv_sec = tv_sec;
+ tp.tv_nsec = tv_nsec;
ret = clock_settime((clockid_t)clk_id, &tp);
if (ret != 0) {