ipv6: addrconf: add missing validate_link_af handler
authorDaniel Borkmann <daniel@iogearbox.net>
Thu, 5 Feb 2015 13:39:11 +0000 (14:39 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 6 Feb 2015 20:51:33 +0000 (12:51 -0800)
We still need a validate_link_af() handler with an appropriate nla policy,
similarly as we have in IPv4 case, otherwise size validations are not being
done properly in that case.

Fixes: f53adae4eae5 ("net: ipv6: add tokenized interface identifier support")
Fixes: bc91b0f07ada ("ipv6: addrconf: implement address generation modes")
Cc: Jiri Pirko <jiri@resnulli.us>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Jiri Pirko <jiri@resnulli.us>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/addrconf.c

index 62900ae..754e683 100644 (file)
@@ -4587,6 +4587,22 @@ static int inet6_set_iftoken(struct inet6_dev *idev, struct in6_addr *token)
        return 0;
 }
 
+static const struct nla_policy inet6_af_policy[IFLA_INET6_MAX + 1] = {
+       [IFLA_INET6_ADDR_GEN_MODE]      = { .type = NLA_U8 },
+       [IFLA_INET6_TOKEN]              = { .len = sizeof(struct in6_addr) },
+};
+
+static int inet6_validate_link_af(const struct net_device *dev,
+                                 const struct nlattr *nla)
+{
+       struct nlattr *tb[IFLA_INET6_MAX + 1];
+
+       if (dev && !__in6_dev_get(dev))
+               return -EAFNOSUPPORT;
+
+       return nla_parse_nested(tb, IFLA_INET6_MAX, nla, inet6_af_policy);
+}
+
 static int inet6_set_link_af(struct net_device *dev, const struct nlattr *nla)
 {
        int err = -EINVAL;
@@ -5408,6 +5424,7 @@ static struct rtnl_af_ops inet6_ops = {
        .family           = AF_INET6,
        .fill_link_af     = inet6_fill_link_af,
        .get_link_af_size = inet6_get_link_af_size,
+       .validate_link_af = inet6_validate_link_af,
        .set_link_af      = inet6_set_link_af,
 };