ENV01-A. Do not make assumptions about the size of an environment variable

Do not make any assumptions about the size of environment variables. Calculate the length of the strings yourself, and dynamically allocate memory for your copies. There is nothing you can do to avoid the race conditions inherent here, but you can limit your exposure.

Non-Compliant Coding Example

This non-compliant code example copies into a buffer of fixed size. This can result in a buffer overflow.

char *temp;
char buff[16];

temp = getenv("TEST_ENV");

if(temp != NULL)
  strcpy(buff, temp);

Compliant Solution

Use strlen to calculate size and dynamically allocate space.

char *temp;
char *copy;

if ((temp = getenv("TEST_ENV")) != NULL) {
  copy = malloc(strlen(temp) + 1);

  if (copy != NULL) {
    strcpy(copy, temp);
  }
  else {
    /* handle error condition */
  }
}
else {
  return -1;
}

Risk Assessment

References

[[ISO/IEC 9899-1999:TC2]] Section 7.20.4, "Communication with the environment"
[[Open Group 04]] Chapter 8, "Environment Variables", strdup
[[Viega 03]] Section 3.6, "Using Environment Variables Securely"

http://it-reading.org/book/secure%20programming%20cookbook%20c%20and%20c++/content/0596003943_secureprgckbk-chp-3-sect-6.html