diff options
| author |   Mohammed Hawari <mohammed@hawari.fr> | 2021-05-07 17:27:51 +0200 |
|---|---|---|
| committer |   Damjan Marion <dmarion@me.com> | 2021-05-08 08:56:01 +0000 |
| commit | 90ce4fd180272d3a6fec071b6d2a08650ffd9113 (patch) | |
| tree | 4651bb435267ea86d76b3817471d270cc0858c92 /src/plugins/avf | |
| parent | c44402f7b54d3290e7d1605e8e855f859ffe5331 (diff) | |
avf: fix coverity
Change-Id: I54025eb0e4ee861b2b2fed554b006405cba38bb9
Signed-off-by: Mohammed Hawari <mohammed@hawari.fr>
Type: fix
Fixes: bd376a8ce8425f45b23074398949e8055de29280
Diffstat (limited to 'src/plugins/avf')
| -rw-r--r-- | src/plugins/avf/output.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/src/plugins/avf/output.c b/src/plugins/avf/output.c index decd8b5b939..78043358e77 100644 --- a/src/plugins/avf/output.c +++ b/src/plugins/avf/output.c @@ -81,14 +81,13 @@ avf_tx_prepare_cksum (vlib_buffer_t * b, u8 is_tso) ip6_header_t *ip6 = (void *) (b->data + l3_hdr_offset); tcp_header_t *tcp = (void *) (b->data + l4_hdr_offset); udp_header_t *udp = (void *) (b->data + l4_hdr_offset); - u16 l4_len = - is_tcp ? tcp_header_bytes (tcp) : is_udp ? sizeof (udp_header_t) : 0; + u16 l4_len = is_tcp ? tcp_header_bytes (tcp) : sizeof (udp_header_t); u16 sum = 0; flags |= AVF_TXD_OFFSET_MACLEN (l2_len) | AVF_TXD_OFFSET_IPLEN (l3_len) | AVF_TXD_OFFSET_L4LEN (l4_len); flags |= is_ip4 ? AVF_TXD_CMD_IIPT_IPV4 : AVF_TXD_CMD_IIPT_IPV6; - flags |= is_tcp ? AVF_TXD_CMD_L4T_TCP : is_udp ? AVF_TXD_CMD_L4T_UDP : 0; + flags |= is_tcp ? AVF_TXD_CMD_L4T_TCP : AVF_TXD_CMD_L4T_UDP; if (is_ip4) ip4->checksum = 0; @@ -101,8 +100,6 @@ avf_tx_prepare_cksum (vlib_buffer_t * b, u8 is_tso) ip6->payload_length = 0; } - if (is_tcp || is_udp) - { if (is_ip4) { struct avf_ip4_psh psh = { 0 }; @@ -124,12 +121,12 @@ avf_tx_prepare_cksum (vlib_buffer_t * b, u8 is_tso) psh.l4len = is_tso ? 0 : ip6->payload_length; sum = ~ip_csum (&psh, sizeof (psh)); } - } + /* ip_csum does a byte swap for some reason... */ sum = clib_net_to_host_u16 (sum); if (is_tcp) tcp->checksum = sum; - else if (is_udp) + else udp->checksum = sum; return flags; } |