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: 35 rules found
symbolic-execution
    Impact
      Clean code attribute
        1. The address of an automatic object should not be assigned to another object that may persist after the first object has ceased to exist

           Bug
        2. Variables should be initialized before use

           Bug
        3. Variables should not be accessed outside of their scope

           Bug
        4. Account validity should be verified when authenticating users with PAM

           Vulnerability
        5. Changing directories improperly when using "chroot" is security-sensitive

           Security Hotspot
        6. POSIX functions should not be called with arguments that trigger buffer overflows

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

           Vulnerability
        8. "pthread_mutex_t" should be unlocked in the reverse order they were locked

           Bug
        9. Only valid arguments should be passed to UNIX/POSIX functions

           Code Smell
        10. "pthread_mutex_t" should be properly initialized and destroyed

           Bug
        11. "pthread_mutex_t" should not be locked when already locked, or unlocked when already unlocked

           Bug
        12. Only valid arguments should be passed to stream functions

           Code Smell
        13. Using publicly writable directories is security-sensitive

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

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

           Code Smell
        16. Return value of "setuid" family of functions should always be checked

           Code Smell
        17. Size of variable length arrays should be greater than zero

           Code Smell
        18. "mktemp" family of functions templates should have at least six trailing "X"s

           Code Smell
        19. Appropriate size arguments should be passed to "strncat" and "strlcpy"

           Code Smell
        20. Server certificates should be verified during SSL/TLS connections

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

           Vulnerability
        22. Integral operations should not overflow

           Bug
        23. Parameter values should be appropriate

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

           Bug
        25. Closed resources should not be accessed

           Bug
        26. Dynamically allocated memory should be released

           Bug
        27. Freed memory should not be used

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

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

           Bug
        30. Zero should not be a possible denominator

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

           Vulnerability
        32. "nonnull" parameters and return values of "returns_nonnull" functions should not be null

           Bug
        33. Null pointers should not be dereferenced

           Bug
        34. Resources should be closed

           Bug
        35. Unused assignments should be removed

           Code Smell

        Dynamically allocated memory should be released

        intentionality - complete
        reliability
        Bug
        • cwe
        • symbolic-execution
        • leak
        • denial-of-service
        • cert

        Memory allocated dynamically with calloc, malloc, realloc, or new should be released when it is not needed anymore. Failure to do so will result in a memory leak that could severely hinder application performance or abort it or the entire host machine.

        Why is this an issue?

        How can I fix it?

        More Info

        Memory is a limited resource shared between all the applications running on the same host machine.

        C and C++ do not automatically reclaim unused memory. The developer has to release the memory claimed for their application that is no longer needed. Unlike the stack that automatically allocates local variables on a function call and deallocates them on a function return, the heap offers no automatic memory management. The developer has to make sure to deallocate the memory they allocate dynamically on the heap.

        This rule raises an issue when memory is allocated dynamically and not freed within the same function.

        What is the potential impact?

        Neglecting to free the memory leads to a memory leak.

        The application that leaks memory will consume more and more of it over time, eventually claiming all the memory available on the host machine. When this happens and the system runs out of memory, it typically does one of the following:

        • The operating system (if any) terminates the application.
        • The operating system (if any) terminates some other application, and the problem reoccurs when the reclaimed memory gets used up by the leaking application.
        • The operating system (if any) starts offloading some of the memory pages to disk and slows down some memory accesses by orders of magnitude.
        • The entire system crashes as a whole and reboots automatically or hangs waiting for a manual reboot.

        Moreover, memory leaks can help an attacker to take over the system. An attacker could use a memory leak to fill the memory with malicious code. This facilitates remote code execution through another chained vulnerability.

        Even if the attacker cannot take over the system she can intentionally trigger the condition leading to a memory leak to make use of the issue above and cause denial-of-service (DoS) of the system.

        A memory leak can have a significant impact on the energy footprint of an application.

        • If an application demands more memory than necessary, the user will have to install more memory banks than necessary. Each memory bank consumes additional power.
        • As the application continues to reserve more and more memory, it places an increased load on the memory management subsystem. This increased load can lead to a larger computation demand, which in turn translates to higher power consumption by the CPU.

        Finally, memory leaks degrade the user experience. The user often experiences a system slowdown caused by the uncontrolled memory use of an application. Delayed response time, system freezes, and crashes degrade the user experience and discourage the further use of the application.

        Exceptions

        If a function returns a pointer to the caller or stores it in an external structure, this pointer is said to escape (it is now accessible outside of function, and no longer local to it). This includes storing the pointer in a static or global variable, passing it to a function that can potentially do that, or returning the pointer directly or as part of an aggregate object.

        The memory pointed to by an escaping pointer might be used somewhere else in the program. For that reason, the analyzer cannot proclaim a leak for an escaping pointer by only looking at a function scope.

        While in some cases the leak might be detectable in the scope of a caller, in others, the analyzer would need to simulate the entire program to verify that the memory is not used anywhere, which is not feasible.

        For this technical reason, this rule often ignores escaping pointers.

          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