[PATCH 2/3] cpuusagetop.c: Fix three Missing break in switch Coverity errors

Ryan Long thisisryanlong at gmail.com
Wed Mar 3 21:26:00 UTC 2021


CID 1399726: Missing break in switch in task_usage().
CID 1399728: Missing break in switch in task_usage().
CID 1399742: Missing break in switch in task_usage().

Closes #4278
---
 cpukit/libmisc/cpuuse/cpuusagetop.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/cpukit/libmisc/cpuuse/cpuusagetop.c b/cpukit/libmisc/cpuuse/cpuusagetop.c
index 9f19b27..dad11ad 100644
--- a/cpukit/libmisc/cpuuse/cpuusagetop.c
+++ b/cpukit/libmisc/cpuuse/cpuusagetop.c
@@ -228,9 +228,11 @@ task_usage(Thread_Control* thread, void* arg)
           if (CPU_usage_Equal_to(&usage, &data->zero) ||
               CPU_usage_Less_than(&usage, &data->usage[j]))
             continue;
+          /* Fall through */
         case RTEMS_TOP_SORT_REAL_PRI:
           if (thread->Real_priority.priority > data->tasks[j]->Real_priority.priority)
             continue;
+          /* Fall through */
         case RTEMS_TOP_SORT_CURRENT_PRI:
           if (
             _Thread_Get_priority( thread )
@@ -238,6 +240,7 @@ task_usage(Thread_Control* thread, void* arg)
           ) {
             continue;
           }
+          /* Fall through */
         case RTEMS_TOP_SORT_ID:
           if (thread->Object.id < data->tasks[j]->Object.id)
             continue;
-- 
1.8.3.1



More information about the devel mailing list