From: Derrick Brashear <shadow@dementia.org>
Date: Wed, 25 Aug 2004 08:13:09 +0000 (+0000)
Subject: STABLE14-attachserverproc-dont-dequeue-if-not-in-queue-take-two-20040818
X-Git-Tag: openafs-devel-1_3_71~11
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=5b60572c5d70ecf87a1e2727e038bf66951acf8d;p=packages%2Fo%2Fopenafs.git

STABLE14-attachserverproc-dont-dequeue-if-not-in-queue-take-two-20040818


nWaiting ended up negative, because the dequeuer (GetCall) decremented.


(cherry picked from commit 399785acf166dac15123889c28974ff06d6d2de0)
---

diff --git a/src/rx/rx.c b/src/rx/rx.c
index 892c2230a..2ae09e0ef 100644
--- a/src/rx/rx.c
+++ b/src/rx/rx.c
@@ -4009,11 +4009,12 @@ rxi_AttachServerProc(register struct rx_call *call,
 	if (call->flags & RX_CALL_WAIT_PROC) {
 	    /* Conservative:  I don't think this should happen */
 	    call->flags &= ~RX_CALL_WAIT_PROC;
-	    MUTEX_ENTER(&rx_stats_mutex);
-	    rx_nWaiting--;
-	    MUTEX_EXIT(&rx_stats_mutex);
-	    if (queue_IsOnQueue(call))
-	        queue_Remove(call);
+	    if (queue_IsOnQueue(call)) {
+		queue_Remove(call);
+		MUTEX_ENTER(&rx_stats_mutex);
+		rx_nWaiting--;
+		MUTEX_EXIT(&rx_stats_mutex);
+	    }
 	}
 	call->state = RX_STATE_ACTIVE;
 	call->mode = RX_MODE_RECEIVING;