diff options
| author | Jay Vosburgh <fubar@us.ibm.com> | 2006-09-22 21:53:25 -0700 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-09-25 20:08:09 -0400 |
| commit | a50d8de2cc872818b61e60c20c75be3f19aa6887 (patch) | |
| tree | 9ea764b51534cc2a85111515d8dd95da419c0ea4 | |
| parent | 65509645ae05886eccc81b8a453afea07f0eabb6 (diff) | |
| download | op-kernel-dev-a50d8de2cc872818b61e60c20c75be3f19aa6887.zip op-kernel-dev-a50d8de2cc872818b61e60c20c75be3f19aa6887.tar.gz | |
[PATCH] bonding: Remove unneeded NULL test
Remove unneeded test for NULL. Reported by Thomas Dillig
<tdillig@stanford.edu> and Isil Dillig <isil@stanford.edu> via Stephen
Hemminger <shemminger@osdl.org>.
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
| -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 cfe4dc3..15b6a29 100644 --- a/drivers/net/bonding/bond_sysfs.c +++ b/drivers/net/bonding/bond_sysfs.c @@ -1093,7 +1093,7 @@ static ssize_t bonding_store_active_slave(struct class_device *cd, const char *b strlen(slave->dev->name)) == 0) { old_active = bond->curr_active_slave; new_active = slave; - if (new_active && (new_active == old_active)) { + if (new_active == old_active) { /* do nothing */ printk(KERN_INFO DRV_NAME ": %s: %s is already the current active slave.\n", |
