summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorYueHaibing <yuehaibing@huawei.com>2020-03-18 03:01:57 +0100
committerDavid S. Miller <davem@davemloft.net>2020-03-22 04:20:36 +0100
commit09984483db080b541c8242d846c30bc1e6a194e1 (patch)
tree7bdf9f8db89d5fcc29ccef9e877829d2f2b03d9b /net
parentMerge branch 'selftests-expand-txtimestamp-with-new-features' (diff)
downloadlinux-09984483db080b541c8242d846c30bc1e6a194e1.tar.xz
linux-09984483db080b541c8242d846c30bc1e6a194e1.zip
mptcp: Remove set but not used variable 'can_ack'
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>
Diffstat (limited to 'net')
-rw-r--r--net/mptcp/options.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/net/mptcp/options.c b/net/mptcp/options.c
index 55f3ce7638a0..aea1a62d9999 100644
--- a/net/mptcp/options.c
+++ b/net/mptcp/options.c
@@ -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)) {