Remove dead code in pf_pull_hdr().
authorbluhm <bluhm@openbsd.org>
Tue, 10 Oct 2023 11:25:31 +0000 (11:25 +0000)
committerbluhm <bluhm@openbsd.org>
Tue, 10 Oct 2023 11:25:31 +0000 (11:25 +0000)
commit46650f23db713388da0ecb54ba1bc00221ce6b2d
treeb6c375be130b32ad636bb1089594fc6a395d5f07
parentaa41ab081c55d8a1e87222abe01b2e13009a5339
Remove dead code in pf_pull_hdr().

pf_pull_hdr() allows to pass an action pointer parameter as output
value.  This is never used, all callers pass a NULL argument.  Remove
ACTION_SET() entirely.

The logic (fragoff >= len) in pf_pull_hdr() does not work since
revision 1.4.  Before it was used to drop short TCP or UDP fragments
that contained only part of the header.  Current code in pf_pull_hdr()
drops the packets anyway, so always set reason PFRES_FRAG.

OK kn@ sashan@
sys/net/pf.c
sys/net/pf_norm.c
sys/net/pf_osfp.c
sys/net/pfvar.h