From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicholas Krause Date: Fri, 2 Jan 2015 02:34:01 -0500 Subject: [PATCH] drivers:net:wireless: Add proper locking for the function, b43_op_beacon_set_tim in main.c Message-ID: <1420184041-6788-1-git-send-email-xerofoify@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: stefano.brivio-hl5o88x/ua9eoWH0uzbU5w@public.gmane.org Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, b43-dev-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, kvalo-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org This adds proper locking for the function, b43_op_beacon_set_tim in main.c by using the mutex lock in the structure pointer wl, as embedded into this pointer as a mutex in order to protect against multiple access to the pointer wl when updating the templates for this pointer in the function, b43_update_templates internally in the function, b43_op_beacon_set_tim. Signed-off-by: Nicholas Krause --- drivers/net/wireless/b43/main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c index 47731cb..d568fc8 100644 --- a/drivers/net/wireless/b43/main.c +++ b/drivers/net/wireless/b43/main.c @@ -5094,8 +5094,9 @@ static int b43_op_beacon_set_tim(struct ieee80211_hw *hw, { struct b43_wl *wl = hw_to_b43_wl(hw); - /* FIXME: add locking */ + mutex_lock(&wl->mutex); b43_update_templates(wl); + mutex_unlock(&wl->mutex); return 0; } -- 2.1.0