diff options
| author | YueHaibing <yuehaibing@huawei.com> | 2020-03-18 03:01:57 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-03-22 04:20:36 +0100 |
| commit | 09984483db080b541c8242d846c30bc1e6a194e1 (patch) | |
| tree | 7bdf9f8db89d5fcc29ccef9e877829d2f2b03d9b /net/mptcp/options.c | |
| parent | Merge branch 'selftests-expand-txtimestamp-with-new-features' (diff) | |
| download | linux-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/mptcp/options.c')
| -rw-r--r-- | net/mptcp/options.c | 2 |
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)) { |