Commit e911a819 authored by Johannes Berg's avatar Johannes Berg

wifi: mac80211: lock wiphy for aggregation debugfs

To change aggregation status may call into the driver,
lock the wiphy for this.
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent 0ab6cba0
...@@ -5,7 +5,7 @@ ...@@ -5,7 +5,7 @@
* Copyright 2007 Johannes Berg <johannes@sipsolutions.net> * Copyright 2007 Johannes Berg <johannes@sipsolutions.net>
* Copyright 2013-2014 Intel Mobile Communications GmbH * Copyright 2013-2014 Intel Mobile Communications GmbH
* Copyright(c) 2016 Intel Deutschland GmbH * Copyright(c) 2016 Intel Deutschland GmbH
* Copyright (C) 2018 - 2022 Intel Corporation * Copyright (C) 2018 - 2023 Intel Corporation
*/ */
#include <linux/debugfs.h> #include <linux/debugfs.h>
...@@ -420,6 +420,7 @@ static ssize_t sta_agg_status_write(struct file *file, const char __user *userbu ...@@ -420,6 +420,7 @@ static ssize_t sta_agg_status_write(struct file *file, const char __user *userbu
if (ret || tid >= IEEE80211_NUM_TIDS) if (ret || tid >= IEEE80211_NUM_TIDS)
return -EINVAL; return -EINVAL;
wiphy_lock(sta->local->hw.wiphy);
if (tx) { if (tx) {
if (start) if (start)
ret = ieee80211_start_tx_ba_session(&sta->sta, tid, ret = ieee80211_start_tx_ba_session(&sta->sta, tid,
...@@ -431,6 +432,7 @@ static ssize_t sta_agg_status_write(struct file *file, const char __user *userbu ...@@ -431,6 +432,7 @@ static ssize_t sta_agg_status_write(struct file *file, const char __user *userbu
3, true); 3, true);
ret = 0; ret = 0;
} }
wiphy_unlock(sta->local->hw.wiphy);
return ret ?: count; return ret ?: count;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment