diff options
author | 2017-01-30 06:44:58 -0800 | |
---|---|---|
committer | 2017-01-31 12:31:59 +0000 | |
commit | 05b2bf2e31a90d97425230dcbf2bca6df54519ff (patch) | |
tree | f960625681b74ff857e365eac0ddaa3e782282c8 /src/vat/api_format.c | |
parent | 676112f81e52cd91c88a7ab985e957bd64baafd0 (diff) |
MFIB Coverity warnings. The lock macro is functionally equivalent but more expressive (and might appease coverity)
Change-Id: Ie3c9b2896a487a0302903bfbdd6348f6f091c67d
Signed-off-by: Neale Ranns <nranns@cisco.com>
Diffstat (limited to 'src/vat/api_format.c')
-rw-r--r-- | src/vat/api_format.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/vat/api_format.c b/src/vat/api_format.c index d4834b70..26df1aff 100644 --- a/src/vat/api_format.c +++ b/src/vat/api_format.c @@ -6706,9 +6706,6 @@ api_ip_mroute_add_del (vat_main_t * vam) S; /* Wait for a reply... */ W; - - /* Return the good/bad news */ - return (vam->retval); } static int @@ -16546,7 +16543,10 @@ static void vat_json_object_add_uint (node, "src-if-index", sw_if_index_from); vat_json_object_add_string_copy (node, "src-if-name", sw_if_from_name); vat_json_object_add_uint (node, "dst-if-index", sw_if_index_to); - vat_json_object_add_string_copy (node, "dst-if-name", sw_if_to_name); + if (0 != sw_if_to_name) + { + vat_json_object_add_string_copy (node, "dst-if-name", sw_if_to_name); + } vat_json_object_add_uint (node, "state", mp->state); } |