The branch main has been updated by tuexen:
URL:
https://cgit.FreeBSD.org/src/commit/?id=a7ba00a438a64b34219f2990b3f7348126f7b4ab
commit a7ba00a438a64b34219f2990b3f7348126f7b4ab
Author: Michael Tuexen <tue...@freebsd.org>
AuthorDate: 2021-12-28 09:23:31 +0000
Commit: Michael Tuexen <tue...@freebsd.org>
CommitDate: 2021-12-28 09:23:31 +0000
sctp: minor improvements in sctp_get_frag_point
MFC after: 3 days
---
sys/netinet/sctp_output.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/sys/netinet/sctp_output.c b/sys/netinet/sctp_output.c
index e77517de2973..17a4df2243e2 100644
--- a/sys/netinet/sctp_output.c
+++ b/sys/netinet/sctp_output.c
@@ -6237,13 +6237,15 @@ sctp_get_frag_point(struct sctp_tcb *stcb)
overhead += sctp_get_auth_chunk_len(asoc->peer_hmac_id);
}
} else {
- overhead += sizeof(struct sctp_idata_chunk);
+ overhead += sizeof(struct sctp_data_chunk);
if (sctp_auth_is_required_chunk(SCTP_DATA,
asoc->peer_auth_chunks)) {
overhead += sctp_get_auth_chunk_len(asoc->peer_hmac_id);
}
}
+ KASSERT(overhead % 4 == 0,
+ ("overhead (%u) not a multiple of 4", overhead));
/* Consider padding. */
- if (asoc->smallest_mtu % 4) {
+ if (asoc->smallest_mtu % 4 > 0) {
overhead += (asoc->smallest_mtu % 4);
}
KASSERT(asoc->smallest_mtu > overhead,
