From: Andrew Deason <adeason@sinenomine.net>
Date: Wed, 14 Nov 2012 05:30:41 +0000 (-0600)
Subject: rx: Lock call for KeepAliveOn/KeepAliveOff
X-Git-Tag: upstream/1.8.0_pre1^2~1776
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=866c05825c9964aa92740bcb0f20f26b451e65ea;p=packages%2Fo%2Fopenafs.git

rx: Lock call for KeepAliveOn/KeepAliveOff

rxi_KeepAliveOn/Off expect the call lock to be held after the call has
been initialized. So, hold it in the rx_KeepAliveOn/Off callers.

Change-Id: Ic2e96853bc9553e97f62e4502a5c92bbb28c63a2
Reviewed-on: http://gerrit.openafs.org/8463
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Simon Wilkinson <simonxwilkinson@gmail.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
---

diff --git a/src/rx/rx.c b/src/rx/rx.c
index 1a08e305b..5b560d1f8 100644
--- a/src/rx/rx.c
+++ b/src/rx/rx.c
@@ -6652,12 +6652,16 @@ rxi_KeepAliveOn(struct rx_call *call)
 void
 rx_KeepAliveOff(struct rx_call *call)
 {
+    MUTEX_ENTER(&call->lock);
     rxi_KeepAliveOff(call);
+    MUTEX_EXIT(&call->lock);
 }
 void
 rx_KeepAliveOn(struct rx_call *call)
 {
+    MUTEX_ENTER(&call->lock);
     rxi_KeepAliveOn(call);
+    MUTEX_EXIT(&call->lock);
 }
 
 static void