Re: BUG: IPv4: Attempt to release TCP socket in state 1

On Fri, 2013-03-15 at 00:19 +0100, Eric Dumazet wrote:

quoted

Thanks thats really useful, we might miss to increment socket refcount
in a timer setup.

Hmm, please add following debugging patch as well

diff --git a/include/net/sock.h b/include/net/sock.h
index 14f6e9d..fe7c8a6 100644
--- a/include/net/sock.h
+++ b/include/net/sock.h

@@ -530,7 +530,9 @@ static inline void sock_hold(struct sock *sk)
  */
 static inline void __sock_put(struct sock *sk)
 {
-	atomic_dec(&sk->sk_refcnt);
+	int newref = atomic_dec_return(&sk->sk_refcnt);
+
+	BUG_ON(newref <= 0);
 }