summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPiotr Haber <phaber@broadcom.com>2012-10-11 14:05:15 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2012-10-31 10:03:02 -0700
commit44b977a87305bb075adb5b764f115237e474e626 (patch)
tree155da2470deac05a0469e7856e06d22e41f483bf
parent7e8cec32110be45739569e04f35d83d47064dd86 (diff)
downloadlinux-3.10-44b977a87305bb075adb5b764f115237e474e626.tar.gz
linux-3.10-44b977a87305bb075adb5b764f115237e474e626.tar.bz2
linux-3.10-44b977a87305bb075adb5b764f115237e474e626.zip
bcma: fix unregistration of cores
commit 1fffa905adffbf0d3767fc978ef09afb830275eb upstream. When cores are unregistered, entries need to be removed from cores list in a safe manner. Reported-by: Stanislaw Gruszka <sgruszka@redhat.com> Reviewed-by: Arend Van Spriel <arend@broadcom.com> Signed-off-by: Piotr Haber <phaber@broadcom.com> Signed-off-by: John W. Linville <linville@tuxdriver.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/bcma/main.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/bcma/main.c b/drivers/bcma/main.c
index 7e138ec2135..edb9233b26d 100644
--- a/drivers/bcma/main.c
+++ b/drivers/bcma/main.c
@@ -131,9 +131,10 @@ static int bcma_register_cores(struct bcma_bus *bus)
static void bcma_unregister_cores(struct bcma_bus *bus)
{
- struct bcma_device *core;
+ struct bcma_device *core, *tmp;
- list_for_each_entry(core, &bus->cores, list) {
+ list_for_each_entry_safe(core, tmp, &bus->cores, list) {
+ list_del(&core->list);
if (core->dev_registered)
device_unregister(&core->dev);
}