Page History

...

When a larger scope than necessary is used, code becomes less readable, harder to maintain, and more likely to reference unintended variables . (See see DCL01-C. Do not reuse variable names in subscopes).)

Noncompliant Code Example

In this noncompliant code example, the function counter() increments the global variable count and then returns immediately if this variable exceeds a maximum value.:

unsigned int count = 0;

void counter() {
  if (count++ > MAX_COUNT) return;
  /* ... */

}

...

The counter variable i is declared outside of the for loop, which goes against this recommendation because it is not declared in the block in which it is used. If this snippet this code were reused with another index variable j, but there was a previously declared variable i, the loop could iterate over the wrong variable.

public void doStuff(...){
  size_t i = 0;

  for (i=0; i < 10; i++){
    /* Perform operations */
  }
}

Compliant Solution

Complying with this recommendation requires that you declare variables where they are used, which improves readability and reusability. In this example, you would declare the loop's index variable i within the initialization of the for loop. This requirement was recently relaxed in the C standardStandard.

public void doStuff(...){
  for (size_t i=0; i < 10; i++) {
  /* Perform operations */
}

Noncompliant Code Example (Function Declaration)

In this noncompliant code example, the function f() is called only from within the function g(), which is defined in the same compilation unit. By default, function declarations are extern, meaning that these functions are placed in the global symbol table and are available from other compilation units.

int f(int i) {
  /* Function definition */
}

int g(int i) {
  int j = f(i);
  /* ... */
} 

Compliant Solution

In this compliant solution, the function f() is declared with internal linkage. This practice limits the scope of the function declaration to the current compilation unit and prevents the function from being included in the external symbol table. It also limits cluttering in the global name space and prevents the function from being accidentally or intentionally invoked from another compilation unit. See DCL15-C. Declare file-scope objects or functions that do not need external linkage as static for more information.

static int f(int i) {
  /* Function definition */
}

int g(int i) {
  int j = f(i);
  /* ... */
} Perform operations */
  }
}

Risk Assessment

Failure to minimize scope could result in less reliable, readable, and reusable code.

Automated Detection

Related Guidelines

...

Related Vulnerabilities

Search for vulnerabilities resulting from the violation of this rule on the CERT website.

Related Guidelines

ISO/IEC 9899:2011 Appendix D.1.15, "Declaration in for-Loop Statement"

...

...

Image Modified Image Modified Image Modified