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: 30 rules found
misra-advisory
    Impact
      Clean code attribute
        1. Floating-point arithmetic should be used appropriately

           Bug
        2. The "goto" statement should not be used

           Code Smell
        3. The built-in unary "+" operator should not be used

           Code Smell
        4. Functions with "limited visibility" should be "used" at least once

           Code Smell
        5. All variables should be initialized

           Code Smell
        6. "Dynamic memory" should not be used

           Code Smell
        7. The "#pragma" directive and the "_Pragma" operator should not be used

           Code Smell
        8. The "#" and "##" preprocessor operators should not be used

           Code Smell
        9. Program-terminating functions should not be used

           Code Smell
        10. Bit-fields should not be declared

           Code Smell
        11. A "declaration" should not declare more than one variable or member variable

           Code Smell
        12. The target type of a pointer or "lvalue" reference parameter should be const-qualified appropriately

           Code Smell
        13. The result of an assignment operator should not be "used"

           Code Smell
        14. An unsigned arithmetic operation with constant operands should not wrap

           Bug
        15. The comma operator should not be used

           Code Smell
        16. The right-hand operand of a logical "&&" or "||" operator should not contain "persistent side effects"

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

           Bug
        18. A cast should not convert a pointer type to an integral type

           Code Smell
        19. The "declaration" of an object should contain no more than two levels of pointer indirection

           Code Smell
        20. Parentheses should be used to make the meaning of an expression appropriately explicit

           Code Smell
        21. The names of the "standard signed integer types" and "standard unsigned integer types" should not be used

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

           Code Smell
        23. "#undef" should only be used for macros defined previously in the same file

           Code Smell
        24. "#include" directives should only be preceded by preprocessor directives or comments

           Code Smell
        25. Sections of code should not be "commented out"

           Code Smell
        26. "Trigraph-like sequences" should not be used

           Code Smell
        27. A value should not be "unnecessarily written" to a local object

           Code Smell
        28. Types with "limited visibility" should be "used" at least once

           Code Smell
        29. Variables with "limited visibility" should be "used" at least once

           Code Smell
        30. Controlling expressions should not be invariant

           Bug

        Parentheses should be used to make the meaning of an expression appropriately explicit

        intentionality - clear
        maintainability
        reliability
        Code Smell
        • confusing
        • misra-c++2023
        • misra-advisory

        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 8.0.1 - Parentheses should be used to make the meaning of an expression appropriately explicit

        Category: Advisory

        Analysis: Decidable,Single Translation Unit

        Amplification

        The following table is used in the definition of this rule.

        Description

        Operator or Operand

        Ranking

        Other

        Any operator or operand not listed below

        14 (high)

        Multiplicative

        * / %

        13

        Additive

        + -

        12

        Bitwise shift

        << >>

        11

        Relational

        < > <= >=

        10

        Equality

        == !=

        9

        Bitwise AND

        &

        8

        Bitwise XOR

        ^

        7

        Bitwise OR

        |

        6

        Logical AND

        &&

        5

        Logical OR

        ||

        4

        Conditional

        ?:

        3

        Assignment

        = *= /= %= += -= <<= >>= &= ^= |=

        2

        Throw

        throw

        1

        Comma

        ,

        0 (low)

        The rankings used in this table are chosen to allow a concise description of the rule. They are not necessarily the same as those that might be encountered in the C++ Standard’s descriptions of operator precedence.

        Notes:

        • Operators having alternative token representations (see [lex.digraph]) have the same ranking as their primary form.
        • The additive row does not include unary plus and unary minus, which have rank 14.

        An expression is appropriately explicit when:

        • Its ranking is 0, 1, 2 or 14; or
        • Each operand:
        • Is parenthesized; or
        • Has a ranking of 14; or
        • Has ranking less than or equal to that of the expression.

        Additionally, the operand to the sizeof operator should be parenthesized.

        For the purposes of this rule, the ranking of an expression is the ranking of the element (operand or operator) at the root of the parse tree for that expression. For a sub-expression, its ranking is that of the element at the "root" of the sub-tree.

        For example, using the syntax and precedence rules from the C++ Standard, the parse tree for the non-compliant expression a << b + c can be represented as:

             <<
            /  \
           a    +
               / \
              b   c
        

        The element at the root of this parse tree is '<<', so the expression has ranking 11. The root of the sub-tree for b + c is +, which has ranking 12.

        Rationale

        The C++ language has a comparatively large number of operators and their relative precedences are not intuitive. This can lead less experienced developers to make mistakes. Using parentheses to make operator bindings explicit removes the possibility that the developer’s expectations are incorrect. It also makes the original developer’s intention clear to reviewers or maintainers of the code.

        It is recognized that overuse of parentheses can clutter the code and reduce its readability. However, too few parentheses can lead to unintuitive code. This rule tries to achieve a reasonable compromise.

        Note: this rule does not require the operands of a comma operator to be parenthesized, even though the result may not meet developer expectation. However, use of the comma operator is not compliant with M23_105: MISRA C++ 2023 Rule 8.19.1.

        x = a, b;   // Parsed as ( x = a ), b
        

        Example

        The following examples show expressions with a unary or postfix operator whose operands are either primary-expressions or expressions whose top-level operators have ranking 14:

        a[ i ]->n;               // Compliant - no need to write ( a[ i ] )->n
        *p++;                    // Compliant - no need to write *( p++ )
        sizeof x + y;            // Non-compliant - write either sizeof ( x ) + y
                                 // or sizeof ( x + y )
        

        The following examples show expressions containing operators of the same ranking:

        a + b - c + d;           // Compliant
        ( a + b ) - ( c + d );   // Compliant - produces a different result
        

        The following examples show a variety of mixed-operator expressions:

        x = f ( a + b, c );      // Compliant - no need to write f ( ( a + b ), c )
        
        x = a == b ? a : a - b;  // Non-compliant - operands of conditional operator
                                 // (ranking 3) are:
                                 //   == (ranking 9) needs parentheses
                                 //   a  (ranking 14) does not need parentheses
                                 //   -  (ranking 12) needs parentheses
        
        x = ( a == b ) ? a : ( a - b );   // Compliant version of previous example
        

        Note: the assignment operators in the previous two examples are compliant — the ranking of the assignment operator is less than 3, so its operands do not need parentheses.

        x = a << b + c;       // Non-compliant - operands of << operator
                              // (ranking 11) are:
                              //  a  (ranking 14) does not need parentheses
                              //  +  (ranking 12) needs parentheses
        
        a && b && c;          // Compliant - all operators are the same.
        a && b || c;          // Non-compliant - || (ranking 4) has operand && (ranking 5)
        a || b && c;          // Non-compliant - || (ranking 4) has operand && (ranking 5)
        a || b || c;          // Compliant - all operators are the same
        
        #if defined( A ) && defined( B ) || defined( C )  // 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