diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2007-10-09 02:24:22 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-10 19:49:07 -0400 |
commit | 1b8d7ae42d02e483ad94035cca851e4f7fbecb40 (patch) | |
tree | 81f8cc0ee49ef99cc67dfed3dc7b7ecb510abf8b /net/bluetooth/l2cap.c | |
parent | 457c4cbc5a3dde259d2a1f15d5f9785290397267 (diff) |
[NET]: Make socket creation namespace safe.
This patch passes in the namespace a new socket should be created in
and has the socket code do the appropriate reference counting. By
virtue of this all socket create methods are touched. In addition
the socket create methods are modified so that they will fail if
you attempt to create a socket in a non-default network namespace.
Failing if we attempt to create a socket outside of the default
network namespace ensures that as we incrementally make the network stack
network namespace aware we will not export functionality that someone
has not audited and made certain is network namespace safe.
Allowing us to partially enable network namespaces before all of the
exotic protocols are supported.
Any protocol layers I have missed will fail to compile because I now
pass an extra parameter into the socket creation code.
[ Integrated AF_IUCV build fixes from Andrew Morton... -DaveM ]
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/bluetooth/l2cap.c')
-rw-r--r-- | net/bluetooth/l2cap.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/net/bluetooth/l2cap.c b/net/bluetooth/l2cap.c index c4e4ce4ebb2b..36ef27b625db 100644 --- a/net/bluetooth/l2cap.c +++ b/net/bluetooth/l2cap.c | |||
@@ -518,11 +518,11 @@ static struct proto l2cap_proto = { | |||
518 | .obj_size = sizeof(struct l2cap_pinfo) | 518 | .obj_size = sizeof(struct l2cap_pinfo) |
519 | }; | 519 | }; |
520 | 520 | ||
521 | static struct sock *l2cap_sock_alloc(struct socket *sock, int proto, gfp_t prio) | 521 | static struct sock *l2cap_sock_alloc(struct net *net, struct socket *sock, int proto, gfp_t prio) |
522 | { | 522 | { |
523 | struct sock *sk; | 523 | struct sock *sk; |
524 | 524 | ||
525 | sk = sk_alloc(PF_BLUETOOTH, prio, &l2cap_proto, 1); | 525 | sk = sk_alloc(net, PF_BLUETOOTH, prio, &l2cap_proto, 1); |
526 | if (!sk) | 526 | if (!sk) |
527 | return NULL; | 527 | return NULL; |
528 | 528 | ||
@@ -543,7 +543,7 @@ static struct sock *l2cap_sock_alloc(struct socket *sock, int proto, gfp_t prio) | |||
543 | return sk; | 543 | return sk; |
544 | } | 544 | } |
545 | 545 | ||
546 | static int l2cap_sock_create(struct socket *sock, int protocol) | 546 | static int l2cap_sock_create(struct net *net, struct socket *sock, int protocol) |
547 | { | 547 | { |
548 | struct sock *sk; | 548 | struct sock *sk; |
549 | 549 | ||
@@ -560,7 +560,7 @@ static int l2cap_sock_create(struct socket *sock, int protocol) | |||
560 | 560 | ||
561 | sock->ops = &l2cap_sock_ops; | 561 | sock->ops = &l2cap_sock_ops; |
562 | 562 | ||
563 | sk = l2cap_sock_alloc(sock, protocol, GFP_ATOMIC); | 563 | sk = l2cap_sock_alloc(net, sock, protocol, GFP_ATOMIC); |
564 | if (!sk) | 564 | if (!sk) |
565 | return -ENOMEM; | 565 | return -ENOMEM; |
566 | 566 | ||
@@ -1425,7 +1425,7 @@ static inline int l2cap_connect_req(struct l2cap_conn *conn, struct l2cap_cmd_hd | |||
1425 | goto response; | 1425 | goto response; |
1426 | } | 1426 | } |
1427 | 1427 | ||
1428 | sk = l2cap_sock_alloc(NULL, BTPROTO_L2CAP, GFP_ATOMIC); | 1428 | sk = l2cap_sock_alloc(parent->sk_net, NULL, BTPROTO_L2CAP, GFP_ATOMIC); |
1429 | if (!sk) | 1429 | if (!sk) |
1430 | goto response; | 1430 | goto response; |
1431 | 1431 | ||