May 22, 2025 · May 20, 2025 · May 20, 2025 · May 20, 2025 · May 20, 2025 · May 20, 2025
diff --git a/Lib/test/lock_tests.py b/Lib/test/lock_tests.py
        lock.release()
        self.assertFalse(lock.locked())

    def test_locked_2threads(self):
        l = []
        rlock = self.locktype()
        def acquire():
            rlock.acquire()
            l.append(rlock.locked())

        with Bunch(acquire, 1):
            pass
        self.assertTrue(rlock.locked())
        self.assertTrue(l[0])
        del rlock

    def test_release_save_unacquired(self):
        # Cannot _release_save an unacquired lock
        lock = self.locktype()
diff --git a/Lib/threading.py b/Lib/threading.py

    def locked(self):
        """Return whether this object is locked."""
        return self._count > 0
        return self._block.locked()

    # Internal methods used by condition variables

diff --git a/Misc/NEWS.d/next/Library/2025-05-20-19-16-30.gh-issue-134323.ZQZGvw.rst b/Misc/NEWS.d/next/Library/2025-05-20-19-16-30.gh-issue-134323.ZQZGvw.rst
 Fix the public :meth:`locked<threading.RLock>` and its implementation in the ``_thread`` module.
 Also adding a unit test in ``lock_test.py`` file.
diff --git a/Modules/_threadmodule.c b/Modules/_threadmodule.c
 static PyObject *
 rlock_locked(PyObject *op, PyObject *Py_UNUSED(ignored))
 {
    rlockobject *self = rlockobject_CAST(op);
    int is_locked = _PyRecursiveMutex_IsLockedByCurrentThread(&self->lock);
    /*
    see gh-134323: the `_PyRecursiveMutex_IsLocked` function does not exist, so we cast the `op`
    to `lockobject` in order to call `PyMutex_IsLocked`.
    */
    lockobject *self = lockobject_CAST(op);
    int is_locked = PyMutex_IsLocked(&self->lock);
    return PyBool_FromLong(is_locked);
 }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -370,6 +370,19 @@ def test_locked(self):
		lock.release()
		self.assertFalse(lock.locked())

		def test_locked_2threads(self):
YvesDup marked this conversation as resolved. Show resolvedHide resolved
		l = []
YvesDup marked this conversation as resolved. Show resolvedHide resolved
		rlock = self.locktype()
YvesDup marked this conversation as resolved. Show resolvedHide resolved
		def acquire():
		rlock.acquire()
YvesDup marked this conversation as resolved. Show resolvedHide resolved
		l.append(rlock.locked())

		with Bunch(acquire, 1):
		pass
		self.assertTrue(rlock.locked())
		self.assertTrue(l[0])
		del rlock
YvesDup marked this conversation as resolved. Show resolvedHide resolved

		def test_release_save_unacquired(self):
		# Cannot _release_save an unacquired lock
		lock = self.locktype()
Expand Down
Original file line number	Diff line number	Diff line change
Expand Up		@@ -237,7 +237,7 @@ def __exit__(self, t, v, tb):

		def locked(self):
		"""Return whether this object is locked."""
		return self._count > 0
		return self._block.locked()
YvesDup marked this conversation as resolved. Show resolvedHide resolved

		# Internal methods used by condition variables

Expand Down
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Fix the public :meth:`locked<threading.RLock>` and its implementation in the ``_thread`` module.
YvesDup marked this conversation as resolved. Show resolvedHide resolved
		Also adding a unit test in ``lock_test.py`` file.
YvesDup marked this conversation as resolved. Show resolvedHide resolved
Original file line number	Diff line number	Diff line change
Expand Up		@@ -1110,8 +1110,12 @@ Release the lock.");
		static PyObject *
		rlock_locked(PyObject op, PyObject Py_UNUSED(ignored))
		{
		rlockobject *self = rlockobject_CAST(op);
		int is_locked = _PyRecursiveMutex_IsLockedByCurrentThread(&self->lock);
		/*
		see gh-134323: the `_PyRecursiveMutex_IsLocked` function does not exist, so we cast the `op`
		to `lockobject` in order to call `PyMutex_IsLocked`.
YvesDup marked this conversation as resolved. Show resolvedHide resolved
		*/
		lockobject *self = lockobject_CAST(op);
		int is_locked = PyMutex_IsLocked(&self->lock);
YvesDup marked this conversation as resolved. Show resolvedHide resolved YvesDup marked this conversation as resolved. Show resolvedHide resolved
		return PyBool_FromLong(is_locked);
		}

Expand Down