[SERVER-59468] Coverity analysis defect 120437: Missing break in switch Created: 20/Aug/21 Updated: 20/Sep/21 Resolved: 20/Sep/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Coverity Collector User | Assignee: | Ted Tuckman |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | coverity, greenerbuild, neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Operating System: | ALL | ||||||||
| Sprint: | QO 2021-10-04 | ||||||||
| Participants: | |||||||||
| Description |
|
Missing break in switch Execution falls through to the next case statement or default; this might indicate a common typo. Missing break statement between cases in switch statement |
| Comments |
| Comment by Kyle Suarez [ 20/Aug/21 ] |
|
All of the cases in this switch statement should be checked. Either each case needs a break, or the inner if statements need a default else, since as written it does look like a mistake. |