commit 2951d901a5b6ac48b0203b2da9e35ea419b9b4d6
author Chris Luke <chrisy@flirble.org> Tue Sep 05 11:59:55 2017 -0400
committer Dave Wallace <dwallacelf@gmail.com> Wed Sep 06 16:29:49 2017 +0000
tree e3d28e4997d6e51c1a44d69fde60266771ada9a7
parent 704018cf117b6667f08b09d6db5fbec105bf6d57

Fixes for issues raised by Coverity (VPP-972)

Change-Id: I4b1f27b95d67d48b7a13750ff8754c344ed7afa7
Signed-off-by: Chris Luke <chrisy@flirble.org>

src/plugins/memif/memif_test.c

diff --git a/src/plugins/memif/memif_test.c b/src/plugins/memif/memif_test.c
index cbef4df..a7c23da 100644
--- a/src/plugins/memif/memif_test.c
+++ b/src/plugins/memif/memif_test.c
@@ -186,10 +186,10 @@
       strncpy ((char *) mp->socket_filename, (char *) socket_filename, 127);
       vec_free (socket_filename);
     }
-  if (socket_filename != 0)
+  if (secret != 0)
     {
       strncpy ((char *) mp->secret, (char *) secret, 16);
-      vec_free (socket_filename);
+      vec_free (secret);
     }
   memcpy (mp->hw_addr, hw_addr, 6);
   mp->rx_queues = rx_queues;

src/vnet/mpls/mpls_api.c

diff --git a/src/vnet/mpls/mpls_api.c b/src/vnet/mpls/mpls_api.c
index 2af6af8..a44b1a2 100644
--- a/src/vnet/mpls/mpls_api.c
+++ b/src/vnet/mpls/mpls_api.c
@@ -68,9 +68,6 @@
   vnm = vnet_get_main ();
   vnm->api_errno = 0;
 
-
-  rv = (rv == 0) ? vnm->api_errno : rv;
-
   REPLY_MACRO (VL_API_MPLS_TABLE_ADD_DEL_REPLY);
 }
 

src/vppinfra/bihash_template.c

diff --git a/src/vppinfra/bihash_template.c b/src/vppinfra/bihash_template.c
index 704d165..2a5a5cd 100644
--- a/src/vppinfra/bihash_template.c
+++ b/src/vppinfra/bihash_template.c
@@ -509,10 +509,11 @@
 
   for (i = 0; i < BIHASH_KVP_CACHE_SIZE; i++)
     s = format (s, "[%d] ", ((cache_lru >> (3 * i)) & 7));
+
+  return (s);
 #else
   return format (s, "cache not configured");
 #endif
-  return (s);
 }
 
 void