diff options
author | Andrew Morton <akpm@osdl.org> | 2005-10-28 18:12:02 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@mandriva.com> | 2005-10-31 13:41:45 -0500 |
commit | a3d7a9d77533d7516a8cfb8e2b612cd5ead4fa59 (patch) | |
tree | 62ea3dce8a8d7c6b984c0815347de9ca187e8335 /net/rose/rose_timer.c | |
parent | 6b7d31fcdda5938e5d3f1f8b0922cc25aa200dfc (diff) |
[ROSE]: rose_heartbeat_expiry() locking fix
Missing unlock, as noted by Ted Unangst <tedu@coverity.com>.
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Arnaldo Carvalho de Melo <acme@mandriva.com>
Diffstat (limited to 'net/rose/rose_timer.c')
-rw-r--r-- | net/rose/rose_timer.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/rose/rose_timer.c b/net/rose/rose_timer.c index 50ae0371dab8..b6c8f38cc26c 100644 --- a/net/rose/rose_timer.c +++ b/net/rose/rose_timer.c | |||
@@ -138,6 +138,7 @@ static void rose_heartbeat_expiry(unsigned long param) | |||
138 | is accepted() it isn't 'dead' so doesn't get removed. */ | 138 | is accepted() it isn't 'dead' so doesn't get removed. */ |
139 | if (sock_flag(sk, SOCK_DESTROY) || | 139 | if (sock_flag(sk, SOCK_DESTROY) || |
140 | (sk->sk_state == TCP_LISTEN && sock_flag(sk, SOCK_DEAD))) { | 140 | (sk->sk_state == TCP_LISTEN && sock_flag(sk, SOCK_DEAD))) { |
141 | bh_unlock_sock(sk); | ||
141 | rose_destroy_socket(sk); | 142 | rose_destroy_socket(sk); |
142 | return; | 143 | return; |
143 | } | 144 | } |