From: Dan Carpenter Date: Fri, 1 Nov 2013 10:18:44 +0000 (+0300) Subject: bonding: bond_get_size() returns wrong size X-Git-Tag: upstream/snapshot3+hdmi~3994^2~78 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e139862eeec985d7139b11b09deeb9a32e3f3af2;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git 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 Acked-by: Veaceslav Falico Reviewed-by: Jiri Pirko Signed-off-by: David S. Miller --- diff --git a/drivers/net/bonding/bond_netlink.c b/drivers/net/bonding/bond_netlink.c index 7661261..40e7b1c 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, static size_t bond_get_size(const struct net_device *bond_dev) { - return nla_total_size(sizeof(u8)); /* IFLA_BOND_MODE */ - + nla_total_size(sizeof(u32)); /* IFLA_BOND_ACTIVE_SLAVE */ + return nla_total_size(sizeof(u8)) + /* IFLA_BOND_MODE */ + nla_total_size(sizeof(u32)); /* IFLA_BOND_ACTIVE_SLAVE */ } static int bond_fill_info(struct sk_buff *skb,