vat: fix coverity warning for show_threads 82/15082/2
authorMohsin Kazmi <sykazmi@cisco.com>
Mon, 1 Oct 2018 15:41:08 +0000 (17:41 +0200)
committerDamjan Marion <dmarion@me.com>
Mon, 1 Oct 2018 17:02:02 +0000 (17:02 +0000)
Change-Id: I8c226b91cc45f6f345444afd8d168e1292371582
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
src/vat/api_format.c

index 583185e..96a79bf 100644 (file)
@@ -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;
 }