mptcp: Remove set but not used variable 'can_ack'
authorYueHaibing <yuehaibing@huawei.com>
Wed, 18 Mar 2020 02:01:57 +0000 (02:01 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 22 Mar 2020 03:20:36 +0000 (20:20 -0700)
Fixes gcc '-Wunused-but-set-variable' warning:

net/mptcp/options.c: In function 'mptcp_established_options_dss':
net/mptcp/options.c:338:7: warning:
 variable 'can_ack' set but not used [-Wunused-but-set-variable]

commit dc093db5cc05 ("mptcp: drop unneeded checks")
leave behind this unused, remove it.

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Acked-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/mptcp/options.c

index 55f3ce7638a0e7cc5898b5fa3711242fab751cb4..aea1a62d9999cae5be91f4a0cec4fa1c308027af 100644 (file)
@@ -335,7 +335,6 @@ static bool mptcp_established_options_dss(struct sock *sk, struct sk_buff *skb,
        struct mptcp_sock *msk;
        unsigned int ack_size;
        bool ret = false;
-       bool can_ack;
        u8 tcp_fin;
 
        if (skb) {
@@ -364,7 +363,6 @@ static bool mptcp_established_options_dss(struct sock *sk, struct sk_buff *skb,
        /* passive sockets msk will set the 'can_ack' after accept(), even
         * if the first subflow may have the already the remote key handy
         */
-       can_ack = true;
        opts->ext_copy.use_ack = 0;
        msk = mptcp_sk(subflow->conn);
        if (!READ_ONCE(msk->can_ack)) {