]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
pf: handle multiple IPv6 fragment headers
authorKristof Provost <kp@FreeBSD.org>
Thu, 13 Jul 2023 08:25:49 +0000 (10:25 +0200)
committerKristof Provost <kp@FreeBSD.org>
Fri, 4 Aug 2023 14:08:05 +0000 (16:08 +0200)
commit3a0461f23a4f4fe8fc82b3445285d3d07787b016
treed44ed1ccc7f2ba48f99579d0617b9eda1fecd111
parent06185987d36ffdeff4e34608bb910fe629c76da5
pf: handle multiple IPv6 fragment headers

With 'scrub fragment reassemble' if a packet contains multiple IPv6
fragment headers we would reassemble the packet and immediately
continue processing it.

That is, we'd remove the first fragment header and expect the next
header to be a final header (i.e. TCP, UDP, ICMPv6, ...). However, if
it's another fragment header we'd not treat the packet correctly.
That is, we'd fail to recognise the payload and treat it as if it were
an IPv6 fragment rather than as its actual payload.

Fix this by restarting the normalisation on the reassembled packet.
If there are multiple fragment headers drop the packet.

Reported by: Enrico Bassetti bassetti@di.uniroma1.it (NetSecurityLab @ Sapienza University of Rome)
MFC after: instant
Sponsored by: Rubicon Communications, LLC ("Netgate")

(cherry picked from commit 76afcbb52492f9b3e72ee7d4c4ed0a54c25e1c48)
sys/netpfil/pf/pf_norm.c