CWE-170 不恰当的空终结符

Improper Null Termination

结构: Simple

Abstraction: Base

状态: Incomplete

被利用可能性: Medium

基本描述

The software does not terminate or incorrectly terminates a string or array with a null character or equivalent terminator.

扩展描述

Null termination errors frequently occur in two different ways. An off-by-one error could cause a null to be written out of bounds, leading to an overflow. Or, a program could use a strncpy() function call incorrectly, which prevents a null terminator from being added at all. Other scenarios are possible.

相关缺陷

  • cwe_Nature: ChildOf cwe_CWE_ID: 707 cwe_View_ID: 1000 cwe_Ordinal: Primary

  • cwe_Nature: ChildOf cwe_CWE_ID: 138 cwe_View_ID: 699 cwe_Ordinal: Primary

  • cwe_Nature: CanPrecede cwe_CWE_ID: 120 cwe_View_ID: 1000

  • cwe_Nature: CanPrecede cwe_CWE_ID: 126 cwe_View_ID: 1000

  • cwe_Nature: CanAlsoBe cwe_CWE_ID: 147 cwe_View_ID: 1000

  • cwe_Nature: PeerOf cwe_CWE_ID: 464 cwe_View_ID: 1000

  • cwe_Nature: PeerOf cwe_CWE_ID: 463 cwe_View_ID: 1000

  • cwe_Nature: ChildOf cwe_CWE_ID: 20 cwe_View_ID: 700 cwe_Ordinal: Primary

适用平台

Language: [{'cwe_Name': 'C', 'cwe_Prevalence': 'Undetermined'}, {'cwe_Name': 'C++', 'cwe_Prevalence': 'Undetermined'}]

常见的影响

范围 影响 注释
['Confidentiality', 'Integrity', 'Availability'] ['Read Memory', 'Execute Unauthorized Code or Commands'] The case of an omitted null character is the most dangerous of the possible issues. This will almost certainly result in information disclosure, and possibly a buffer overflow condition, which may be exploited to execute arbitrary code.
['Confidentiality', 'Integrity', 'Availability'] ['DoS: Crash, Exit, or Restart', 'Read Memory', 'DoS: Resource Consumption (CPU)', 'DoS: Resource Consumption (Memory)'] If a null character is omitted from a string, then most string-copying functions will read data until they locate a null character, even outside of the intended boundaries of the string. This could: cause a crash due to a segmentation fault cause sensitive adjacent memory to be copied and sent to an outsider trigger a buffer overflow when the copy is being written to a fixed-size buffer.
['Integrity', 'Availability'] ['Modify Memory', 'DoS: Crash, Exit, or Restart'] Misplaced null characters may result in any number of security problems. The biggest issue is a subset of buffer overflow, and write-what-where conditions, where data corruption occurs from the writing of a null character over valid data, or even instructions. A randomly placed null character may put the system into an undefined state, and therefore make it prone to crashing. A misplaced null character may corrupt other data in memory.
['Integrity', 'Confidentiality', 'Availability', 'Access Control', 'Other'] ['Alter Execution Logic', 'Execute Unauthorized Code or Commands'] Should the null character corrupt the process flow, or affect a flag controlling access, it may lead to logical errors which allow for the execution of arbitrary code.

可能的缓解方案

Requirements

策略:

Use a language that is not susceptible to these issues. However, be careful of null byte interaction errors (CWE-626) with lower-level constructs that may be written in a language that is susceptible.

Implementation

策略:

Ensure that all string functions used are understood fully as to how they append null characters. Also, be wary of off-by-one errors when appending nulls to the end of strings.

Implementation

策略:

If performance constraints permit, special code can be added that validates null-termination of string buffers, this is a rather naive and error-prone solution.

Implementation

策略:

Switch to bounded string manipulation functions. Inspect buffer lengths involved in the buffer overrun trace reported with the defect.

Implementation

策略:

Add code that fills buffers with nulls (however, the length of buffers still needs to be inspected, to ensure that the non null-terminated string is not written at the physical end of the buffer).

示例代码

