diff options
| author | David S. Miller <davem@davemloft.net> | 2008-06-17 03:01:47 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-06-17 03:01:47 -0700 |
| commit | 0efffaf9d57f54a864b739176d1f659a69486e8b (patch) | |
| tree | 24948ff780a7a44c93daa5736df7e5fdb9706f6b /net/econet | |
| parent | b61d38e05542200f6d001b5ea9975dc0dc5bab27 (diff) | |
| download | lwn-0efffaf9d57f54a864b739176d1f659a69486e8b.tar.gz lwn-0efffaf9d57f54a864b739176d1f659a69486e8b.zip | |
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.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/net/econet/af_econet.c b/net/econet/af_econet.c index 7c9bb13b1539..d35127bb84e1 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 */ |