diff options
author | Jiri Pirko <jpirko@redhat.com> | 2011-01-25 11:03:25 +0000 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-01-25 13:13:16 -0800 |
commit | 672bda337060fa2ff99866a6ebfa3ae036f8b23b (patch) | |
tree | 3dde2d776d9a38494790a9d8c6eeb593676218b7 /drivers/net/bonding | |
parent | a512b92b3af4b03fc6834617a042dc85fbd4e34e (diff) | |
download | op-kernel-dev-672bda337060fa2ff99866a6ebfa3ae036f8b23b.zip op-kernel-dev-672bda337060fa2ff99866a6ebfa3ae036f8b23b.tar.gz |
bonding: fix return value of couple of store functions
count is incorrectly returned even in case of fail. Return ret instead.
Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding')
-rw-r--r-- | drivers/net/bonding/bond_sysfs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/bonding/bond_sysfs.c b/drivers/net/bonding/bond_sysfs.c index 8fd0174..72bb0f6 100644 --- a/drivers/net/bonding/bond_sysfs.c +++ b/drivers/net/bonding/bond_sysfs.c @@ -1198,7 +1198,7 @@ static ssize_t bonding_store_carrier(struct device *d, bond->dev->name, new_value); } out: - return count; + return ret; } static DEVICE_ATTR(use_carrier, S_IRUGO | S_IWUSR, bonding_show_carrier, bonding_store_carrier); @@ -1595,7 +1595,7 @@ static ssize_t bonding_store_slaves_active(struct device *d, } } out: - return count; + return ret; } static DEVICE_ATTR(all_slaves_active, S_IRUGO | S_IWUSR, bonding_show_slaves_active, bonding_store_slaves_active); |