diff options
author | Jay Vosburgh <fubar@us.ibm.com> | 2006-09-23 06:53:25 +0200 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2006-09-26 02:08:09 +0200 |
commit | a50d8de2cc872818b61e60c20c75be3f19aa6887 (patch) | |
tree | 9ea764b51534cc2a85111515d8dd95da419c0ea4 | |
parent | [PATCH] bonding: Format fix in seq_printf call (diff) | |
download | linux-a50d8de2cc872818b61e60c20c75be3f19aa6887.tar.xz linux-a50d8de2cc872818b61e60c20c75be3f19aa6887.zip |
[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 cfe4dc3a93a3..15b6a29bb4d4 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", |