Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: Migrated to Confluence 4.0

...

Recommendation

Severity

Likelihood

Remediation Cost

Priority

Level

MSC01-C

medium

probable

medium

P8

L2

Automated Detection

Tool

Version

Checker

Description

Section

LDRA tool suite

Include Page
c:LDRA_Vc:
LDRA_V
Section

12 S

Section

Fully Implemented

Section

GCC

Include Page
c:GCC_Vc:
GCC_V

 

Section

can detect some violations of this recommendation when the -Wswitch and -Wswitch-default flags are used

Section

Compass/ROSE

 

 

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
bgColor#ccccff
  if (x > 0) {
	  /* ... */
  } else if (x < 0) {
    /* ... */
  } else if (x == 0) {
    /* ... */
  }
Section

Klocwork

Include Page
c:Klocwork_Vc:
Klocwork_V
Section

LA_UNUSED

 

...

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