diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2022-10-19 11:43:44 +1100 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2022-10-19 03:34:21 +0200 |
commit | 7abd2b0d727b4dad0f92d8c3130931d023e58514 (patch) | |
tree | 04faf9fbd338dd346543f06945ee4da0c7624f22 /tap.c | |
parent | b3f359167be0ca9a6fea2554b2e2545177181269 (diff) | |
download | passt-7abd2b0d727b4dad0f92d8c3130931d023e58514.tar passt-7abd2b0d727b4dad0f92d8c3130931d023e58514.tar.gz passt-7abd2b0d727b4dad0f92d8c3130931d023e58514.tar.bz2 passt-7abd2b0d727b4dad0f92d8c3130931d023e58514.tar.lz passt-7abd2b0d727b4dad0f92d8c3130931d023e58514.tar.xz passt-7abd2b0d727b4dad0f92d8c3130931d023e58514.tar.zst passt-7abd2b0d727b4dad0f92d8c3130931d023e58514.zip |
Add csum_icmp6() helper for calculating ICMPv6 checksums
At least two places in passt calculate ICMPv6 checksums, ndp() and
tap_ip_send(). Add a helper to handle this calculation in both places.
For future flexibility, the new helper takes parameters for the fields in
the IPv6 pseudo-header, so an IPv6 header or pseudo-header doesn't need to
be explicitly constructed. It also allows the ICMPv6 header and payload to
be in separate buffers, although we don't use this yet.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'tap.c')
-rw-r--r-- | tap.c | 6 |
1 files changed, 2 insertions, 4 deletions
@@ -191,10 +191,8 @@ void tap_ip_send(const struct ctx *c, const struct in6_addr *src, uint8_t proto, } else if (proto == IPPROTO_ICMPV6) { struct icmp6hdr *ih = (struct icmp6hdr *)(ip6h + 1); - ih->icmp6_cksum = 0; - ih->icmp6_cksum = csum_unaligned(ip6h, - len + sizeof(*ip6h), - 0); + csum_icmp6(ih, &ip6h->saddr, &ip6h->daddr, + ih + 1, len - sizeof(*ih)); } ip6h->version = 6; ip6h->nexthdr = proto; |