summaryrefslogtreecommitdiffstats
path: root/net/econet
diff options
context:
space:
mode:
authorDavid S. Miller <davem@davemloft.net>2008-06-17 03:01:47 -0700
committerDavid S. Miller <davem@davemloft.net>2008-06-17 03:01:47 -0700
commit0efffaf9d57f54a864b739176d1f659a69486e8b (patch)
tree24948ff780a7a44c93daa5736df7e5fdb9706f6b /net/econet
parentb61d38e05542200f6d001b5ea9975dc0dc5bab27 (diff)
downloadop-kernel-dev-0efffaf9d57f54a864b739176d1f659a69486e8b.zip
op-kernel-dev-0efffaf9d57f54a864b739176d1f659a69486e8b.tar.gz
econet: Use sock_orphan() instead of open-coded (and buggy) variant.
It doesn't grab the sk_callback_lock, it doesn't NULL out the sk->sk_sleep waitqueue pointer, etc. Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/econet')
-rw-r--r--net/econet/af_econet.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/net/econet/af_econet.c b/net/econet/af_econet.c
index 7c9bb13..d35127b 100644
--- a/net/econet/af_econet.c
+++ b/net/econet/af_econet.c
@@ -573,9 +573,7 @@ static int econet_release(struct socket *sock)
sk->sk_state_change(sk); /* It is useless. Just for sanity. */
- sock->sk = NULL;
- sk->sk_socket = NULL;
- sock_set_flag(sk, SOCK_DEAD);
+ sock_orphan(sk);
/* Purge queues */
OpenPOWER on IntegriCloud