Page MenuHomeFreeBSD

Fix harmless locking bug in tcp_fastopen_check_cookie()
ClosedPublic

Authored by pkelsey on Jan 25 2018, 6:12 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 25, 8:37 AM
Unknown Object (File)
Jan 31 2024, 3:08 AM
Unknown Object (File)
Jan 31 2024, 3:08 AM
Unknown Object (File)
Jan 31 2024, 3:08 AM
Unknown Object (File)
Jan 29 2024, 7:10 PM
Unknown Object (File)
Jan 10 2024, 1:45 AM
Unknown Object (File)
Nov 26 2023, 2:23 AM
Unknown Object (File)
Nov 26 2023, 2:23 AM
Subscribers

Details

Summary

The keylist lock was not being acquired early enough in tcp_fastopen_check_cookie().

This opens a potential race between keylist updates and generating a cookie to give to a client who has not supplied a valid one, which can result in the effective add time of a new key being slightly later that it would have been otherwise, as viewed by the client. There is no danger here, but it should be fixed.

Test Plan

Found by inspection.

Continued correct TFO server operation can be verified using the tools at https://people.freebsd.org/~pkelsey/tfo-tools/

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 14570