[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