diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2005-08-06 09:33:15 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2005-08-06 09:33:15 -0400 |
| commit | 6fc0b4a7a73a81e74d0004732df358f4f9975be2 (patch) | |
| tree | ac2c68e206efbfe378728a50be30ef0bccd0cdff | |
| parent | 534afb90a9cd0b9643f62d660c164e1d924f39cf (diff) | |
[IPSEC]: Restrict socket policy loading to CAP_NET_ADMIN.
The interface needs much redesigning if we wish to allow
normal users to do this in some way.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/ipv4/ip_sockglue.c | 3 | ||||
| -rw-r--r-- | net/ipv6/ipv6_sockglue.c | 3 |
2 files changed, 6 insertions, 0 deletions
diff --git a/net/ipv4/ip_sockglue.c b/net/ipv4/ip_sockglue.c index fc7c481d0d79..ff4bd067b397 100644 --- a/net/ipv4/ip_sockglue.c +++ b/net/ipv4/ip_sockglue.c | |||
| @@ -848,6 +848,9 @@ mc_msf_out: | |||
| 848 | 848 | ||
| 849 | case IP_IPSEC_POLICY: | 849 | case IP_IPSEC_POLICY: |
| 850 | case IP_XFRM_POLICY: | 850 | case IP_XFRM_POLICY: |
| 851 | err = -EPERM; | ||
| 852 | if (!capable(CAP_NET_ADMIN)) | ||
| 853 | break; | ||
| 851 | err = xfrm_user_policy(sk, optname, optval, optlen); | 854 | err = xfrm_user_policy(sk, optname, optval, optlen); |
| 852 | break; | 855 | break; |
| 853 | 856 | ||
diff --git a/net/ipv6/ipv6_sockglue.c b/net/ipv6/ipv6_sockglue.c index f3ef4c38d315..3bc144a79fa5 100644 --- a/net/ipv6/ipv6_sockglue.c +++ b/net/ipv6/ipv6_sockglue.c | |||
| @@ -504,6 +504,9 @@ done: | |||
| 504 | break; | 504 | break; |
| 505 | case IPV6_IPSEC_POLICY: | 505 | case IPV6_IPSEC_POLICY: |
| 506 | case IPV6_XFRM_POLICY: | 506 | case IPV6_XFRM_POLICY: |
| 507 | retv = -EPERM; | ||
| 508 | if (!capable(CAP_NET_ADMIN)) | ||
| 509 | break; | ||
| 507 | retv = xfrm_user_policy(sk, optname, optval, optlen); | 510 | retv = xfrm_user_policy(sk, optname, optval, optlen); |
| 508 | break; | 511 | break; |
| 509 | 512 | ||
