mwifiex: handle race during mwifiex_usb_disconnect
authorGanapathi Bhat <gbhat@marvell.com>
Thu, 24 May 2018 13:48:27 +0000 (19:18 +0530)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 29 May 2018 07:22:57 +0000 (10:22 +0300)
commitb817047ae70c0bd67b677b65d0d69d72cd6e9728
treea526dc62165ef7cd4b4c89e19ce569665d704b4d
parent618fd1ed17d4cf193d85747c04a8b836b8fc107e
mwifiex: handle race during mwifiex_usb_disconnect

Race condition is observed during rmmod of mwifiex_usb:

1. The rmmod thread will call mwifiex_usb_disconnect(), download
   SHUTDOWN command and do wait_event_interruptible_timeout(),
   waiting for response.

2. The main thread will handle the response and will do a
   wake_up_interruptible(), unblocking rmmod thread.

3. On getting unblocked, rmmod thread  will make rx_cmd.urb = NULL in
   mwifiex_usb_free().

4. The main thread will try to resubmit rx_cmd.urb in
   mwifiex_usb_submit_rx_urb(), which is NULL.

To fix, wait for main thread to complete before calling
mwifiex_usb_free().

Signed-off-by: Ganapathi Bhat <gbhat@marvell.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/marvell/mwifiex/usb.c