summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKalle Valo <kalle.valo@nokia.com>2009-06-12 14:16:20 +0300
committerJohn W. Linville <linville@tuxdriver.com>2009-07-10 14:57:47 -0400
commitcee4fd2712a3db21f432bdff14e59aed160453b2 (patch)
tree92b6fb3e3d0b0d4bbea26e24bf4b4b9a70c02cab
parent6021b2895891b161f73ede9938c101234c63218e (diff)
downloadop-kernel-dev-cee4fd2712a3db21f432bdff14e59aed160453b2.zip
op-kernel-dev-cee4fd2712a3db21f432bdff14e59aed160453b2.tar.gz
wl12xx: protect wl12xx_op_set_rts_threshold()
The function doesn't lock the mutex before sending the acx. Signed-off-by: Kalle Valo <kalle.valo@nokia.com> Reviewed-by: Vidhya Govindan <vidhya.govindan@nokia.com> Signed-off-by: Luciano Coelho <luciano.coelho@nokia.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r--drivers/net/wireless/wl12xx/main.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/net/wireless/wl12xx/main.c b/drivers/net/wireless/wl12xx/main.c
index 8feba36..7ec1749 100644
--- a/drivers/net/wireless/wl12xx/main.c
+++ b/drivers/net/wireless/wl12xx/main.c
@@ -957,11 +957,15 @@ static int wl12xx_op_set_rts_threshold(struct ieee80211_hw *hw, u32 value)
struct wl12xx *wl = hw->priv;
int ret;
+ mutex_lock(&wl->mutex);
+
ret = wl12xx_acx_rts_threshold(wl, (u16) value);
if (ret < 0)
wl12xx_warning("wl12xx_op_set_rts_threshold failed: %d", ret);
+ mutex_unlock(&wl->mutex);
+
return ret;
}
OpenPOWER on IntegriCloud