SonarSource Rules
  • Products

    In-IDE

    Code Quality and Security in your IDE with SonarQube Ide

    IDE extension that lets you fix coding issues before they exist!

    Discover SonarQube for IDE

    SaaS

    Code Quality and Security in the cloud with SonarQube Cloud

    Setup is effortless and analysis is automatic for most languages

    Discover SonarQube Cloud

    Self-Hosted

    Code Quality and Security Self-Hosted with SonarQube Server

    Fast, accurate analysis; enterprise scalability

    Discover SonarQube Server
  • SecretsSecrets
  • ABAPABAP
  • AnsibleAnsible
  • ApexApex
  • AzureResourceManagerAzureResourceManager
  • CC
  • C#C#
  • C++C++
  • CloudFormationCloudFormation
  • COBOLCOBOL
  • CSSCSS
  • DartDart
  • DockerDocker
  • FlexFlex
  • GitHub ActionsGitHub Actions
  • GoGo
  • HTMLHTML
  • JavaJava
  • JavaScriptJavaScript
  • JSONJSON
  • JCLJCL
  • KotlinKotlin
  • KubernetesKubernetes
  • Objective CObjective C
  • PHPPHP
  • PL/IPL/I
  • PL/SQLPL/SQL
  • PythonPython
  • RPGRPG
  • RubyRuby
  • RustRust
  • ScalaScala
  • SwiftSwift
  • TerraformTerraform
  • TextText
  • TypeScriptTypeScript
  • T-SQLT-SQL
  • VB.NETVB.NET
  • VB6VB6
  • XMLXML
  • YAMLYAML
C

C static code analysis

Unique rules to find Bugs, Vulnerabilities, Security Hotspots, and Code Smells in your C code

  • All rules 315
  • Vulnerability13
  • Bug76
  • Security Hotspot19
  • Code Smell207

  • Quick Fix 19
Filtered: 15 rules found
misra-c++2008
    Impact
      Clean code attribute
        1. Pointer and reference parameters should be "const" if the corresponding object is not modified

           Code Smell
        2. "abort", "exit", "getenv" and "system" from <stdlib.h> should not be used

           Bug
        3. "offsetof" macro should not be used

           Code Smell
        4. There shall be at most one occurrence of the # or ## operators in a single macro definition

           Code Smell
        5. In the definition of a function-like macro, each instance of a parameter shall be enclosed in parentheses, unless it is used as the operand of # or ##

           Code Smell
        6. #include directives in a file should only be preceded by other preprocessor directives or comments

           Code Smell
        7. Functions should not be defined with a variable number of arguments

           Code Smell
        8. Evaluation of the operand to the sizeof operator shall not contain side effects

           Bug
        9. Switch statement conditions should not have essentially boolean type

           Code Smell
        10. Switch labels should not be nested inside non-switch blocks

           Code Smell
        11. A cast shall not remove any const or volatile qualification from the type of a pointer or reference

           Code Smell
        12. If a function has internal linkage then all re-declarations shall include the static storage class specifer

           Code Smell
        13. Functions should not be declared at block scope

           Code Smell
        14. Assembly language should be encapsulated and isolated

           Code Smell
        15. Control should not be transferred into a complex logic block using a "goto" or a "switch" statement

           Code Smell

        Control should not be transferred into a complex logic block using a "goto" or a "switch" statement

        intentionality - clear
        maintainability
        Code Smell
        • lock-in
        • cert
        • misra-c++2008
        • pitfall

        Why is this an issue?

        More Info

        Having a switch and its cases wholly encompassed by a control structure such as a try, @try, catch, @catch, or a loop is perfectly acceptable. (try and catch are used hereafter to refer to both variants.) It is also acceptable to have a goto and its target label wholly encompassed in a control structure.

        What is not acceptable is using a goto or case to suddenly jump into the body of a try, catch, Objective-C @finally, or loop structure. Tangling labels or switch blocks with other control structures results in code that is difficult, if not impossible to understand. More importantly, when it compiles (some of these constructs won’t compile under ISO-conformant compilers), it can lead to unexpected results. Therefore this usage should be strictly avoided.

        This C++ code sample, which is also applicable to Objective-C if try and catch are converted to @try and @catch, demonstrates jumping into a switch and into a try and catch :

        Noncompliant code example

        void f ( int32_t i )
        {
          if ( 10 == i )
          {
            goto Label_10; // Noncompliant; goto transfers control into try block
          }
        
          if ( 11 == i )
          {
            goto Label_11; // Noncompliant; goto transfers control into catch block
          }
        
          switch ( i )
          {
            case 1:
              try
              {
                Label_10:
                case 2:  // Noncompliant; switch transfers control into try block
                  // Action
                  break;
              }
              catch ( ... )
              {
                Label_11:
                case 3: // Noncompliant; switch transfers control into catch block
                  // Action
                  break;
              }
              break;
            default:
            {
              // Default Action
              break;
            }
          }
        }
        

        Compliant solution

        void f ( int32_t i )
        {
          switch ( i )
          {
            case 1:
            case 2:
              // Action
              break;
            case 3:
              // Action
              break;
            case 10:
        
            default:
            {
              // Default Action
              break;
            }
          }
        
          try
          {
            if ( 2 == i || 10 == i)
            {
              // Action
            }
          }
          catch ( ... )
          {
            if (3 == i || 11 == i)
            {
              // Action
            }
          }
        }
        
          Available In:
        • SonarQube IdeCatch issues on the fly,
          in your IDE
        • SonarQube CloudDetect issues in your GitHub, Azure DevOps Services, Bitbucket Cloud, GitLab repositories
        • SonarQube ServerAnalyze code in your
          on-premise CI
          Developer Edition
          Available Since
          9.1

        © 2008-2025 SonarSource SA. All rights reserved.

        Privacy Policy | Cookie Policy | Terms of Use