stream: avoid ineffective state bump in TFO

Do not set session state to established, as it is unconditionally
overwritten to syn_recv afterwards.
pull/13817/head
Victor Julien 8 months ago committed by Victor Julien
parent 49b98ab3bf
commit d815d0748c

@ -1859,10 +1859,7 @@ static void TcpStateQueueInitFromPktSynAck(const Packet *p, TcpStateQueue *q)
{
#if defined(DEBUG_VALIDATION) || defined(DEBUG)
const TcpSession *ssn = p->flow->protoctx;
if ((ssn->flags & STREAMTCP_FLAG_TCP_FAST_OPEN) == 0)
BUG_ON(ssn->state != TCP_SYN_SENT);
else
BUG_ON(ssn->state != TCP_ESTABLISHED);
BUG_ON(ssn->state != TCP_SYN_SENT);
#endif
memset(q, 0, sizeof(*q));
@ -2020,7 +2017,6 @@ static int StreamTcpPacketStateSynSent(
return -1;
}
ssn->flags |= STREAMTCP_FLAG_TCP_FAST_OPEN;
StreamTcpPacketSetState(p, ssn, TCP_ESTABLISHED);
}
const bool ts_mismatch = !StateSynSentValidateTimestamp(ssn, p);

Loading…
Cancel
Save