net: Enable a userns root rtnl calls that are safe for unprivilged users
- Only allow moving network devices to network namespaces you have CAP_NET_ADMIN privileges over. - Enable creating/deleting/modifying interfaces - Enable adding/deleting addresses - Enable adding/setting/deleting neighbour entries - Enable adding/removing routes - Enable adding/removing fib rules - Enable setting the forwarding state - Enable adding/removing ipv6 address labels - Enable setting bridge parameter Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
					parent
					
						
							
								c027aab4a6
							
						
					
				
			
			
				commit
				
					
						b51642f6d7
					
				
			
		
					 9 changed files with 4 additions and 54 deletions
				
			
		|  | @ -240,9 +240,6 @@ int br_setlink(struct net_device *dev, struct nlmsghdr *nlh) | ||||||
| 	struct nlattr *tb[IFLA_BRPORT_MAX]; | 	struct nlattr *tb[IFLA_BRPORT_MAX]; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	ifm = nlmsg_data(nlh); | 	ifm = nlmsg_data(nlh); | ||||||
| 
 | 
 | ||||||
| 	protinfo = nlmsg_find_attr(nlh, sizeof(*ifm), IFLA_PROTINFO); | 	protinfo = nlmsg_find_attr(nlh, sizeof(*ifm), IFLA_PROTINFO); | ||||||
|  |  | ||||||
|  | @ -275,9 +275,6 @@ static int fib_nl_newrule(struct sk_buff *skb, struct nlmsghdr* nlh, void *arg) | ||||||
| 	struct nlattr *tb[FRA_MAX+1]; | 	struct nlattr *tb[FRA_MAX+1]; | ||||||
| 	int err = -EINVAL, unresolved = 0; | 	int err = -EINVAL, unresolved = 0; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	if (nlh->nlmsg_len < nlmsg_msg_size(sizeof(*frh))) | 	if (nlh->nlmsg_len < nlmsg_msg_size(sizeof(*frh))) | ||||||
| 		goto errout; | 		goto errout; | ||||||
| 
 | 
 | ||||||
|  | @ -427,9 +424,6 @@ static int fib_nl_delrule(struct sk_buff *skb, struct nlmsghdr* nlh, void *arg) | ||||||
| 	struct nlattr *tb[FRA_MAX+1]; | 	struct nlattr *tb[FRA_MAX+1]; | ||||||
| 	int err = -EINVAL; | 	int err = -EINVAL; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	if (nlh->nlmsg_len < nlmsg_msg_size(sizeof(*frh))) | 	if (nlh->nlmsg_len < nlmsg_msg_size(sizeof(*frh))) | ||||||
| 		goto errout; | 		goto errout; | ||||||
| 
 | 
 | ||||||
