tcp_reset: Don't copy the peer window

The current code just leave the window value from the segment
from the peer. It doesn't make sense.

Instead, always use 0.
This matches what NetBSD and Linux do.
(As far as I read their code correctly.)
This commit is contained in:
YAMAMOTO Takashi 2021-06-29 22:49:39 +09:00 committed by Xiang Xiao
parent 98e7c6924d
commit 1ce13ee731

View File

@ -510,6 +510,14 @@ void tcp_reset(FAR struct net_driver_s *dev)
tcp->srcport = tcp->destport;
tcp->destport = tmp16;
/* Initialize the rest of the tcp header to sane values.
*
* Note: urgp is set by tcp_ipv4_sendcomplete/tcp_ipv6_sendcomplete.
*/
tcp->wnd[0] = 0;
tcp->wnd[1] = 0;
/* Set the packet length and swap IP addresses. */
#ifdef CONFIG_NET_IPv6