NAT64: coverity fix 84/7084/3
authorMatus Fabian <matfabia@cisco.com>
Fri, 9 Jun 2017 11:55:54 +0000 (04:55 -0700)
committerOle Trøan <otroan@employees.org>
Mon, 12 Jun 2017 10:04:27 +0000 (10:04 +0000)
Change-Id: I08544b2f06f993c8a6435901232a0616d0548b94
Signed-off-by: Matus Fabian <matfabia@cisco.com>
src/plugins/snat/nat64_cli.c

index 3fad75e..106d9ae 100644 (file)
@@ -278,8 +278,8 @@ nat64_add_del_static_bib_command_fn (vlib_main_t *
   u8 is_add = 1;
   ip6_address_t in_addr;
   ip4_address_t out_addr;
-  u16 in_port;
-  u16 out_port;
+  u16 in_port = 0;
+  u16 out_port = 0;
   u32 vrf_id = 0;
   snat_protocol_t proto = 0;
   u8 p = 0;
@@ -310,6 +310,18 @@ nat64_add_del_static_bib_command_fn (vlib_main_t *
        }
     }
 
+  if (!in_port)
+    {
+      error = clib_error_return (0, "inside port and address  must be set");
+      goto done;
+    }
+
+  if (!out_port)
+    {
+      error = clib_error_return (0, "outside port and address  must be set");
+      goto done;
+    }
+
   p = snat_proto_to_ip_proto (proto);
 
   rv =