fix(system_monitor): incorrect counter increment in CPU Usage monitor (…

…autowarefoundation#1783) Signed-off-by: v-nakayama7440-esol <v-nakayama7440@esol.co.jp> Signed-off-by: v-nakayama7440-esol <v-nakayama7440@esol.co.jp> Co-authored-by: ito-san <57388357+ito-san@users.noreply.github.com>
boyali · Sep 28, 2022 · cdc3353 · cdc3353
1 parent a0c903d
commit cdc3353
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/system/system_monitor/src/cpu_monitor/cpu_monitor_base.cpp b/system/system_monitor/src/cpu_monitor/cpu_monitor_base.cpp
@@ -298,7 +298,8 @@ int CPUMonitorBase::CpuUsageToLevel(const std::string & cpu_name, float usage)
   if (usage >= usage_warn_) {
     if (usage_warn_check_cnt_[idx] < usage_warn_count_) {
       usage_warn_check_cnt_[idx]++;
-    } else {
+    }
+    if (usage_warn_check_cnt_[idx] >= usage_warn_count_) {
       level = DiagStatus::WARN;
     }
   } else {
@@ -307,7 +308,8 @@ int CPUMonitorBase::CpuUsageToLevel(const std::string & cpu_name, float usage)
   if (usage >= usage_error_) {
     if (usage_error_check_cnt_[idx] < usage_error_count_) {
       usage_error_check_cnt_[idx]++;
-    } else {
+    }
+    if (usage_error_check_cnt_[idx] >= usage_error_count_) {
       level = DiagStatus::ERROR;
     }
   } else {