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: 41 rules found
unpredictable
    Impact
      Clean code attribute
        1. Globals should not depend on possibly not yet initialized variables

           Code Smell
        2. Coroutine should have co_return on each execution path or provide return_void

           Bug
        3. Thread local variables should not be used in coroutines

           Code Smell
        4. Arguments evaluation order should not be relied on

           Bug
        5. A single statement should not have more than one resource allocation

           Code Smell
        6. Functions that throw exceptions should not be used as hash functions

           Code Smell
        7. A call to "wait()" on a "std::condition_variable" should have a condition

           Bug
        8. Keywords shall not be used as macros identifiers

           Code Smell
        9. Incomplete types should not be deleted

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

           Code Smell
        11. Header guards should be followed by a matching "#define" macro

           Code Smell
        12. "memcmp" should only be called with pointers to trivially copyable types with no padding

           Bug
        13. Stack allocated memory and non-owned memory should not be freed

           Bug
        14. Destructors should not be called explicitly

           Code Smell
        15. Floating-point arithmetic should be used appropriately

           Bug
        16. There shall be no occurrence of "undefined" or "critical unspecified behaviour"

           Bug
        17. An "object pointer type" shall not be cast to an integral type other than "std::uintptr_t" or "std::intptr_t"

           Code Smell
        18. The pointers returned by the C++ Standard Library functions "localeconv", "getenv", "setlocale" or "strerror" must only be used as if they have pointer to const-qualified type

           Bug
        19. An object shall not be accessed outside of its lifetime

           Bug
        20. Reads and writes on the same file stream shall be separated by a positioning operation

           Bug
        21. A pointer to an incomplete "class" type shall not be deleted

           Bug
        22. An object shall not be used while in a "potentially moved-from state"

           Code Smell
        23. A comparison of a "potentially virtual" pointer to member function shall only be with "nullptr"

           Bug
        24. Local variables shall not have static storage duration

           Code Smell
        25. The pointer returned by the C++ Standard Library functions "asctime", "ctime", "gmtime", "localtime", "localeconv", "getenv", "setlocale" or "strerror" must not be used following a subsequent call to the same function

           Bug
        26. The library functions "atof", "atoi", "atol" and "atoll" from "<cstdlib>" shall not be used

           Bug
        27. The "defined" preprocessor operator shall be used appropriately

           Bug
        28. There should be at least one exception handler to catch all otherwise unhandled exceptions

           Bug
        29. The value of an object must not be read before it has been set

           Bug
        30. The built-in unary "-" operator should not be applied to an expression of unsigned type

           Bug
        31. The built-in relational operators ">", ">=", "<" and "<=" shall not be applied to objects of pointer type, except where they point to elements of the same array

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

           Bug
        33. Pointer arithmetic shall not form an invalid pointer

           Bug
        34. Operations on a memory location shall be sequenced appropriately

           Bug
        35. The numeric value of an "unscoped enumeration" with no fixed "underlying type" shall not be used

           Code Smell
        36. A function or object with external linkage should be "introduced" in a "header file"

           Code Smell
        37. The source code used to implement an "entity" shall appear only once

           Bug
        38. The "one-definition rule" shall not be violated

           Bug
        39. All "declarations" of a variable or function shall have the same type

           Bug
        40. A line whose first token is "#" shall be a valid preprocessing directive

           Bug
        41. An object or subobject must not be copied to an overlapping object

           Bug

        Globals should not depend on possibly not yet initialized variables

        intentionality - logical
        maintainability
        reliability
        Code Smell
        • unpredictable

        This rule raises an issue if the initializer of a global variable a, refers to another global variable b, when there is no guarantee that b is initialized before a.

        Why is this an issue?

        How can I fix it?

        More Info

        C++ allows global variables to have arbitrally complex initializers.

        Such variables are zero-initialized statically, and further initialization is performed at runtime, in the process referred to as dynamic initialization.

        The dynamic initialization of such variables is performed in the order of their definitions within a single source file. However, the order of initialization of variables defined in different source files is not specified. Therefore, when a global variable refers to a variable defined in a different source file, the behavior of the program is not guaranteed:

        extern int global1;
        // Defined in different source file as:
        // int global1 = 20;
        
        int const global2 = global1 + 10; // Noncompliant
        

        Depending on the order of initialization of global1 and global2, global2 may be set to:

        • 30 if global1 is initialized before global2
        • 10 if global2 is initialized before global1, as global1 will only be zero-initialized to 0.

        This rule raises an issue if the initializer of a global variable a, directly refers to another global variable b, when there is no guarantee that b is initialized before a.

        This is a subset of the issues related to initialization of global variables, commonly referred to as a static initialization order fiasco.

        What is the potential impact?

        The behavior of any program that is prone to static initialization order fiasco issues, is very fragile. The order of initialization may change at any time, for instance, when performing unrelated changes to the source code, or even differ between the builds of the same source. Such issues are very hard to reproduce and debug.

        Furthermore, the impact is not only limited to observing fluctuating values of global variables, as presented in the previous example.

        For class types, invoking any member functions of an object that is only zero-initialized and for which the constructor has not run, may cause undefined behavior and crash the program:

        extern std::string const string1;
        std::string const string2 = string1; // Noncompliant: undefined behavior
        

        Even if, for a particular implementation, zero-initialized objects are equivalent to default-initialized objects, the program may still encounter undefined behavior:

        extern std::vector<std::string> const list;
        // Defined in different source file as:
        // std::vector<std::string> const list{"entry1", "entry2"};
        
        std::string const firstElem = list[0]; // Noncompliant: undefined behavior
        

        Static class data members also use dynamic initialization

        Static data members of classes are dynamically initialized if their initializers are not constant expressions. Therefore, this rule will consider static data members of classes as if they were global variables.

        // Clazz.hpp
        extern int global;
        class Clazz {
          static int staticMem;
        };
        
        // Clazz.cpp
        #include "Clazz.hpp"
        
        int Clazz::staticMem = global + 10; // Noncompliant: "global" may not yet be fully initialized
        
        // Source.cpp
        #include "Clazz.hpp"
        
        int global = 10;
        int otherGlobal = Clazz::staticMem; // Noncompliant: "Clazz::staticMem" may not yet be fully initialized
        

        Initialization of instantiated variable is unordered

        If a global variable instantiated from a template requires a dynamic initialization (because its initialization is non-constant), the ordering of this initialization is unspecified. It may be performed before the initialization of a global variable declared before it. As a consequence, such variables should not refer to any dynamically initialized global variable in their initializers, or be referred to from the initializer of any global variable with dynamic initialization.

        An instantiated variable may be produced directly from the instantiation of a variable template (introduced in C++14):

        int global1 = runtimeFunc();
        
        template<typename T>
        T varTempl = global1; // Noncompliant: "global1" may be initialized later
        
        int global2 = varTempl<long>; // Noncompliant: "varTempl" may be initialized later
        

        Similarly, static data members of class template instantiation behave as instantiated global variables:

        int global1 = runtimeFunc();
        
        template<typename T>
        struct Clazz {
          static T staticMem;
        };
        
        template<typename T>
        T Clazz<T>::staticMem = global1; // Noncompliant: "global1" may be initialized later
        
        int global2 = Clazz<long>::staticMem; // Noncompliant: "Clazz<long>::staticMem" may be initialized later
        
          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
          10.8

        © 2025 SonarSource Sàrl. All rights reserved.

        Privacy Policy | Cookie Policy | Terms of Use