diff options
author | Stephen Hemminger <shemminger@vyatta.com> | 2009-06-12 19:02:47 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-06-13 23:28:56 -0700 |
commit | 9e71626c1c23ec69372c43c6fe66c1171032bf42 (patch) | |
tree | 664e2c6d342c210a6f041b97ed9a3d9c1bc58f06 /drivers/net/bonding/bond_sysfs.c | |
parent | 7e0838404541d2758bee089632690aabd82f3d5d (diff) | |
download | op-kernel-dev-9e71626c1c23ec69372c43c6fe66c1171032bf42.zip op-kernel-dev-9e71626c1c23ec69372c43c6fe66c1171032bf42.tar.gz |
bonding: fix destructor
It is not safe to use a network device destructor that is a function in
the module, since it can be called after module is unloaded if sysfs
handle is open.
When eventually using netlink, the device cleanup code needs to be done
via uninit function.
Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding/bond_sysfs.c')
-rw-r--r-- | drivers/net/bonding/bond_sysfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_sysfs.c b/drivers/net/bonding/bond_sysfs.c index 8041b67..7235759 100644 --- a/drivers/net/bonding/bond_sysfs.c +++ b/drivers/net/bonding/bond_sysfs.c @@ -127,7 +127,7 @@ static ssize_t bonding_store_bonds(struct class *cls, const char *buffer, size_t printk(KERN_INFO DRV_NAME ": %s is being deleted...\n", bond->dev->name); - bond_destroy(bond); + unregister_netdevice(bond->dev); goto out_unlock; } |