aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarcelo Tosatti <mtosatti@redhat.com>2007-05-25 16:28:20 -0400
committerJohn W. Linville <linville@tuxdriver.com>2007-06-11 14:28:42 -0400
commitd43fb8ee3dab261e475d4f5189cf86182139b7f4 (patch)
tree5fb03e34b001f3b0506ddf07ac640a9643787ced
parent0e37275e9af07be6aa796b3ee981054525980b74 (diff)
[PATCH] libertas: fix oops on rmmod
Use list_for_each_entry_safe, to protect against list_del(). Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r--drivers/net/wireless/libertas/if_usb.c7
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/net/wireless/libertas/if_usb.c b/drivers/net/wireless/libertas/if_usb.c
index 10698a1c1c3..de7a2e2babe 100644
--- a/drivers/net/wireless/libertas/if_usb.c
+++ b/drivers/net/wireless/libertas/if_usb.c
@@ -970,15 +970,12 @@ static int if_usb_init_module(void)
static void if_usb_exit_module(void)
{
- struct list_head *ptr;
- struct usb_card_rec *cardp;
+ struct usb_card_rec *cardp, *cardp_temp;
lbs_deb_enter(LBS_DEB_MAIN);
- list_for_each(ptr, &usb_devices) {
- cardp = list_entry(ptr, struct usb_card_rec, list);
+ list_for_each_entry_safe(cardp, cardp_temp, &usb_devices, list)
if_usb_reset_device((wlan_private *) cardp->priv);
- }
/* API unregisters the driver from USB subsystem */
usb_deregister(&if_usb_driver);