Analyse header layout in ether_extract_headers().
authorbluhm <bluhm@openbsd.org>
Tue, 13 Feb 2024 13:58:19 +0000 (13:58 +0000)
committerbluhm <bluhm@openbsd.org>
Tue, 13 Feb 2024 13:58:19 +0000 (13:58 +0000)
commite78a66e5775d3e9cf59b16f193be327ae6d56426
tree1dd1384f98d658653765a46f962a03580f54f2a2
parentfed3efa7ebe28a42f8790bbd65433d2483df6870
Analyse header layout in ether_extract_headers().

Several drivers need IPv4 header length and TCP offset for checksum
offload, TSO and LRO.  Accessing these fields directly caused crashes
on sparc64 due to misaligned access.  It cannot be guaranteed that
IP and TCP header is 4 byte aligned in driver level.  Also gcc 4.2.1
assumes that bit fields can be accessed with 32 bit load instructions.

Use memcpy() in ether_extract_headers() to get the bits from IPv4
and TCP header and store the header length in struct ether_extracted.
From there network drivers can esily use it without caring about
alignment and bit shift.  Do some sanity checks with the length
values to prevent that invalid values from evil packets get stored
into hardware registers.  If check fails, clear the pointer to the
header to hide it from the driver.  Add debug prints that help to
figure out the reason for bad packets and provide information when
debugging drivers.

OK mglocker@
sys/dev/pci/if_bnxt.c
sys/dev/pci/if_em.c
sys/dev/pci/if_igc.c
sys/dev/pci/if_ix.c
sys/dev/pci/if_ixl.c
sys/dev/pv/if_vio.c
sys/net/if_ethersubr.c
sys/netinet/if_ether.h