ip: coverity fix 38/23838/2
authorjohny <jan.cavojsky@pantheon.tech>
Fri, 6 Dec 2019 12:58:35 +0000 (13:58 +0100)
committerOle Trøan <otroan@employees.org>
Sun, 8 Dec 2019 19:02:11 +0000 (19:02 +0000)
Delete Null-checking "p0" in function ip6_tcp_udp_icmp_bad_length,
because it's not necessary.

Type: fix

Signed-off-by: johny <jan.cavojsky@pantheon.tech>
Change-Id: I2bf43a60c1c1d76e42581df27f2285c9e9563093

src/vnet/ip/ip6_forward.c

index 959ff89..128204f 100644 (file)
@@ -1161,15 +1161,13 @@ ip6_tcp_udp_icmp_bad_length (vlib_main_t * vm, vlib_buffer_t * p0)
   payload_length_host_byte_order = clib_net_to_host_u16 (ip0->payload_length);
   n_bytes_left = n_this_buffer = payload_length_host_byte_order;
 
-  if (p0)
+
+  u32 n_ip_bytes_this_buffer =
+    p0->current_length - (((u8 *) ip0 - p0->data) - p0->current_data);
+  if (n_this_buffer + headers_size > n_ip_bytes_this_buffer)
     {
-      u32 n_ip_bytes_this_buffer =
-       p0->current_length - (((u8 *) ip0 - p0->data) - p0->current_data);
-      if (n_this_buffer + headers_size > n_ip_bytes_this_buffer)
-       {
-         n_this_buffer = p0->current_length > headers_size ?
-           n_ip_bytes_this_buffer - headers_size : 0;
-       }
+      n_this_buffer = p0->current_length > headers_size ?
+       n_ip_bytes_this_buffer - headers_size : 0;
     }
 
   n_bytes_left -= n_this_buffer;