[rtems commit] grtc.c: Fix four Missing break in switch Coverity errors

Joel Sherrill joel at rtems.org
Fri Mar 5 20:19:38 UTC 2021


Module:    rtems
Branch:    master
Commit:    13d6fae8dd495834d9d412b45da9c3deced7943a
Changeset: http://git.rtems.org/rtems/commit/?id=13d6fae8dd495834d9d412b45da9c3deced7943a

Author:    Ryan Long <ryan.long at oarcorp.com>
Date:      Mon Mar  1 15:33:06 2021 -0500

grtc.c: Fix four Missing break in switch Coverity errors

CID 1399761: Missing break in switch in process_dma().
CID 1399765: Missing break in switch in process_dma().
CID 1399766: Missing break in switch in process_dma().
CID 1399777: Missing break in switch in process_dma().

Closes #4277

---

 bsps/shared/grlib/tmtc/grtc.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/bsps/shared/grlib/tmtc/grtc.c b/bsps/shared/grlib/tmtc/grtc.c
index e9acea4..f734f2e 100644
--- a/bsps/shared/grlib/tmtc/grtc.c
+++ b/bsps/shared/grlib/tmtc/grtc.c
@@ -1365,6 +1365,7 @@ static int process_dma(struct grtc_priv *pDev)
 		/* Start of frame found, Try to copy header */
 		pDev->frm = NULL;
 		pDev->frame_state = FRM_STATE_HDR;
+		/* Fall through */
 
 		case FRM_STATE_HDR:
 		DBG2("FRAME_STATE_HDR\n");
@@ -1396,6 +1397,7 @@ static int process_dma(struct grtc_priv *pDev)
 			return -1;
 		}
 		pDev->frame_state = FRM_STATE_ALLOC;
+		/* Fall through */
 
 		case FRM_STATE_ALLOC:
 		DBG2("FRAME_STATE_ALLOC\n");
@@ -1449,6 +1451,7 @@ static int process_dma(struct grtc_priv *pDev)
 
 		pDev->frame_state = FRM_STATE_PAYLOAD;
 		pDev->frm = frm;
+		/* Fall through */
 
 		case FRM_STATE_PAYLOAD:
 		DBG2("FRAME_STATE_PAYLOAD\n");
@@ -1476,6 +1479,7 @@ static int process_dma(struct grtc_priv *pDev)
 		}
 		frm->len += ret;
 		pDev->frame_state = FRM_STATE_FILLER;
+		/* Fall through */
 
 		case FRM_STATE_FILLER:
 		DBG2("FRAME_STATE_FILLER\n");



More information about the vc mailing list