punt.c -- coverity woes 44/18044/2
authorSteven Luong <sluong@cisco.com>
Tue, 5 Mar 2019 23:57:52 +0000 (15:57 -0800)
committerNeale Ranns <nranns@cisco.com>
Wed, 6 Mar 2019 07:13:47 +0000 (07:13 +0000)
Coverity complains about identical code is executed for if and else branch.
Clean them up by removing the useless code.

Change-Id: Ie53f1dff055440ab2c3c3d2ea91edb1e50204b38
Signed-off-by: Steven Luong <sluong@cisco.com>
src/vnet/ip/punt.c

index 6e2fceb..e1c13f6 100644 (file)
@@ -807,7 +807,6 @@ punt_cli (vlib_main_t * vm,
   if (error)
     {
       clib_error_report (error);
-      goto done;
     }
 
 done:
@@ -877,10 +876,6 @@ punt_socket_register_cmd (vlib_main_t * vm,
   error =
     vnet_punt_socket_add (vm, 1, is_ipv4, protocol, port,
                          (char *) socket_name);
-  if (error)
-    {
-      goto done;
-    }
 done:
   return error;
 }
@@ -931,10 +926,6 @@ punt_socket_deregister_cmd (vlib_main_t * vm,
     }
 
   error = vnet_punt_socket_del (vm, is_ipv4, protocol, port);
-  if (error)
-    {
-      goto done;
-    }
 done:
   return error;
 }