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
VB6

VB6 static code analysis

Unique rules to find Bugs, Security Hotspots, and Code Smells in your VB6 code

  • All rules 45
  • Bug1
  • Security Hotspot1
  • Code Smell43
 
Tags
    Impact
      Clean code attribute
        1. "GoTo" statements should not be used

           Code Smell
        2. "GoSub" statements should not be used

           Code Smell
        3. "Option Explicit" should be enabled

           Code Smell
        4. Track parsing failures

           Code Smell
        5. "Select Case" statements should not be nested

           Code Smell
        6. Variable data types should be declared explicitly

           Code Smell
        7. The bang ("!") operator should not be used

           Code Smell
        8. Multiple variables should not be declared on the same line

           Code Smell
        9. Error handlers should not declare their own error handlers

           Bug
        10. "Option Base" should not be used

           Code Smell
        11. Loop invariants should not be calculated inside the loop

           Code Smell
        12. "Asc[W]" should not be called on string constants

           Code Smell
        13. "Chr[W]$()" should not be used for certain characters

           Code Smell
        14. '$' should not be used in string variable names

           Code Smell
        15. String-specific functions should be used

           Code Smell
        16. Strings should not be set to empty string

           Code Smell
        17. Strings should not be compared with empty string

           Code Smell
        18. The "&" operator should be used to concatenate strings

           Code Smell
        19. Sub and function names should comply with a naming convention

           Code Smell
        20. Subs and functions should not be too complex

           Code Smell
        21. "Select Case" statements should not have too many "Case" clauses

           Code Smell
        22. Comments should not be located at the end of lines of code

           Code Smell
        23. Subs and functions should not have too many lines

           Code Smell
        24. Using hardcoded IP addresses is security-sensitive

           Security Hotspot
        25. "Select" statements should end with a "Case Else" clause

           Code Smell
        26. "Select Case" statements should have at least 3 "Case" clauses

           Code Smell
        27. Statements should be on separate lines

           Code Smell
        28. Methods should not be empty

           Code Smell
        29. "Case" clauses should not have too many lines

           Code Smell
        30. Constant names should comply with a naming convention

           Code Smell
        31. Track uses of "TODO" tags

           Code Smell
        32. Track uses of "FIXME" tags

           Code Smell
        33. Lines should not end with trailing whitespaces

           Code Smell
        34. Return of boolean expressions should not be wrapped into an "if-then-else" statement

           Code Smell
        35. Boolean literals should not be redundant

           Code Smell
        36. Source code should be indented consistently

           Code Smell
        37. Empty statements should be removed

           Code Smell
        38. Magic numbers should not be used

           Code Smell
        39. Nested blocks of code should not be left empty

           Code Smell
        40. Procedures should not have too many parameters

           Code Smell
        41. Unused private variables should be removed

           Code Smell
        42. Expressions should not be too complex

           Code Smell
        43. Tabulation characters should not be used

           Code Smell
        44. Files should not have too many lines of code

           Code Smell
        45. Lines should not be too long

           Code Smell

        Using hardcoded IP addresses is security-sensitive

        responsibility - trustworthy
        security
        Security Hotspot

          Hardcoding IP addresses is security-sensitive. It has led in the past to the following vulnerabilities:

          • CVE-2006-5901
          • CVE-2005-3725

          Today’s services have an ever-changing architecture due to their scaling and redundancy needs. It is a mistake to think that a service will always have the same IP address. When it does change, the hardcoded IP will have to be modified too. This will have an impact on the product development, delivery, and deployment:

          • The developers will have to do a rapid fix every time this happens, instead of having an operation team change a configuration file.
          • It misleads to use the same address in every environment (dev, sys, qa, prod).

          Last but not least it has an effect on application security. Attackers might be able to decompile the code and thereby discover a potentially sensitive address. They can perform a Denial of Service attack on the service, try to get access to the system, or try to spoof the IP address to bypass security checks. Such attacks can always be possible, but in the case of a hardcoded IP address solving the issue will take more time, which will increase an attack’s impact.

          Ask Yourself Whether

          The disclosed IP address is sensitive, e.g.:

          • Can give information to an attacker about the network topology.
          • It’s a personal (assigned to an identifiable person) IP address.

          There is a risk if you answered yes to any of these questions.

          Recommended Secure Coding Practices

          Don’t hard-code the IP address in the source code, instead make it configurable with environment variables, configuration files, or a similar approach. Alternatively, if confidentially is not required a domain name can be used since it allows to change the destination quickly without having to rebuild the software.

          Sensitive Code Example

          Private Sub cmdConnect_Click()
              sockMain.RemoteHost = "192.168.12.42"
              sockMain.RemotePort = "80"
              sockMain.Connect
          End Sub
          

          Compliant Solution

          Private Sub cmdConnect_Click()
              sockMain.RemoteHost = txtHost.Text
              sockMain.RemotePort = txtPort.Text
              sockMain.Connect
          End Sub
          

          Exceptions

          No issue is reported for the following cases because they are not considered sensitive:

          • Loopback addresses 127.0.0.0/8 in CIDR notation (from 127.0.0.0 to 127.255.255.255)
          • Broadcast address 255.255.255.255
          • Non-routable address 0.0.0.0
          • Strings of the form 2.5.<number>.<number> as they often match Object Identifiers (OID)
          • Addresses in the ranges 192.0.2.0/24, 198.51.100.0/24, 203.0.113.0/24, reserved for documentation purposes by RFC 5737
          • Addresses in the range 2001:db8::/32, reserved for documentation purposes by RFC 3849

          See

          • OWASP - Top 10 2021 Category A1 - Broken Access Control
          • OWASP - Top 10 2017 Category A3 - Sensitive Data Exposure
            Available In:
          • SonarQube ServerAnalyze code in your
            on-premise CI
            Enterprise
            Edition
            Available Since
            9.1

          © 2008-2025 SonarSource SA. All rights reserved.

          Privacy Policy | Cookie Policy | Terms of Use