net/ipv4: fix macro definition sk_for_each_bound_bhash
authorHongbo Li <lihongbo22@huawei.com>
Fri, 23 Aug 2024 07:04:53 +0000 (15:04 +0800)
committerJakub Kicinski <kuba@kernel.org>
Mon, 26 Aug 2024 21:04:25 +0000 (14:04 -0700)
The macro sk_for_each_bound_bhash accepts a parameter
__sk, but it was not used, rather the sk2 is directly
used, so we replace the sk2 with __sk in macro.

Signed-off-by: Hongbo Li <lihongbo22@huawei.com>
Link: https://patch.msgid.link/20240823070453.3327832-1-lihongbo22@huawei.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/ipv4/inet_connection_sock.c

index 64d07b842e736f21654ce79e4e90903e58563dd1..ce4d77f49243655a540c7ed4350b91f979abc79e 100644 (file)
@@ -236,7 +236,7 @@ static bool inet_bhash2_conflict(const struct sock *sk,
 
 #define sk_for_each_bound_bhash(__sk, __tb2, __tb)                     \
        hlist_for_each_entry(__tb2, &(__tb)->bhash2, bhash_node)        \
-               sk_for_each_bound(sk2, &(__tb2)->owners)
+               sk_for_each_bound((__sk), &(__tb2)->owners)
 
 /* This should be called only when the tb and tb2 hashbuckets' locks are held */
 static int inet_csk_bind_conflict(const struct sock *sk,