summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafael J. Wysocki <rjw@sisk.pl>2008-04-22 22:57:50 +0200
committerLinus Torvalds <torvalds@linux-foundation.org>2008-04-22 17:40:36 -0700
commit29591b92e19f409d5ad4c099c2b7b5ea56f50dfa (patch)
tree0e3625c691318934f97f384d0cac862b13b29719
parent08119e8966e993993d0ba92b2fba38c582c8f787 (diff)
downloadblackbird-op-linux-29591b92e19f409d5ad4c099c2b7b5ea56f50dfa.tar.gz
blackbird-op-linux-29591b92e19f409d5ad4c099c2b7b5ea56f50dfa.zip
bus_remove_device: be more careful about incomplete initialization
Prevent bus_remove_device() from crashing if dev->knode_bus has not been initialized before it's called. This can happen if the device_add() ended up breaking out early due to an error, for example. Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl> Acked-by: Greg Kroah-Hartman <gregkh@suse.de> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--drivers/base/bus.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/base/bus.c b/drivers/base/bus.c
index be1cc5143354..ef522ae55480 100644
--- a/drivers/base/bus.c
+++ b/drivers/base/bus.c
@@ -530,7 +530,8 @@ void bus_remove_device(struct device *dev)
sysfs_remove_link(&dev->bus->p->devices_kset->kobj,
dev->bus_id);
device_remove_attrs(dev->bus, dev);
- klist_del(&dev->knode_bus);
+ if (klist_node_attached(&dev->knode_bus))
+ klist_del(&dev->knode_bus);
pr_debug("bus: '%s': remove device %s\n",
dev->bus->name, dev->bus_id);
OpenPOWER on IntegriCloud