mptcp: don't send RST for single subflow
When a bad checksum is detected and a single subflow is in use, don't send RST + MP_FAIL, send data_ack + MP_FAIL instead. So invoke tcp_send_active_reset() only when mptcp_has_another_subflow() is true. Signed-off-by: Geliang Tang <geliang.tang@suse.com> Signed-off-by: Mat Martineau <mathew.j.martineau@linux.intel.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
31693d02b0
commit
1761fed256
|
@ -1206,14 +1206,14 @@ fallback:
|
||||||
/* RFC 8684 section 3.7. */
|
/* RFC 8684 section 3.7. */
|
||||||
if (subflow->send_mp_fail) {
|
if (subflow->send_mp_fail) {
|
||||||
if (mptcp_has_another_subflow(ssk)) {
|
if (mptcp_has_another_subflow(ssk)) {
|
||||||
|
ssk->sk_err = EBADMSG;
|
||||||
|
tcp_set_state(ssk, TCP_CLOSE);
|
||||||
|
subflow->reset_transient = 0;
|
||||||
|
subflow->reset_reason = MPTCP_RST_EMIDDLEBOX;
|
||||||
|
tcp_send_active_reset(ssk, GFP_ATOMIC);
|
||||||
while ((skb = skb_peek(&ssk->sk_receive_queue)))
|
while ((skb = skb_peek(&ssk->sk_receive_queue)))
|
||||||
sk_eat_skb(ssk, skb);
|
sk_eat_skb(ssk, skb);
|
||||||
}
|
}
|
||||||
ssk->sk_err = EBADMSG;
|
|
||||||
tcp_set_state(ssk, TCP_CLOSE);
|
|
||||||
subflow->reset_transient = 0;
|
|
||||||
subflow->reset_reason = MPTCP_RST_EMIDDLEBOX;
|
|
||||||
tcp_send_active_reset(ssk, GFP_ATOMIC);
|
|
||||||
WRITE_ONCE(subflow->data_avail, MPTCP_SUBFLOW_NODATA);
|
WRITE_ONCE(subflow->data_avail, MPTCP_SUBFLOW_NODATA);
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue