diff options
| author |   Chris Luke <chrisy@flirble.org> | 2017-09-05 11:59:55 -0400 |
|---|---|---|
| committer |   Dave Wallace <dwallacelf@gmail.com> | 2017-09-06 16:29:49 +0000 |
| commit | 2951d901a5b6ac48b0203b2da9e35ea419b9b4d6 (patch) | |
| tree | e3d28e4997d6e51c1a44d69fde60266771ada9a7 | |
| parent | 704018cf117b6667f08b09d6db5fbec105bf6d57 (diff) | |
Fixes for issues raised by Coverity (VPP-972)
Change-Id: I4b1f27b95d67d48b7a13750ff8754c344ed7afa7
Signed-off-by: Chris Luke <chrisy@flirble.org>
| -rw-r--r-- | src/plugins/memif/memif_test.c | 4 | ||||
| -rw-r--r-- | src/vnet/mpls/mpls_api.c | 3 | ||||
| -rw-r--r-- | src/vppinfra/bihash_template.c | 3 |
3 files changed, 4 insertions, 6 deletions
diff --git a/src/plugins/memif/memif_test.c b/src/plugins/memif/memif_test.c index cbef4dfa..a7c23daa 100644 --- a/src/plugins/memif/memif_test.c +++ b/src/plugins/memif/memif_test.c @@ -186,10 +186,10 @@ api_memif_create (vat_main_t * vam) 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; diff --git a/src/vnet/mpls/mpls_api.c b/src/vnet/mpls/mpls_api.c index 2af6af8f..a44b1a25 100644 --- a/src/vnet/mpls/mpls_api.c +++ b/src/vnet/mpls/mpls_api.c @@ -68,9 +68,6 @@ vl_api_mpls_table_add_del_t_handler (vl_api_mpls_table_add_del_t * mp) vnm = vnet_get_main (); vnm->api_errno = 0; - - rv = (rv == 0) ? vnm->api_errno : rv; - REPLY_MACRO (VL_API_MPLS_TABLE_ADD_DEL_REPLY); } diff --git a/src/vppinfra/bihash_template.c b/src/vppinfra/bihash_template.c index 704d1659..2a5a5cd2 100644 --- a/src/vppinfra/bihash_template.c +++ b/src/vppinfra/bihash_template.c @@ -509,10 +509,11 @@ u8 *BV (format_bihash_lru) (u8 * s, va_list * args) 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 |