[PATCH 0/3] Fix Missing break in switch Coverity issues

Gedare Bloom gedare at rtems.org
Fri Mar 5 15:27:19 UTC 2021


Should we add a macro for this, e.g., "RTEMS_CASE_NO_BREAK" so that we
can update them in future if needed for other tools?

Regardless of the addition of a macro these are fine, the grlib one
came through separately I think. But I'd like to see a macro proposed.

On Wed, Mar 3, 2021 at 2:26 PM Ryan Long <thisisryanlong at gmail.com> wrote:
>
> Hi,
>
> To fix these Coverity issues, I just needed to add a comment to tell
> Coverity to ignore the error. I did this using the same method as shown
> in this article
> https://www.synopsys.com/blogs/software-security/gimme-a-break/.
>
> Thanks,
> Ryan
>
> Ryan Long (3):
>   grtc.c: Fix four Missing break in switch Coverity errors
>   cpuusagetop.c: Fix three Missing break in switch Coverity errors
>   pci_cfg_read.c: Fix Missing break in switch (CID #1399723)
>
>  bsps/shared/grlib/tmtc/grtc.c       | 4 ++++
>  cpukit/libmisc/cpuuse/cpuusagetop.c | 3 +++
>  cpukit/libpci/pci_cfg_read.c        | 1 +
>  3 files changed, 8 insertions(+)
>
> --
> 1.8.3.1
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list