Remove spurious timestamp check. RFC 1323 explicitly states that timestamps MAY
be transmitted if negotiated.
This commit is contained in:
parent
08536bf155
commit
a571860f41
@ -890,23 +890,7 @@ syncache_expand(struct in_conninfo *inc, struct tcpopt *to, struct tcphdr *th,
|
||||
"rejected\n", s, __func__, th->th_seq, sc->sc_irs);
|
||||
goto failed;
|
||||
}
|
||||
#if 0
|
||||
/*
|
||||
* If timestamps were present in the SYN and we accepted
|
||||
* them in our SYN|ACK we require them to be present from
|
||||
* now on. And vice versa.
|
||||
*
|
||||
* Unfortunately, during testing of 7.0 some users found
|
||||
* network devices that violate this constraint, so it must
|
||||
* be disabled.
|
||||
*/
|
||||
if ((sc->sc_flags & SCF_TIMESTAMP) && !(to->to_flags & TOF_TS)) {
|
||||
if ((s = tcp_log_addrs(inc, th, NULL, NULL)))
|
||||
log(LOG_DEBUG, "%s; %s: Timestamp missing, "
|
||||
"segment rejected\n", s, __func__);
|
||||
goto failed;
|
||||
}
|
||||
#endif
|
||||
|
||||
if (!(sc->sc_flags & SCF_TIMESTAMP) && (to->to_flags & TOF_TS)) {
|
||||
if ((s = tcp_log_addrs(inc, th, NULL, NULL)))
|
||||
log(LOG_DEBUG, "%s; %s: Timestamp not expected, "
|
||||
|
Loading…
Reference in New Issue
Block a user