tag packets going out a sec interface to prevent route/encap loops.
authordlg <dlg@openbsd.org>
Wed, 24 Jan 2024 00:17:01 +0000 (00:17 +0000)
committerdlg <dlg@openbsd.org>
Wed, 24 Jan 2024 00:17:01 +0000 (00:17 +0000)
commit2d1127bb1f3769fa5bd5b17c7dfe0eacede81ca4
treeea4ef140bc5fc14a232209b15626e0492a7f9040
parent83d14a11301aad2b03fc3cb4bd68dfa7d1591840
tag packets going out a sec interface to prevent route/encap loops.

sec(4) was already looking for this mbuf tag so it could drop packets
that had already been sent out on the same interface, but i forgot
the code that adds the tag.

this was reported by jason tubnor who experienced spins/lockups
when using sec and a physical interface was disconnected. rather
than being a locking problem like we initially assumed, it turned
out that unplugging a physical interface caused a route for ipsec
encapsulated traffic to go out over sec(4), causing the packet to
loop in the stack.

the fix was also tested and verified by jason. sorry for taking so
long to look at it.
sys/net/if_sec.c