summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJay Vosburgh <fubar@us.ibm.com>2006-09-22 21:53:25 -0700
committerJeff Garzik <jeff@garzik.org>2006-09-25 20:08:09 -0400
commita50d8de2cc872818b61e60c20c75be3f19aa6887 (patch)
tree9ea764b51534cc2a85111515d8dd95da419c0ea4
parent65509645ae05886eccc81b8a453afea07f0eabb6 (diff)
downloadop-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.c2
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",
OpenPOWER on IntegriCloud