Page MenuHomeFreeBSD

tcp: The rack stack can incorrectly have an overflow when calculating a burst delay.
ClosedPublic

Authored by rrs on Oct 26 2021, 11:47 AM.
Tags
None
Referenced Files
F107325399: D32668.diff
Sun, Jan 12, 12:48 PM
Unknown Object (File)
Wed, Jan 8, 7:29 AM
Unknown Object (File)
Tue, Jan 7, 3:54 AM
Unknown Object (File)
Nov 20 2024, 11:01 AM
Unknown Object (File)
Nov 18 2024, 7:23 PM
Unknown Object (File)
Nov 18 2024, 5:44 AM
Unknown Object (File)
Nov 14 2024, 8:37 PM
Unknown Object (File)
Nov 14 2024, 12:43 PM
Subscribers

Details

Summary

If the congestion window is very large the fact that we multiply it by 1000 (for microseconds) can
cause the uint32_t to overflow and we incorrectly calculate a very small divisor. This will then
cause the burst timer to be very large when it should be 0. Instead lets make the three variables
uint64_t and avoid the issue.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

rrs requested review of this revision.Oct 26 2021, 11:47 AM

While where at it, there is a duplicate srtt that should be just one var at the higher scope. Lets fix
that and make the needed casts when comparing to slot. That way we never can get a
confused compiler :)

This revision is now accepted and ready to land.Oct 26 2021, 3:37 PM