]> granicus.if.org Git - python/commitdiff
[2.7] bpo-30807: signal.setitimer() may disable the timer by mistake (GH-2493) (...
authorAntoine Pitrou <pitrou@free.fr>
Fri, 30 Jun 2017 08:54:54 +0000 (10:54 +0200)
committerGitHub <noreply@github.com>
Fri, 30 Jun 2017 08:54:54 +0000 (10:54 +0200)
* bpo-30807: signal.setitimer() may disable the timer by mistake

* Add NEWS blurb.
(cherry picked from commit 729780a810bbcb12b245a1b652302a601fc9f6fd)

Lib/test/test_signal.py
Misc/NEWS.d/next/Library/2017-06-29-22-04-44.bpo-30807.sLtjY-.rst [new file with mode: 0644]
Modules/signalmodule.c

index 7483f64123ed1bb210afc2a5dc8e320b50d71e2b..27054f17ba7085e97050738383d38be8b816fd47 100644 (file)
@@ -490,6 +490,16 @@ class ItimerTest(unittest.TestCase):
         # and the handler should have been called
         self.assertEqual(self.hndl_called, True)
 
+    def test_setitimer_tiny(self):
+        # bpo-30807: C setitimer() takes a microsecond-resolution interval.
+        # Check that float -> timeval conversion doesn't round
+        # the interval down to zero, which would disable the timer.
+        self.itimer = signal.ITIMER_REAL
+        signal.setitimer(self.itimer, 1e-6)
+        time.sleep(1)
+        self.assertEqual(self.hndl_called, True)
+
+
 def test_main():
     test_support.run_unittest(BasicSignalTests, InterProcessSignalTests,
                               WakeupFDTests, WakeupSignalTests,
diff --git a/Misc/NEWS.d/next/Library/2017-06-29-22-04-44.bpo-30807.sLtjY-.rst b/Misc/NEWS.d/next/Library/2017-06-29-22-04-44.bpo-30807.sLtjY-.rst
new file mode 100644 (file)
index 0000000..ce6f48a
--- /dev/null
@@ -0,0 +1,6 @@
+signal.setitimer() may disable the timer when passed a tiny value.
+
+Tiny values (such as 1e-6) are valid non-zero values for setitimer(), which
+is specified as taking microsecond-resolution intervals. However, on some
+platform, our conversion routine could convert 1e-6 into a zero interval,
+therefore disabling the timer instead of (re-)scheduling it.
index 1d7ba4ba48af95a82daa4da4b83836baa5084c6c..c7bf1f0853438076b6fc3c662f5e6c50de76f088 100644 (file)
@@ -111,6 +111,10 @@ timeval_from_double(double d, struct timeval *tv)
 {
     tv->tv_sec = floor(d);
     tv->tv_usec = fmod(d, 1.0) * 1000000.0;
+    /* Don't disable the timer if the computation above rounds down to zero. */
+    if (d > 0.0 && tv->tv_sec == 0 && tv->tv_usec == 0) {
+        tv->tv_usec = 1;
+    }
 }
 
 Py_LOCAL_INLINE(double)