diff options
author | Michael Buesch <mb@bu3sch.de> | 2009-09-14 17:22:08 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-09-23 11:35:45 -0400 |
commit | e899a3f232125c393cdf7f7bf6533501ef8808fb (patch) | |
tree | bf6dae95b5e9da612ccdc24f56807c66ca310d0a /drivers/net/wireless/b43/main.c | |
parent | 9308779aad721cab8434fd5f98d3c85d3716fdec (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.c | 2 |
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 | } |