Page MenuHomeFreeBSD
Differential D36729

netinet*: remove dead code from TCP, UDP, SCTP control input
ClosedPublic
Actions

Authored by glebius on Sep 26 2022, 7:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jul 21, 2:04 AM
Unknown Object (File)
Sun, Jul 13, 5:07 AM
Unknown Object (File)
Fri, Jul 11, 10:36 AM
Unknown Object (File)
Wed, Jul 9, 1:27 AM
Unknown Object (File)
Tue, Jul 1, 10:41 PM
Unknown Object (File)
Mon, Jun 30, 9:14 PM
Unknown Object (File)
Jun 30 2025, 5:00 AM
Unknown Object (File)
Jun 29 2025, 4:33 AM
View All 85 Files
Subscribers
ae
imp
melifaro

Details

Reviewers
None
Group Reviewers
network
transport
Commits
rGc0fc81e9138b: netinet*: remove dead code from TCP, UDP, SCTP control input
Summary

Now these functions are called only from icmp*_input(). The pointer to
the ICMP data is never NULL and cmd has a limited set of values.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

glebius created this revision.Sep 26 2022, 7:02 PM
Herald added a reviewer: transport. · View Herald TranscriptSep 26 2022, 7:02 PM
Herald added subscribers: melifaro, ae, imp. · View Herald Transcript
glebius requested review of this revision.Sep 26 2022, 7:02 PM
Harbormaster completed remote builds in B47548: Diff 111017.Sep 26 2022, 7:02 PM
glebius added a parent revision: D36728: netinet: filter out invalid ICMP responses in ip_icmp().Sep 26 2022, 7:02 PM
glebius added a child revision: D36730: netipsec: move specific ipsecmethods declarations to ipsec_support.h.
melifaro added inline comments.Oct 3 2022, 5:07 PM
sys/netinet/sctp_usrreq.c
277

Worth mentioning out why we can reference ip+ header fields

281

C99 init?

sys/netinet/tcp_subr.c
2898

Worth clarifying in the description that no one reacts on HOSTDEAD.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 4 2022, 3:58 AM
Closed by commit rGc0fc81e9138b: netinet*: remove dead code from TCP, UDP, SCTP control input (authored by glebius). · Explain Why
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rGc0fc81e9138b: netinet*: remove dead code from TCP, UDP, SCTP control input.

Revision Contents
Changeset List

PathSize
sys/
netinet/
145 lines
95 lines
65 lines
netinet6/
196 lines
88 lines

Diff 111406

View Options

sys/netinet/sctp_usrreq.c

Loading...
View Options

sys/netinet/tcp_subr.c

Loading...
View Options

sys/netinet/udp_usrreq.c

Loading...
View Options

sys/netinet6/sctp6_usrreq.c

Loading...
View Options

sys/netinet6/udp6_usrreq.c

Loading...