Fixed coverity issue in acl_auto_configure.cpp: Type: Unused value (U…

…NUSED_VALUE)
intel · Dec 13, 2022 · 52b0682 · 52b0682
1 parent 5994a1c
commit 52b0682
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/src/acl_auto_configure.cpp b/src/acl_auto_configure.cpp
@@ -724,14 +724,12 @@ static bool read_accel_defs(const std::string &config_str,
     accel[i].mem.next = reinterpret_cast<void *>(0x00020000);
 
     int total_fields_kernel = 0;
-    if (result) {
-      result = read_int_counters(config_str, curr_pos, total_fields_kernel,
-                                 counters);
-    }
+    result = result && read_int_counters(config_str, curr_pos,
+                                         total_fields_kernel, counters);
     counters.emplace_back(total_fields_kernel);
 
-    result =
-        read_string_counters(config_str, curr_pos, hal_info[i].name, counters);
+    result = result && read_string_counters(config_str, curr_pos,
+                                            hal_info[i].name, counters);
 
     if (!result)
       break;