...
Recommendation | Severity | Likelihood | Remediation Cost | Priority | Level |
---|
MSC01-C | medium | probable | medium | P8 | L2 |
Automated Detection
Tool | Version | Checker | Description |
---|
| | | Section |
---|
Fully Implemented |
|
| | | Section |
---|
can detect some violations of this recommendation when the -Wswitch and -Wswitch-default flags are used |
|
| | | Section |
---|
can detect some violations of this recommendation. In particular, it flags switch statements that do not have a default clause. ROSE should also detect "fake switches," as well (that is, a chain of if statements each checking the value of the same variable). These if statements should always end in an "else" clause, or they should mathematically cover every possibility. For instance, consider the following: Code Block |
---|
|
if (x > 0) {
/* ... */
} else if (x < 0) {
/* ... */
} else if (x == 0) {
/* ... */
}
|
|
|
| | | |
...
ISO/IEC TR 24772 "CLL Switch statements and static analysis"
Bibliography
Wiki Markup |
\[[Hatton 1995|AA. Bibliography#Hatton 95]\] Section 2.7.2, "Errors of omission and addition"
\
[[Viega 2005|AA. Bibliography#Viega 05] \] Section 5.2.17, "Failure to account for default case in switch"
\[
[http://www.aeroxp.org/2009/01/lesson-on-infinite-loops]\] for analysis on the Zune 30 bug
...
49. Miscellaneous (MSC) MSC02-C. Avoid errors of omission