|  |  | ||||||
|  | @ -1620,9 +1620,6 @@ static int neigh_delete(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | ||||||
| 	struct net_device *dev = NULL; | 	struct net_device *dev = NULL; | ||||||
| 	int err = -EINVAL; | 	int err = -EINVAL; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	ASSERT_RTNL(); | 	ASSERT_RTNL(); | ||||||
| 	if (nlmsg_len(nlh) < sizeof(*ndm)) | 	if (nlmsg_len(nlh) < sizeof(*ndm)) | ||||||
| 		goto out; | 		goto out; | ||||||
|  | @ -1687,9 +1684,6 @@ static int neigh_add(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | ||||||
| 	struct net_device *dev = NULL; | 	struct net_device *dev = NULL; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	ASSERT_RTNL(); | 	ASSERT_RTNL(); | ||||||
| 	err = nlmsg_parse(nlh, sizeof(*ndm), tb, NDA_MAX, NULL); | 	err = nlmsg_parse(nlh, sizeof(*ndm), tb, NDA_MAX, NULL); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
|  | @ -1968,9 +1962,6 @@ static int neightbl_set(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | ||||||
| 	struct nlattr *tb[NDTA_MAX+1]; | 	struct nlattr *tb[NDTA_MAX+1]; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = nlmsg_parse(nlh, sizeof(*ndtmsg), tb, NDTA_MAX, | 	err = nlmsg_parse(nlh, sizeof(*ndtmsg), tb, NDTA_MAX, | ||||||
| 			  nl_neightbl_policy); | 			  nl_neightbl_policy); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
|  |  | ||||||
|  | @ -1316,6 +1316,10 @@ static int do_setlink(struct net_device *dev, struct ifinfomsg *ifm, | ||||||
| 			err = PTR_ERR(net); | 			err = PTR_ERR(net); | ||||||
| 			goto errout; | 			goto errout; | ||||||
| 		} | 		} | ||||||
|  | 		if (!ns_capable(net->user_ns, CAP_NET_ADMIN)) { | ||||||
|  | 			err = -EPERM; | ||||||
|  | 			goto errout; | ||||||
|  | 		} | ||||||
| 		err = dev_change_net_namespace(dev, net, ifname); | 		err = dev_change_net_namespace(dev, net, ifname); | ||||||
| 		put_net(net); | 		put_net(net); | ||||||
| 		if (err) | 		if (err) | ||||||
|  | @ -1547,9 +1551,6 @@ static int rtnl_setlink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | ||||||
| 	struct nlattr *tb[IFLA_MAX+1]; | 	struct nlattr *tb[IFLA_MAX+1]; | ||||||
| 	char ifname[IFNAMSIZ]; | 	char ifname[IFNAMSIZ]; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFLA_MAX, ifla_policy); | 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFLA_MAX, ifla_policy); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		goto errout; | 		goto errout; | ||||||
|  | @ -1593,9 +1594,6 @@ static int rtnl_dellink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | ||||||
| 	int err; | 	int err; | ||||||
| 	LIST_HEAD(list_kill); | 	LIST_HEAD(list_kill); | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFLA_MAX, ifla_policy); | 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFLA_MAX, ifla_policy); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		return err; | 		return err; | ||||||
|  | @ -1726,9 +1724,6 @@ static int rtnl_newlink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | ||||||
| 	struct nlattr *linkinfo[IFLA_INFO_MAX+1]; | 	struct nlattr *linkinfo[IFLA_INFO_MAX+1]; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| #ifdef CONFIG_MODULES | #ifdef CONFIG_MODULES | ||||||
| replay: | replay: | ||||||
| #endif | #endif | ||||||
|  |  | ||||||
|  | @ -539,9 +539,6 @@ static int inet_rtm_deladdr(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg | ||||||
| 
 | 
 | ||||||
| 	ASSERT_RTNL(); | 	ASSERT_RTNL(); | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFA_MAX, ifa_ipv4_policy); | 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFA_MAX, ifa_ipv4_policy); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		goto errout; | 		goto errout; | ||||||
|  | @ -649,9 +646,6 @@ static int inet_rtm_newaddr(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg | ||||||
| 
 | 
 | ||||||
| 	ASSERT_RTNL(); | 	ASSERT_RTNL(); | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	ifa = rtm_to_ifaddr(net, nlh); | 	ifa = rtm_to_ifaddr(net, nlh); | ||||||
| 	if (IS_ERR(ifa)) | 	if (IS_ERR(ifa)) | ||||||
| 		return PTR_ERR(ifa); | 		return PTR_ERR(ifa); | ||||||
|  |  | ||||||
|  | @ -613,9 +613,6 @@ static int inet_rtm_delroute(struct sk_buff *skb, struct nlmsghdr *nlh, void *ar | ||||||
| 	struct fib_table *tb; | 	struct fib_table *tb; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = rtm_to_fib_config(net, skb, nlh, &cfg); | 	err = rtm_to_fib_config(net, skb, nlh, &cfg); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		goto errout; | 		goto errout; | ||||||
|  | @ -638,9 +635,6 @@ static int inet_rtm_newroute(struct sk_buff *skb, struct nlmsghdr *nlh, void *ar | ||||||
| 	struct fib_table *tb; | 	struct fib_table *tb; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = rtm_to_fib_config(net, skb, nlh, &cfg); | 	err = rtm_to_fib_config(net, skb, nlh, &cfg); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		goto errout; | 		goto errout; | ||||||
|  |  | ||||||
|  | @ -3514,9 +3514,6 @@ inet6_rtm_deladdr(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | ||||||
| 	struct in6_addr *pfx; | 	struct in6_addr *pfx; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFA_MAX, ifa_ipv6_policy); | 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFA_MAX, ifa_ipv6_policy); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		return err; | 		return err; | ||||||
|  | @ -3587,9 +3584,6 @@ inet6_rtm_newaddr(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | ||||||
| 	u8 ifa_flags; | 	u8 ifa_flags; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFA_MAX, ifa_ipv6_policy); | 	err = nlmsg_parse(nlh, sizeof(*ifm), tb, IFA_MAX, ifa_ipv6_policy); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		return err; | 		return err; | ||||||
|  |  | ||||||
|  | @ -425,9 +425,6 @@ static int ip6addrlbl_newdel(struct sk_buff *skb, struct nlmsghdr *nlh, | ||||||
| 	u32 label; | 	u32 label; | ||||||
| 	int err = 0; | 	int err = 0; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = nlmsg_parse(nlh, sizeof(*ifal), tb, IFAL_MAX, ifal_policy); | 	err = nlmsg_parse(nlh, sizeof(*ifal), tb, IFAL_MAX, ifal_policy); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		return err; | 		return err; | ||||||
|  |  | ||||||
|  | @ -2446,9 +2446,6 @@ static int inet6_rtm_delroute(struct sk_buff *skb, struct nlmsghdr* nlh, void *a | ||||||
| 	struct fib6_config cfg; | 	struct fib6_config cfg; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = rtm_to_fib6_config(skb, nlh, &cfg); | 	err = rtm_to_fib6_config(skb, nlh, &cfg); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		return err; | 		return err; | ||||||
|  | @ -2464,9 +2461,6 @@ static int inet6_rtm_newroute(struct sk_buff *skb, struct nlmsghdr* nlh, void *a | ||||||
| 	struct fib6_config cfg; | 	struct fib6_config cfg; | ||||||
| 	int err; | 	int err; | ||||||
| 
 | 
 | ||||||
| 	if (!capable(CAP_NET_ADMIN)) |  | ||||||
| 		return -EPERM; |  | ||||||
| 
 |  | ||||||
| 	err = rtm_to_fib6_config(skb, nlh, &cfg); | 	err = rtm_to_fib6_config(skb, nlh, &cfg); | ||||||
| 	if (err < 0) | 	if (err < 0) | ||||||
| 		return err; | 		return err; | ||||||
|  |  | ||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue
	
	 Eric W. Biederman
				Eric W. Biederman