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
  • ShellShell
  • 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 798
  • Vulnerability14
  • Bug173
  • Security Hotspot19
  • Code Smell592

  • Quick Fix 99
Filtered: 68 rules found
cwe
    Impact
      Clean code attribute
        1. Function-like macros should not be invoked without all of their arguments

           Bug
        2. Function exit paths should have appropriate return values

           Bug
        3. Non-empty statements should change control flow or have at least one side-effect

           Bug
        4. Bitwise operators should not be applied to signed operands

           Bug
        5. Limited dependence should be placed on operator precedence

           Code Smell
        6. Function pointers should not be converted to any other type

           Bug
        7. Variables should be initialized before use

           Bug
        8. Hard-coded secrets are security-sensitive

           Security Hotspot
        9. "sprintf" should not be used

           Security Hotspot
        10. Changing working directories without verifying the success is security-sensitive

           Security Hotspot
        11. Setting capabilities is security-sensitive

           Security Hotspot
        12. Accessing files should not introduce TOCTOU vulnerabilities

           Vulnerability
        13. Account validity should be verified when authenticating users with PAM

           Vulnerability
        14. Using "tmpnam", "tmpnam_s" or "tmpnam_r" is security-sensitive

           Security Hotspot
        15. Using "strncpy" or "wcsncpy" is security-sensitive

           Security Hotspot
        16. Using "strncat" or "wcsncat" is security-sensitive

           Security Hotspot
        17. Using "strcat" or "wcscat" is security-sensitive

           Security Hotspot
        18. Using "strlen" or "wcslen" is security-sensitive

           Security Hotspot
        19. Changing directories improperly when using "chroot" is security-sensitive

           Security Hotspot
        20. Using "strcpy" or "wcscpy" is security-sensitive

           Security Hotspot
        21. "memset" should not be used to delete sensitive data

           Vulnerability
        22. POSIX functions should not be called with arguments that trigger buffer overflows

           Vulnerability
        23. Relational and subtraction operators should not be used with pointers to different arrays

           Bug
        24. Cipher algorithms should be robust

           Vulnerability
        25. Encryption algorithms should be used with secure mode and padding scheme

           Vulnerability
        26. Server hostnames should be verified during SSL/TLS connections

           Vulnerability
        27. "pthread_mutex_t" should not be locked when already locked, or unlocked when already unlocked

           Bug
        28. Using publicly writable directories is security-sensitive

           Security Hotspot
        29. Using clear-text protocols is security-sensitive

           Security Hotspot
        30. Blocking functions should not be called inside critical sections

           Code Smell
        31. Dereferenced null pointers should not be bound to references

           Code Smell
        32. Expanding archive files without controlling resource consumption is security-sensitive

           Security Hotspot
        33. Server certificates should be verified during SSL/TLS connections

           Vulnerability
        34. Using weak hashing algorithms is security-sensitive

           Security Hotspot
        35. Cryptographic keys should be robust

           Vulnerability
        36. Weak SSL/TLS protocols should not be used

           Vulnerability
        37. Dynamically allocated memory should be released

           Bug
        38. Freed memory should not be used

           Bug
        39. Memory locations should not be released more than once

           Bug
        40. Memory access should be explicitly bounded to prevent buffer overflows

           Bug
        41. Zero should not be a possible denominator

           Bug
        42. "sizeof" should not be called on pointers

           Bug
        43. XML parsers should not be vulnerable to XXE attacks

           Vulnerability
        44. Multiline blocks should be enclosed in curly braces

           Code Smell
        45. "nonnull" parameters and return values of "returns_nonnull" functions should not be null

           Bug
        46. Setting loose POSIX file permissions is security-sensitive

           Security Hotspot
        47. Conditionally executed code should be reachable

           Bug
        48. Exceptions should not be ignored

           Code Smell
        49. Null pointers should not be dereferenced

           Bug
        50. Using pseudorandom number generators (PRNGs) is security-sensitive

           Security Hotspot
        51. Resources should be closed

           Bug
        52. Hard-coded passwords are security-sensitive

           Security Hotspot
        53. Code annotated as deprecated should not be used

           Code Smell
        54. Unused assignments should be removed

           Code Smell
        55. All code should be reachable

           Bug
        56. "switch" statements should have "default" clauses

           Code Smell
        57. Switch cases should end with an unconditional "break" statement

           Code Smell
        58. Generic exceptions should not be caught

           Code Smell
        59. Track uses of "TODO" tags

           Code Smell
        60. Track uses of "FIXME" tags

           Code Smell
        61. Assignments should not be made from within conditions

           Code Smell
        62. Generic exceptions should never be thrown

           Code Smell
        63. Insecure functions should not be used

           Vulnerability
        64. "scanf()" and "fscanf()" format strings should specify a field width for the "%s" string placeholder

           Vulnerability
        65. The value of an object must not be read before it has been set

           Bug
        66. Subtraction between pointers shall only be applied to pointers that address elements of the same array

           Bug
        67. A value should not be "unnecessarily written" to a local object

           Code Smell
        68. A function shall not contain "unreachable" statements

           Bug

        A function shall not contain "unreachable" statements

        intentionality - logical
        reliability
        Bug
        Quick FixIDE quick fixes available with SonarQube for IDE
        • unused
        • cwe
        • misra-c++2023
        • misra-required

        Why is this an issue?

        More Info

        This rule is part of MISRA C++:2023.

        Usage of this content is governed by Sonar’s terms and conditions. Redistribution is prohibited.

        Rule 0.0.1 - A function shall not contain unreachable statements

        [IEC 61508-7] / C.5.9
        [DO-178C] / 6.4.4.3.c
        [ISO 26262-6] / 9.4

        Category: Required

        Analysis: Decidable,Single Translation Unit

        Amplification

        A statement is unreachable if the block containing it is not reachable from the entry block of the Control Flow Graph (CFG) for the function.

        For the purpose of this rule:

        • Both operands of a reachable logical AND (&&) or logical OR (||) operator are considered reachable; and
        • All three operands of a reachable conditional operator (?:) are considered reachable; and
        • The blocks linked by the edges from a condition of a selection-statement or an iteration-statement are all considered reachable, except when the condition is a constant expression, in which case only the blocks linked by edges selected by the condition are considered reachable; and
        • A call to a function declared [[noreturn]] has no CFG out edge; and
        • If a try compound-statement of a (function-)try-block does not contain a reachable, potentially-throwing statement, then all catch-handlers are unreachable, otherwise all catch-handlers are considered reachable subject to the restriction that a catch-handler that appears after a more generic handler of the same try-block is not reachable.

        The rule does not apply to statements in the discarded branch of a constexpr if statement.

        Rationale

        Unreachable code often indicates a defect in the program, as, assuming that the program does not exhibit any undefined behaviour, unreachable code cannot be executed and cannot have any effect on the program’s outputs.

        In order to avoid crosstalk with M23_002: MISRA C++ 2023 Rule 0.0.2, the handling of logical and conditional operators in the conceptual CFG used by this rule differs from that in a traditional CFG.

        Example

        bool f0();
        
        int32_t f1( int32_t c, int32_t & res )
        {
          if ( false && f0() ) { }  // Compliant - statement is considered to be reachable
        
          return res;
        
          res = c;                  // Non-compliant - not reachable from entry block
        
          bool result;              // Non-compliant - not reachable from entry block
        }
        
        void f2( int32_t i )
        {
          while ( true )            // Constant condition - single edge into body of loop
          {
            if ( i != 0 )
            {
              break;                // Adds edge to statements following the loop body
            }
          }
        
          ++i;                      // Compliant - reachable via 'break'
        
          while ( true )            // Constant condition - single edge into body of loop
          {
            f();
          }
        
          ++i;                      // Non-compliant - not reachable from entry block
        }
        
        void f3( int32_t i )
        {
          goto LABEL;
          ++i;                      // Non-compliant - no edge to this block
        
        LABEL:
          ++i;                      // Compliant
        }
        
        class BaseException {};
        class DerivedException: public BaseException {};
        
        void f4()
        {
          try { /* ... */ }
          catch ( BaseException    & b ) { }
          catch ( DerivedException & d ) { }   // Non-compliant - will be caught above
        }
        
        void f5() noexcept;
        
        void f6()
        {
          try { f5(); }
          catch ( int32_t ) { }     // Non-compliant - f5 is not potentially-throwing
          catch ( ... )     { }     // Non-compliant - f5 is not potentially-throwing
        }
        
        void f7( int32_t i )
        {
          try
          {
            throw i;
            ++i;                    // Non-compliant - no edge to this block
          }
          catch ( int32_t ) { }     // Compliant - all catch-handlers are reachable
          catch ( int16_t ) { }     // Compliant - all catch-handlers are reachable
        
          ++i;                      // Compliant
        }
        
        void f8();
        
        int32_t f9( int32_t i )
        {
          try
          {
            f8();                   // Potentially-throwing
            return i * 2;           // Compliant
          }
          catch( int32_t ) { }      // Compliant - all catch-handlers are reachable
        
          return 0;                 // Compliant - even if f8 throws a type
        }                           //             other than int32_t
        
        [[noreturn]] void f10() noexcept;
        
        int32_t f11()
        {
          f10();                    // Does not return
          return 0;                 // Non-compliant
        }
        

        Copyright The MISRA Consortium Limited © 2023

          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

        © 2025 SonarSource Sàrl. All rights reserved.

        Privacy Policy | Cookie Policy | Terms of Use