]> git.proxmox.com Git - ovs.git/commit - tests/system-traffic.at
conntrack: Fix fragmentation checks.
authorDarrell Ball <dlu998@gmail.com>
Fri, 29 Jun 2018 06:39:47 +0000 (23:39 -0700)
committerBen Pfaff <blp@ovn.org>
Thu, 5 Jul 2018 20:22:36 +0000 (13:22 -0700)
commite917d3ee9a1953dc60d8643baeca83d2843518b8
tree4179d9e01c98b2388b99f4f34623e4fa1d586273
parentcdc9a84ad2e3475ce30e81018cbb91680279983c
conntrack: Fix fragmentation checks.

The ipv4 fragmentation check is broken and allows fragments through.
There were fragile and poorly maintainable checks in extract_l3_ipv*
designed to save a few cycles.  The checks make assumptions about what
sanity checks may have been done and could be skipped based on inferring
from the value of another paramater that should be unrelated (l4
pointer needing assignment).  Since the benefit is minimal, remove
the special checks and always do sanity checks.

Four tests are added to better maintain fragmentation support.

This needs backporting to 2.9.

Fixes: c8b1ad49da68("conntrack: Reorder sanity checks in extract_l3_ipvx().")
Fixes: a489b16854b5("conntrack: New userspace connection tracker.")
Signed-off-by: Darrell Ball <dlu998@gmail.com>
lib/conntrack.c
tests/system-traffic.at