aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net/wireless/b43/main.c
diff options
context:
space:
mode:
authorMichael Buesch <mb@bu3sch.de>2009-09-14 17:22:08 -0400
committerJohn W. Linville <linville@tuxdriver.com>2009-09-23 11:35:45 -0400
commite899a3f232125c393cdf7f7bf6533501ef8808fb (patch)
treebf6dae95b5e9da612ccdc24f56807c66ca310d0a /drivers/net/wireless/b43/main.c
parent9308779aad721cab8434fd5f98d3c85d3716fdec (diff)
b43: Don't abuse wl->current_dev in the led work
Don't abuse wl->current_dev in the LED work for checking whether we're going down. Add an explicit variable. This fixes a crash on rmmod dereferencing the wl->current_dev NULL pointer in various other places of the driver. Signed-off-by: Michael Buesch <mb@bu3sch.de> Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/b43/main.c')
-rw-r--r--drivers/net/wireless/b43/main.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
index 7fa947ca7c32..a8840ef63770 100644
--- a/drivers/net/wireless/b43/main.c
+++ b/drivers/net/wireless/b43/main.c
@@ -4987,7 +4987,7 @@ static void b43_remove(struct ssb_device *dev)
4987 * might have modified it. Restoring is important, so the networking 4987 * might have modified it. Restoring is important, so the networking
4988 * stack can properly free resources. */ 4988 * stack can properly free resources. */
4989 wl->hw->queues = wl->mac80211_initially_registered_queues; 4989 wl->hw->queues = wl->mac80211_initially_registered_queues;
4990 wl->current_dev = NULL; 4990 wl->leds.stop = 1;
4991 cancel_work_sync(&wl->leds.work); 4991 cancel_work_sync(&wl->leds.work);
4992 ieee80211_unregister_hw(wl->hw); 4992 ieee80211_unregister_hw(wl->hw);
4993 } 4993 }