diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-11-01 06:18:44 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-11-02 02:08:54 -0400 |
commit | e139862eeec985d7139b11b09deeb9a32e3f3af2 (patch) | |
tree | 4dba5afafc219812e45e2659c57847802c57e0f3 | |
parent | f99f4f8b094065601f87953b6d372d1e082bdf8b (diff) |
bonding: bond_get_size() returns wrong size
There is an extra semi-colon so bond_get_size() doesn't return the
correct value.
Fixes: ec76aa49855f ('bonding: add Netlink support active_slave option')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Veaceslav Falico <vfalico@redhat.com>
Reviewed-by: Jiri Pirko <jiri@resnulli.us>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/bonding/bond_netlink.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/bonding/bond_netlink.c b/drivers/net/bonding/bond_netlink.c index 7661261de2f0..40e7b1cb4aea 100644 --- a/drivers/net/bonding/bond_netlink.c +++ b/drivers/net/bonding/bond_netlink.c | |||
@@ -82,8 +82,8 @@ static int bond_newlink(struct net *src_net, struct net_device *bond_dev, | |||
82 | 82 | ||
83 | static size_t bond_get_size(const struct net_device *bond_dev) | 83 | static size_t bond_get_size(const struct net_device *bond_dev) |
84 | { | 84 | { |
85 | return nla_total_size(sizeof(u8)); /* IFLA_BOND_MODE */ | 85 | return nla_total_size(sizeof(u8)) + /* IFLA_BOND_MODE */ |
86 | + nla_total_size(sizeof(u32)); /* IFLA_BOND_ACTIVE_SLAVE */ | 86 | nla_total_size(sizeof(u32)); /* IFLA_BOND_ACTIVE_SLAVE */ |
87 | } | 87 | } |
88 | 88 | ||
89 | static int bond_fill_info(struct sk_buff *skb, | 89 | static int bond_fill_info(struct sk_buff *skb, |