]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
MFC r317186
authorkp <kp@FreeBSD.org>
Sun, 23 Apr 2017 08:58:50 +0000 (08:58 +0000)
committerkp <kp@FreeBSD.org>
Sun, 23 Apr 2017 08:58:50 +0000 (08:58 +0000)
commit573c4b124ffed246bfbbb7b0d6a99245b7036a71
treea0123f3877952e9833b0918f2a47f12f4b90304c
parent703772c814eedcb43057d622868a158f654ffdad
MFC r317186

pf: Fix possible incorrect IPv6 fragmentation

When forwarding pf tracks the size of the largest fragment in a fragmented
packet, and refragments based on this size.
It failed to ensure that this size was a multiple of 8 (as is required for all
but the last fragment), so it could end up generating incorrect fragments.

For example, if we received an 8 byte and 12 byte fragment pf would emit a first
fragment with 12 bytes of payload and the final fragment would claim to be at
offset 8 (not 12).

We now assert that the fragment size is a multiple of 8 in ip6_fragment(), so
other users won't make the same mistake.

Reported by: Antonios Atlasis <aatlasis at secfu net>
sys/netinet6/ip6_output.c
sys/netpfil/pf/pf_norm.c