tcp: tcp_send_challenge_ack delete useless param `skb`

After this parameter is passed in, there is no usage, and deleting it will
 not bring any impact.

Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Benjamin Yim <yan2228598786@gmail.com>
Link: https://lore.kernel.org/r/20220109130824.2776-1-yan2228598786@gmail.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c
index 8010583f868b2d1cb373f4b7522d3865d954ed88..dc49a3d551eb919baf5ad812ef21698c5c7b9679 100644 (file)
--- a/net/ipv4/tcp_input.c
+++ b/net/ipv4/tcp_input.c
@@ -3601,7 +3601,7 @@ bool tcp_oow_rate_limited(struct net *net, const struct sk_buff *skb,
 }
 
 /* RFC 5961 7 [ACK Throttling] */
-static void tcp_send_challenge_ack(struct sock *sk, const struct sk_buff *skb)
+static void tcp_send_challenge_ack(struct sock *sk)
 {
        /* unprotected vars, we dont care of overwrites */
        static u32 challenge_timestamp;
@@ -3763,7 +3763,7 @@ static int tcp_ack(struct sock *sk, const struct sk_buff *skb, int flag)
                /* RFC 5961 5.2 [Blind Data Injection Attack].[Mitigation] */
                if (before(ack, prior_snd_una - tp->max_window)) {
                        if (!(flag & FLAG_NO_CHALLENGE_ACK))
-                               tcp_send_challenge_ack(sk, skb);
+                               tcp_send_challenge_ack(sk);
                        return -1;
                }
                goto old_ack;
@@ -5726,7 +5726,7 @@ static bool tcp_validate_incoming(struct sock *sk, struct sk_buff *skb,
                        if (tp->syn_fastopen && !tp->data_segs_in &&
                            sk->sk_state == TCP_ESTABLISHED)
                                tcp_fastopen_active_disable(sk);
-                       tcp_send_challenge_ack(sk, skb);
+                       tcp_send_challenge_ack(sk);
                }
                goto discard;
        }
@@ -5741,7 +5741,7 @@ syn_challenge:
                if (syn_inerr)
                        TCP_INC_STATS(sock_net(sk), TCP_MIB_INERRS);
                NET_INC_STATS(sock_net(sk), LINUX_MIB_TCPSYNCHALLENGE);
-               tcp_send_challenge_ack(sk, skb);
+               tcp_send_challenge_ack(sk);
                goto discard;
        }
 
@@ -6456,7 +6456,7 @@ int tcp_rcv_state_process(struct sock *sk, struct sk_buff *skb)
        if (!acceptable) {
                if (sk->sk_state == TCP_SYN_RECV)
                        return 1;       /* send one RST */
-               tcp_send_challenge_ack(sk, skb);
+               tcp_send_challenge_ack(sk);
                goto discard;
        }
        switch (sk->sk_state) {