From 399785acf166dac15123889c28974ff06d6d2de0 Mon Sep 17 00:00:00 2001 From: Derrick Brashear Date: Wed, 18 Aug 2004 23:46:37 +0000 Subject: [PATCH] attachserverproc-dont-dequeue-if-not-in-queue-take-two-20040818 nWaiting ended up negative, because the dequeuer (GetCall) decremented. --- src/rx/rx.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/rx/rx.c b/src/rx/rx.c index 303c81bfa0..4281cc80aa 100644 --- a/src/rx/rx.c +++ b/src/rx/rx.c @@ -3996,11 +3996,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;