Page MenuHomeFreeBSD

pf: initial SCTP support
ClosedPublic

Authored by kp on Jul 4 2023, 7:11 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 13, 12:09 PM
Unknown Object (File)
Tue, Nov 12, 10:07 AM
Unknown Object (File)
Thu, Nov 7, 1:54 PM
Unknown Object (File)
Thu, Nov 7, 1:43 PM
Unknown Object (File)
Thu, Nov 7, 12:44 PM
Unknown Object (File)
Thu, Nov 7, 12:26 PM
Unknown Object (File)
Thu, Nov 7, 12:18 PM
Unknown Object (File)
Thu, Nov 7, 12:17 PM

Details

Summary

Basic state tracking for SCTP. This means we scan through the packet to
identify the different chunks (so we can identify state changes).

MFC after: 3 weeks
Sponsored by: Orange Business Services

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 52412
Build 49303: arc lint + arc unit

Event Timeline

kp requested review of this revision.Jul 4 2023, 7:11 AM
tuexen added inline comments.
sys/netpfil/pf/pf_norm.c
2039

Does this loop provide a way for a DOS attack? Assume an attacker sends a lot of packets containing, for example, (1500 - 12) / 4 = 372 chunks?

2045

If an attacker sends a chunk with chunk_length == 0, doesn't this results in an endless loop?

2145

The same applies to INIT ACK and SHUTDOWN COMPLETE chunks. Why is this not tested?

Improve validation of INIT chunks

initiate_tag, num_inbound_streams and num_outbound_streams may not be zero.
v_tag must be zero.

Ensure that the INIT chunk is the only chunk in the packet.

  • rebase
  • use sctp_calculate_cksum() via pf_sctp_checksum()
This revision was not accepted when it landed; it landed in state Needs Review.Jul 21 2023, 10:34 AM
Closed by commit rG010ee43f5673: pf: initial SCTP support (authored by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.