diff options
Diffstat (limited to 'src/vnet/ip')
-rw-r--r-- | src/vnet/ip/ip4_format.c | 7 | ||||
-rw-r--r-- | src/vnet/ip/ip4_forward.c | 4 | ||||
-rw-r--r-- | src/vnet/ip/ip4_pg.c | 6 |
3 files changed, 9 insertions, 8 deletions
diff --git a/src/vnet/ip/ip4_format.c b/src/vnet/ip/ip4_format.c index 786a01d396b..c6639b20716 100644 --- a/src/vnet/ip/ip4_format.c +++ b/src/vnet/ip/ip4_format.c @@ -150,9 +150,10 @@ format_ip4_header (u8 * s, va_list * args) /* Check and report invalid checksums. */ { - u16 c = ip4_header_checksum (ip); - if (c != ip->checksum) - s = format (s, " (should be 0x%04x)", clib_net_to_host_u16 (c)); + if (!ip4_header_checksum_is_valid (ip)) + s = + format (s, " (should be 0x%04x)", + clib_net_to_host_u16 (ip4_header_checksum (ip))); } s = format (s, " dscp %U ecn %U", diff --git a/src/vnet/ip/ip4_forward.c b/src/vnet/ip/ip4_forward.c index 595a0a1913c..3bf305303d4 100644 --- a/src/vnet/ip/ip4_forward.c +++ b/src/vnet/ip/ip4_forward.c @@ -2063,7 +2063,7 @@ ip4_ttl_inc (vlib_buffer_t * b, ip4_header_t * ip) ttl += 1; ip->ttl = ttl; - ASSERT (ip->checksum == ip4_header_checksum (ip)); + ASSERT (ip4_header_checksum_is_valid (ip)); } /* Decrement TTL & update checksum. @@ -2104,7 +2104,7 @@ ip4_ttl_and_checksum_check (vlib_buffer_t * b, ip4_header_t * ip, u16 * next, } /* Verify checksum. */ - ASSERT ((ip->checksum == ip4_header_checksum (ip)) || + ASSERT (ip4_header_checksum_is_valid (ip) || (b->flags & VNET_BUFFER_F_OFFLOAD_IP_CKSUM)); } diff --git a/src/vnet/ip/ip4_pg.c b/src/vnet/ip/ip4_pg.c index d89424496f0..2ccd2b45a6f 100644 --- a/src/vnet/ip/ip4_pg.c +++ b/src/vnet/ip/ip4_pg.c @@ -90,8 +90,8 @@ compute_length_and_or_checksum (vlib_main_t * vm, ip0->checksum = ~ip_csum_fold (sum0); ip1->checksum = ~ip_csum_fold (sum1); - ASSERT (ip0->checksum == ip4_header_checksum (ip0)); - ASSERT (ip1->checksum == ip4_header_checksum (ip1)); + ASSERT (ip4_header_checksum_is_valid (ip0)); + ASSERT (ip4_header_checksum_is_valid (ip1)); } } @@ -123,7 +123,7 @@ compute_length_and_or_checksum (vlib_main_t * vm, ip4_partial_header_checksum_x1 (ip0, sum0); ip0->checksum = ~ip_csum_fold (sum0); - ASSERT (ip0->checksum == ip4_header_checksum (ip0)); + ASSERT (ip4_header_checksum_is_valid (ip0)); } } } |