af_unix: use DEBUG_NET_WARN_ON_ONCE()
authorEric Dumazet <edumazet@google.com>
Wed, 8 Jun 2022 16:04:35 +0000 (09:04 -0700)
committerJakub Kicinski <kuba@kernel.org>
Fri, 10 Jun 2022 04:53:56 +0000 (21:53 -0700)
Replace four WARN_ON() that have not triggered recently
with DEBUG_NET_WARN_ON_ONCE().

Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/unix/af_unix.c

index 2206e6f..3453e00 100644 (file)
@@ -302,7 +302,7 @@ static void __unix_remove_socket(struct sock *sk)
 
 static void __unix_insert_socket(struct sock *sk)
 {
-       WARN_ON(!sk_unhashed(sk));
+       DEBUG_NET_WARN_ON_ONCE(!sk_unhashed(sk));
        sk_add_node(sk, &unix_socket_table[sk->sk_hash]);
 }
 
@@ -554,9 +554,9 @@ static void unix_sock_destructor(struct sock *sk)
                u->oob_skb = NULL;
        }
 #endif
-       WARN_ON(refcount_read(&sk->sk_wmem_alloc));
-       WARN_ON(!sk_unhashed(sk));
-       WARN_ON(sk->sk_socket);
+       DEBUG_NET_WARN_ON_ONCE(refcount_read(&sk->sk_wmem_alloc));
+       DEBUG_NET_WARN_ON_ONCE(!sk_unhashed(sk));
+       DEBUG_NET_WARN_ON_ONCE(sk->sk_socket);
        if (!sock_flag(sk, SOCK_DEAD)) {
                pr_info("Attempt to release alive unix socket: %p\n", sk);
                return;