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/bond_sysfs.c | |
| parent | net: add sysfs entry for device group (diff) | |
| download | linux-dev-672bda337060fa2ff99866a6ebfa3ae036f8b23b.tar.xz linux-dev-672bda337060fa2ff99866a6ebfa3ae036f8b23b.zip | |
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 '')
| -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 8fd0174c5380..72bb0f6cc9bf 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); |