From: Aram Sargsyan <aram@isc.org>
Date: Tue, 7 Jan 2025 13:30:26 +0000 (+0000)
Subject: Fix a bug in isc_rwlock_trylock()
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=aef3f26aaf88a00c5d5a0688a892a728d231968b;p=thirdparty%2Fbind9.git

Fix a bug in isc_rwlock_trylock()

When isc_rwlock_trylock() fails to get a read lock because another
writer was faster, it should wake up other waiting writers in case
there are no other readers, but the current code forgets about
the currently active writer when evaluating 'cntflag'.

Unset the WRITER_ACTIVE bit in 'cntflag' before checking to see if
there are other readers, otherwise the waiting writers, if they exist,
might not wake up.

(cherry picked from commit 73b6d9e9e5c3405b8f2cd6ab1e999e2a5d756875)
---

diff --git a/lib/isc/rwlock.c b/lib/isc/rwlock.c
index c69b6d7550b..aa1156555db 100644
--- a/lib/isc/rwlock.c
+++ b/lib/isc/rwlock.c
@@ -475,7 +475,8 @@ isc_rwlock_trylock(isc_rwlock_t *rwl, isc_rwlocktype_t type) {
 			 * request.
 			 */
 			cntflag = atomic_fetch_sub_release(&rwl->cnt_and_flag,
-							   READER_INCR);
+							   READER_INCR) &
+				  ~WRITER_ACTIVE;
 			/*
 			 * If no other readers are waiting and we've suspended
 			 * new writers in this short period, wake them up.