Page MenuHomeFreeBSD

Honor net.inet.tcp.drop_synfin in other states than LISTEN
ClosedPublic

Authored by tuexen on Mar 5 2017, 11:08 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 29, 6:26 AM
Unknown Object (File)
Feb 19 2024, 1:03 AM
Unknown Object (File)
Feb 18 2024, 8:04 PM
Unknown Object (File)
Jan 27 2024, 7:24 AM
Unknown Object (File)
Dec 25 2023, 5:20 AM
Unknown Object (File)
Dec 21 2023, 12:12 AM
Unknown Object (File)
Dec 20 2023, 1:07 AM
Unknown Object (File)
Nov 29 2023, 11:09 AM

Details

Summary

Currently the sysctl variable net.inet.tcp.drop_synfin is not honored in all states, for example not in SYN-SENT.
This patch adds code to check the sysctl variable in other states than LISTEN.

Test Plan

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 8563
Build 8871: CI src buildJenkins

Event Timeline

tuexen retitled this revision from to Honor net.inet.tcp.drop_synfin in other states then LISTEN.
tuexen updated this object.
tuexen edited the test plan for this revision. (Show Details)
tuexen added reviewers: gnn, rrs, hiren, transport.
tuexen set the repository for this revision to rS FreeBSD src repository - subversion.
tuexen retitled this revision from Honor net.inet.tcp.drop_synfin in other states then LISTEN to Honor net.inet.tcp.drop_synfin in other states than LISTEN.Mar 5 2017, 11:09 AM
tuexen edited edge metadata.
gnn requested changes to this revision.Mar 30 2017, 1:12 AM

Other than the need for some clarifying parentheses I think this is OK.

sys/netinet/tcp_input.c
1554

I'd like some clarifying parentheses here as well as in all the other similar if statements below.

This revision now requires changes to proceed.Mar 30 2017, 1:12 AM
tuexen edited edge metadata.

Address gnn's comment by adding parentheses.

ae added inline comments.
sys/netinet/tcp_input.c
248

This declaration seems unnecessary. It is already defined at the line 147.

Address Andrey's comment.

This revision is now accepted and ready to land.Apr 11 2017, 2:00 PM
This revision was automatically updated to reflect the committed changes.