The following code reads from cfgfile and copies the input into inputbuf using strcpy(). The code mistakenly assumes that inputbuf will always contain a NULL terminator.

bad C

#define MAXLEN 1024
...
char *pathbuf[MAXLEN];
...
read(cfgfile,inputbuf,MAXLEN); //does not null terminate
strcpy(pathbuf,inputbuf); //requires null terminated input
...

The code above will behave correctly if the data read from cfgfile is null terminated on disk as expected. But if an attacker is able to modify this input so that it does not contain the expected NULL character, the call to strcpy() will continue copying from memory until it encounters an arbitrary NULL character. This will likely overflow the destination buffer and, if the attacker can control the contents of memory immediately following inputbuf, can leave the application susceptible to a buffer overflow attack.

In the following code, readlink() expands the name of a symbolic link stored in pathname and puts the absolute path into buf. The length of the resulting value is then calculated using strlen().

bad C

char buf[MAXPATH];
...
readlink(pathname, buf, MAXPATH);
int length = strlen(buf);
...

The code above will not always behave correctly as readlink() does not append a NULL byte to buf. Readlink() will stop copying characters once the maximum size of buf has been reached to avoid overflowing the buffer, this will leave the value buf not NULL terminated. In this situation, strlen() will continue traversing memory until it encounters an arbitrary NULL character further on down the stack, resulting in a length value that is much larger than the size of string. Readlink() does return the number of bytes copied, but when this return value is the same as stated buf size (in this case MAXPATH), it is impossible to know whether the pathname is precisely that many bytes long, or whether readlink() has truncated the name to avoid overrunning the buffer. In testing, vulnerabilities like this one might not be caught because the unused contents of buf and the memory immediately following it may be NULL, thereby causing strlen() to appear as if it is behaving correctly.

While the following example is not exploitable, it provides a good example of how nulls can be omitted or misplaced, even when "safe" functions are used:

bad C

#include <stdio.h>
#include <string.h>

int main() {

char longString[] = "String signifying nothing";
char shortString[16];

strncpy(shortString, longString, 16);
printf("The last character in shortString is: %c (%1$x)\n", shortString[15]);
return (0);
}

The above code gives the following output: "The last character in shortString is: n (6e)". So, the shortString array does not end in a NULL character, even though the "safe" string function strncpy() was used. The reason is that strncpy() does not impliciitly add a NULL character at the end of the string when the source is equal in length or longer than the provided size.

分析过的案例

标识 说明 链接
CVE-2000-0312 Attacker does not null-terminate argv[] when invoking another program. https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2000-0312
CVE-2003-0777 Interrupted step causes resultant lack of null termination. https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2003-0777
CVE-2004-1072 Fault causes resultant lack of null termination, leading to buffer expansion. https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2004-1072
CVE-2001-1389 Multiple vulnerabilities related to improper null termination. https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2001-1389
CVE-2003-0143 Product does not null terminate a message buffer after snprintf-like call, leading to overflow. https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2003-0143
CVE-2009-2523 Chain: product does not handle when an input string is not NULL terminated (CWE-170), leading to buffer over-read (CWE-125) or heap-based buffer overflow (CWE-122). https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2009-2523

Notes

Relationship Factors: this is usually resultant from other weaknesses such as off-by-one errors, but it can be primary to boundary condition violations such as buffer overflows. In buffer overflows, it can act as an expander for assumed-immutable data. Relationship Overlaps missing input terminator. Applicable Platform

Maintenance As currently described, this entry is more like a category than a weakness.

分类映射

映射的分类名 ImNode ID Fit Mapped Node Name
PLOVER Improper Null Termination
7 Pernicious Kingdoms String Termination Error
CLASP Miscalculated null termination
OWASP Top Ten 2004 A9 CWE More Specific Denial of Service
CERT C Secure Coding POS30-C CWE More Abstract Use the readlink() function properly
CERT C Secure Coding STR03-C Do not inadvertently truncate a null-terminated byte string
CERT C Secure Coding STR32-C Exact Do not pass a non-null-terminated character sequence to a library function that expects a string
Software Fault Patterns SFP11 Improper Null Termination