diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2012-11-15 22:03:00 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-11-18 20:32:44 -0500 |
commit | dfc47ef8639facd77210e74be831943c2fdd9c74 (patch) | |
tree | 5c7e9f93a999bf1d38b216af346ce2159e5f18ec /net/ipv4 | |
parent | 464dc801c76aa0db88e16e8f5f47c6879858b9b2 (diff) |
net: Push capable(CAP_NET_ADMIN) into the rtnl methods
- In rtnetlink_rcv_msg convert the capable(CAP_NET_ADMIN) check
to ns_capable(net->user-ns, CAP_NET_ADMIN). Allowing unprivileged
users to make netlink calls to modify their local network
namespace.
- In the rtnetlink doit methods add capable(CAP_NET_ADMIN) so
that calls that are not safe for unprivileged users are still
protected.
Later patches will remove the extra capable calls from methods
that are safe for unprivilged users.
Acked-by: Serge Hallyn <serge.hallyn@canonical.com>
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
-rw-r--r-- | net/ipv4/devinet.c | 6 | ||||
-rw-r--r-- | net/ipv4/fib_frontend.c | 6 |
2 files changed, 12 insertions, 0 deletions
diff --git a/net/ipv4/devinet.c b/net/ipv4/devinet.c index 6e06e924ed99..417093538916 100644 --- a/net/ipv4/devinet.c +++ b/net/ipv4/devinet.c | |||
@@ -539,6 +539,9 @@ static int inet_rtm_deladdr(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg | |||
539 | 539 | ||
540 | ASSERT_RTNL(); | 540 | ASSERT_RTNL(); |
541 | 541 | ||
542 | if (!capable(CAP_NET_ADMIN)) | ||
543 | return -EPERM; | ||
544 | |||
542 | err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFA_MAX, ifa_ipv4_policy); | 545 | err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFA_MAX, ifa_ipv4_policy); |
543 | if (err < 0) | 546 | if (err < 0) |
544 | goto errout; | 547 | goto errout; |
@@ -646,6 +649,9 @@ static int inet_rtm_newaddr(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg | |||
646 | 649 | ||
647 | ASSERT_RTNL(); | 650 | ASSERT_RTNL(); |
648 | 651 | ||
652 | if (!capable(CAP_NET_ADMIN)) | ||
653 | return -EPERM; | ||
654 | |||
649 | ifa = rtm_to_ifaddr(net, nlh); | 655 | ifa = rtm_to_ifaddr(net, nlh); |
650 | if (IS_ERR(ifa)) | 656 | if (IS_ERR(ifa)) |
651 | return PTR_ERR(ifa); | 657 | return PTR_ERR(ifa); |
diff --git a/net/ipv4/fib_frontend.c b/net/ipv4/fib_frontend.c index 825c608826de..bce4541c6784 100644 --- a/net/ipv4/fib_frontend.c +++ b/net/ipv4/fib_frontend.c | |||
@@ -613,6 +613,9 @@ static int inet_rtm_delroute(struct sk_buff *skb, struct nlmsghdr *nlh, void *ar | |||
613 | struct fib_table *tb; | 613 | struct fib_table *tb; |
614 | int err; | 614 | int err; |
615 | 615 | ||
616 | if (!capable(CAP_NET_ADMIN)) | ||
617 | return -EPERM; | ||
618 | |||
616 | err = rtm_to_fib_config(net, skb, nlh, &cfg); | 619 | err = rtm_to_fib_config(net, skb, nlh, &cfg); |
617 | if (err < 0) | 620 | if (err < 0) |
618 | goto errout; | 621 | goto errout; |
@@ -635,6 +638,9 @@ static int inet_rtm_newroute(struct sk_buff *skb, struct nlmsghdr *nlh, void *ar | |||
635 | struct fib_table *tb; | 638 | struct fib_table *tb; |
636 | int err; | 639 | int err; |
637 | 640 | ||
641 | if (!capable(CAP_NET_ADMIN)) | ||
642 | return -EPERM; | ||
643 | |||
638 | err = rtm_to_fib_config(net, skb, nlh, &cfg); | 644 | err = rtm_to_fib_config(net, skb, nlh, &cfg); |
639 | if (err < 0) | 645 | if (err < 0) |
640 | goto errout; | 646 | goto errout; |