diff options
author | Kristof Provost <kp@FreeBSD.org> | 2024-05-06 07:43:49 +0000 |
---|---|---|
committer | Kristof Provost <kp@FreeBSD.org> | 2024-05-08 11:19:05 +0000 |
commit | 301ec2cebb6a3e472af39eb7b98c3b7ff88e04b0 (patch) | |
tree | 8ab882170eec6b76d35cbba55385ad1301a3833c /release/Makefile.inc1 | |
parent | 0d446a4303a8444c5ba9c7d4a3caee846579d101 (diff) | |
download | src-main.tar.gz src-main.zip |
Users have reported crashes in pf_test_state_udp() where at least one state key
is NULL.
That suggests that pf_detach_state() ran concurrently with pf_test_state_udp().
pf_test_state_udp() holds the state lock (aka the id lock), but
pf_detach_state() does not.
The intent is that detached states are not returned by STATE_LOOKUP/
pf_find_state(), as the state's timeout is set to PFTM_UNLINKED and thus
pf_find_state() does not find the state.
There are other paths to pf_detach_state() (outside of pf_unlink_state())
though, where we did not set the timeout to PFTM_UNLINKED. Fix those, and assert
that the timeout is set correctly when we enter pf_detach_state().
MFC after: 1 week
See also: https://redmine.pfsense.org/issues/15413
Sponsored by: Rubicon Communications, LLC ("Netgate")
Differential Revision: https://reviews.freebsd.org/D45101
Diffstat (limited to 'release/Makefile.inc1')
0 files changed, 0 insertions, 0 deletions