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 420
  • Vulnerability14
  • Bug111
  • Security Hotspot19
  • Code Smell276

  • Quick Fix 27
Filtered: 27 rules found
unpredictable
    Impact
      Clean code attribute
        1. Arguments evaluation order should not be relied on

           Bug
        2. Keywords shall not be used as macros identifiers

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

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

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

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

           Bug
        7. The "<stdlib.h>" functions "bsearch" and "qsort" should not be used

           Bug
        8. Floating-point arithmetic should be used appropriately

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

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

           Code Smell
        11. 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
        12. Reads and writes on the same file stream shall be separated by a positioning operation

           Bug
        13. Local variables shall not have static storage duration

           Code Smell
        14. 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
        15. The "defined" preprocessor operator shall be used appropriately

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

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

           Bug
        18. 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
        19. Subtraction between pointers shall only be applied to pointers that address elements of the same array

           Bug
        20. Pointer arithmetic shall not form an invalid pointer

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

           Bug
        22. A function or object with external linkage should be "introduced" in a "header file"

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

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

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

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

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

           Bug

        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

        intentionality - logical
        reliability
        Bug
        • unpredictable
        • misra-c++2023
        • misra-mandatory

        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 25.5.2 - 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

        [clocale.syn]
        [cstdlib.syn]
        [cstring.syn]
        [C11] / 7.11.1.1; Undefined 8
        [C11] / 7.11.2.1; Undefined 8
        [C11] / 7.22.4.6; Undefined 4
        [C11] / 7.24.6.2; Undefined 3

        Category: Mandatory

        Analysis: Decidable, Single Translation Unit

        Amplification

        The localeconv function returns a pointer of type struct lconv *. This pointer must be regarded as if it had type const struct lconv *.

        A struct lconv object includes pointers of type char * and the getenv, setlocale, and strerror functions each return a pointer of type char *. These pointers are used to access C-style strings (null-terminated arrays of type char). For the purposes of this rule, these pointers must be regarded as if they had type const char *.

        The addressed of these functions shall not be taken.

        Rationale

        The C++ Standard states that undefined behaviour occurs if a program modifies:

        • The structure pointed to by the value returned by localeconv;
        • The strings returned by getenv, setlocale or strerror.

        Note: the C++ Standard does not specify the behaviour that results if the strings referenced by the structure pointed to by the value returned by localeconv are modified. This rule prohibits any changes to these strings as they are considered to be undesirable.

        Treating the pointers returned by the various functions as if they were const-qualified allows an analysis tool to detect any attempt to modify an object through one of the pointers. Additionally, assigning the return values of the functions to const-qualified pointers will result in the compiler issuing a diagnostic if an attempt is made to modify an object.

        Note: if a modified version is required, a program should make and modify a copy of any value covered by this rule.

        Preventing the addresses of these functions from being taken allows compliance checks to be decidable.

        Example

        The following examples are non-compliant as the returned pointers are assigned to non const-qualified pointers. Whilst this will not be reported by a compiler (it is not ill-formed), an analysis tool will be able to report a violation.

        void f1()
        {
          char         * s1   = setlocale( LC_ALL, 0 );   // Non-compliant
          struct lconv * conv = localeconv();             // Non-compliant
        
          s1[ 1 ]             = 'A';                      // Undefined behaviour
          conv->decimal_point = "^";                      // Undefined behaviour
        }
        

        The following examples are compliant as the returned pointers are assigned to const-qualified pointers. Any attempt to modify an object through a pointer will be reported by a compiler or analysis tool as this is ill-formed.

        void f2()
        {
          char str[ 128 ];
        
          ( void ) strcpy( str,
                           setlocale( LC_ALL, 0 ) );      // Compliant - 2nd parameter to
                                                          // strcpy takes a const char *
          const struct lconv * conv = localeconv();       // Compliant
        
          conv->decimal_point = "^";                      // Ill-formed
        }
        

        The following example shows that whilst the use of a const-qualified pointer gives compile time protection of the value returned by localeconv, the same is not true for the strings it references. Modification of these strings can be detected by an analysis tool.

        void f3()
        {
          const struct lconv * conv = localeconv();       // Compliant
        
          conv->grouping[ 2 ] = 'x';                      // 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