tests: fix coverity warnings 99/34099/5
authorKlement Sekera <ksekera@cisco.com>
Wed, 13 Oct 2021 19:45:42 +0000 (21:45 +0200)
committerAndrew Yourtchenko <ayourtch@gmail.com>
Thu, 14 Oct 2021 07:12:43 +0000 (07:12 +0000)
Fix coverity warnings by initialising variables and removing dead code.

Type: fix
Signed-off-by: Klement Sekera <ksekera@cisco.com>
Change-Id: I0254b81020cdc3d7075df003309065438526918f

src/vpp-api/vapi/vapi_cpp_test.cpp

index 1dd58f8..c574dd7 100644 (file)
@@ -146,7 +146,6 @@ START_TEST (test_loopbacks_1)
   { // new context
     bool seen[num_ifs] = {0};
     Sw_interface_dump d (con);
-    d.get_request ().get_payload ();
     auto rv = d.execute ();
     ck_assert_int_eq (VAPI_OK, rv);
     WAIT_FOR_RESPONSE (d, rv);
@@ -186,7 +185,6 @@ START_TEST (test_loopbacks_1)
 
   { // new context
     Sw_interface_dump d (con);
-    d.get_request ().get_payload ();
     auto rv = d.execute ();
     ck_assert_int_eq (VAPI_OK, rv);
     WAIT_FOR_RESPONSE (d, rv);
@@ -207,7 +205,7 @@ END_TEST;
 
 struct Create_loopback_cb
 {
-  Create_loopback_cb () : called{0}, sw_if_index{0} {};
+  Create_loopback_cb () : called{ 0 }, sw_if_index{ 0 }, seen{ false } {};
   int called;
   u32 sw_if_index;
   bool seen;
@@ -222,7 +220,7 @@ struct Create_loopback_cb
 
 struct Delete_loopback_cb
 {
-  Delete_loopback_cb () : called{0}, sw_if_index{0} {};
+  Delete_loopback_cb () : called{ 0 }, sw_if_index{ 0 }, seen{ false } {};
   int called;
   u32 sw_if_index;
   bool seen;
@@ -302,7 +300,6 @@ START_TEST (test_loopbacks_2)
 
   Sw_interface_dump_cb<num_ifs> swdcb (ccbs);
   Sw_interface_dump d (con, std::ref (swdcb));
-  d.get_request ().get_payload ();
   auto rv = d.execute ();
   ck_assert_int_eq (VAPI_OK, rv);
   WAIT_FOR_RESPONSE (d, rv);
@@ -329,7 +326,6 @@ START_TEST (test_loopbacks_2)
 
   { // new context
     Sw_interface_dump d (con);
-    d.get_request ().get_payload ();
     auto rv = d.execute ();
     ck_assert_int_eq (VAPI_OK, rv);
     WAIT_FOR_RESPONSE (d, rv);