]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
gh-140634: Fix a reference counting bug in os.sched_param.__reduce__() (GH-140667)
authorSerhiy Storchaka <storchaka@gmail.com>
Mon, 27 Oct 2025 19:42:32 +0000 (21:42 +0200)
committerGitHub <noreply@github.com>
Mon, 27 Oct 2025 19:42:32 +0000 (21:42 +0200)
Lib/test/test_os/test_posix.py
Misc/NEWS.d/next/Library/2025-10-27-13-49-31.gh-issue-140634.ULng9G.rst [new file with mode: 0644]
Modules/posixmodule.c

index de24719a1ca455eee7b10577c7b6ee7dc7946e15..37da293a441e46c7b6dc6eca82d7e3d478676baa 100644 (file)
@@ -1427,6 +1427,14 @@ class PosixTester(unittest.TestCase):
         self.assertNotEqual(newparam, param)
         self.assertEqual(newparam.sched_priority, 0)
 
+    @requires_sched
+    def test_bug_140634(self):
+        sched_priority = float('inf')  # any new reference
+        param = posix.sched_param(sched_priority)
+        param.__reduce__()
+        del sched_priority, param  # should not crash
+        support.gc_collect()  # just to be sure
+
     @unittest.skipUnless(hasattr(posix, "sched_rr_get_interval"), "no function")
     def test_sched_rr_get_interval(self):
         try:
diff --git a/Misc/NEWS.d/next/Library/2025-10-27-13-49-31.gh-issue-140634.ULng9G.rst b/Misc/NEWS.d/next/Library/2025-10-27-13-49-31.gh-issue-140634.ULng9G.rst
new file mode 100644 (file)
index 0000000..b1ba9b2
--- /dev/null
@@ -0,0 +1 @@
+Fix a reference counting bug in :meth:`!os.sched_param.__reduce__`.
index a30712f75d5d06f14a4bd107a34af5e8cd61edb0..50464b01efba31818a03ee7438e99f9cd1ce3c0b 100644 (file)
@@ -8724,7 +8724,7 @@ os_sched_param_impl(PyTypeObject *type, PyObject *sched_priority)
 static PyObject *
 os_sched_param_reduce(PyObject *self, PyObject *Py_UNUSED(dummy))
 {
-    return Py_BuildValue("(O(N))", Py_TYPE(self), PyStructSequence_GetItem(self, 0));
+    return Py_BuildValue("(O(O))", Py_TYPE(self), PyStructSequence_GetItem(self, 0));
 }
 
 static PyMethodDef os_sched_param_reduce_method = {