From 5df628bb8a30b69f223a7c8b017dcba770c87e43 Mon Sep 17 00:00:00 2001 From: Mohsin Kazmi Date: Mon, 1 Oct 2018 17:41:08 +0200 Subject: [PATCH] vat: fix coverity warning for show_threads Change-Id: I8c226b91cc45f6f345444afd8d168e1292371582 Signed-off-by: Mohsin Kazmi --- src/vat/api_format.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/vat/api_format.c b/src/vat/api_format.c index 583185e4d5f..96a79bf251b 100644 --- a/src/vat/api_format.c +++ b/src/vat/api_format.c @@ -1347,10 +1347,14 @@ static void vl_api_show_threads_reply_t_handler_json vat_main_t *vam = &vat_main; vat_json_node_t node; vl_api_thread_data_t *td; - int i, count = ntohl (mp->count); + i32 retval = ntohl (mp->retval); + int i, count = 0; + + if (retval >= 0) + count = ntohl (mp->count); vat_json_init_object (&node); - vat_json_object_add_int (&node, "retval", ntohl (mp->retval)); + vat_json_object_add_int (&node, "retval", retval); vat_json_object_add_uint (&node, "count", count); for (i = 0; i < count; i++) @@ -1368,7 +1372,7 @@ static void vl_api_show_threads_reply_t_handler_json vat_json_print (vam->ofp, &node); vat_json_free (&node); - vam->retval = ntohl (mp->retval); + vam->retval = retval; vam->result_ready = 1; } -- 2.16.6