diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2024-05-01 16:53:48 +1000 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2024-05-02 16:13:21 +0200 |
commit | 9e22c53aa92552bd5c015c2597512056f8def4d8 (patch) | |
tree | b4068984c67af419bbeaf6aad34c9d6921b5d034 /arp.c | |
parent | 1095a7b0c9a150cb488ff5bd5fd74c897dd9236e (diff) | |
download | passt-9e22c53aa92552bd5c015c2597512056f8def4d8.tar passt-9e22c53aa92552bd5c015c2597512056f8def4d8.tar.gz passt-9e22c53aa92552bd5c015c2597512056f8def4d8.tar.bz2 passt-9e22c53aa92552bd5c015c2597512056f8def4d8.tar.lz passt-9e22c53aa92552bd5c015c2597512056f8def4d8.tar.xz passt-9e22c53aa92552bd5c015c2597512056f8def4d8.tar.zst passt-9e22c53aa92552bd5c015c2597512056f8def4d8.zip |
checksum: Make csum_ip4_header() take a host endian length
csum_ip4_header() takes the packet length as a network endian value. In
general it's very error-prone to pass non-native-endian values as a raw
integer. It's particularly bad here because this differs from other
checksum functions (e.g. proto_ipv4_header_psum()) which take host native
lengths.
It turns out all the callers have easy access to the native endian value,
so switch it to use host order like everything else.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'arp.c')
0 files changed, 0 insertions, 0 deletions