The C Standard identifies the following distinct situations in which undefined behavior (UB) can arise as a result of invalid pointer operations:
Addition or subtraction of a pointer into, or just beyond, an array object and an integer type produces a result that does not point into, or just beyond, the same array object.
Addition or subtraction of a pointer into, or just beyond, an array object and an integer type produces a result that points just beyond the array object and is used as the operand of a unary
An array subscript is out of range, even if an object is apparently accessible with the given subscript, for example, in the lvalue expression
An attempt is made to access, or generate a pointer to just past, a flexible array member of a structure when the referenced object provides no elements for that array.
Noncompliant Code Example (Forming Out-of-Bounds Pointer)
In this noncompliant code example, the function
f() attempts to validate the
index before using it as an offset to the statically allocated
table of integers. However, the function fails to reject negative
index values. When
index is less than zero, the behavior of the addition expression in the return statement of the function is undefined behavior 46. On some implementations, the addition alone can trigger a hardware trap. On other implementations, the addition may produce a result that when dereferenced triggers a hardware trap. Other implementations still may produce a dereferenceable pointer that points to an object distinct from
table. Using such a pointer to access the object may lead to information exposure or cause the wrong object to be modified.
One compliant solution is to detect and reject invalid values of
index if using them in pointer arithmetic would result in an invalid pointer:
Another slightly simpler and potentially more efficient compliant solution is to use an unsigned type to avoid having to check for negative values while still rejecting out-of-bounds positive values of
Noncompliant Code Example (Dereferencing Past-the-End Pointer)
This noncompliant code example shows the flawed logic in the Windows Distributed Component Object Model (DCOM) Remote Procedure Call (RPC) interface that was exploited by the W32.Blaster.Worm. The error is that the
while loop in the
GetMachineName() function (used to extract the host name from a longer string) is not sufficiently bounded. When the character array pointed to by
pwszTemp does not contain the backslash character among the first
MAX_COMPUTERNAME_LENGTH_FQDN + 1 elements, the final valid iteration of the loop will dereference past the end pointer, resulting in exploitable . In this case, the actual exploit allowed the attacker to inject executable code into a running program. Economic damage from the Blaster worm has been estimated to be at least $525 million [Pethia 2003].
For a discussion of this programming error in the Common Weakness Enumeration database, see CWE-119, "Improper Restriction of Operations within the Bounds of a Memory Buffer," and CWE-121, "Stack-based Buffer Overflow" [MITRE 2013].
In this compliant solution, the
while loop in the
GetMachineName() function is bounded so that the loop terminates when a backslash character is found, the null-termination character (
L'\0') is discovered, or the end of the buffer is reached. This code does not result in a buffer overflow even if no backslash character is found in
This compliant solution is for illustrative purposes and is not necessarily the solution implemented by Microsoft. This particular solution may not be correct because there is no guarantee that a backslash is found.
Noncompliant Code Example (Using Past-the-End Index)
Similar to the dereferencing-past-the-end-pointer error, the function
insert_in_table() in this noncompliant code example uses an otherwise valid index to attempt to store a value in an element just past the end of an array.
First, the function incorrectly validates the index
pos against the size of the buffer. When
pos is initially equal to
size, the function attempts to store
value in a memory location just past the end of the buffer.
Second, when the index is greater than
size, the function modifies
size before growing the size of the buffer. If the call to
realloc() fails to increase the size of the buffer, the next call to the function with a value of
pos equal to or greater than the original value of
size will again attempt to store
value in a memory location just past the end of the buffer or beyond.
Third, the function violates INT30-C. Ensure that unsigned integer operations do not wrap, which could lead to wrapping when 1 is added to
pos or when
size is multiplied by the size of
This compliant solution correctly validates the index
pos by using the
<= relational operator, ensures the multiplication will not overflow, and avoids modifying
size until it has verified that the call to
realloc() was successful:
Noncompliant Code Example (Apparently Accessible Out-of-Range Index)
This noncompliant code example declares
matrix to consist of 7 rows and 5 columns in row-major order. The function
init_matrix iterates over all 35 elements in an attempt to initialize each to the value given by the function argument
x. However, because multidimensional arrays are declared in C in row-major order, the function iterates over the elements in column-major order, and when the value of
j reaches the value
COLS during the first iteration of the outer loop, the function attempts to access element
matrix. Because the type of
j subscript is out of range, and the access has undefined behavior .
This compliant solution avoids using out-of-range indices by initializing
matrix elements in the same row-major order as multidimensional objects are declared in C:
Noncompliant Code Example (Pointer Past Flexible Array Member)
In this noncompliant code example, the function
find() attempts to iterate over the elements of the flexible array member
buf, starting with the second element. However, because function
g() does not allocate any storage for the member, the expression
find() attempts to form a pointer just past the end of
buf when there are no elements. This attempt is undefined behavior . (See MSC21-C. Use robust loop termination conditions for more information.)
This compliant solution avoids incrementing the pointer unless a value past the pointer's current value is known to exist:
Noncompliant Code Example (Null Pointer Arithmetic)
This noncompliant code example is similar to an Adobe Flash Player vulnerability that was first exploited in 2008. This code allocates a block of memory and initializes it with some data. The data does not belong at the beginning of the block, which is left uninitialized. Instead, it is placed
offset bytes within the block. The function ensures that the data fits within the allocated block.
This function fails to check if the allocation succeeds, which is a violation of ERR33-C. Detect and handle standard library errors. If the allocation fails, then
malloc() returns a null pointer. The null pointer is added to
offset and passed as the destination argument to
memcpy(). Because a null pointer does not point to a valid object, the result of the pointer arithmetic is undefined behavior 46.
An attacker who can supply the arguments to this function can exploit it to execute arbitrary code. This can be accomplished by providing an overly large value for
block_size, which causes
malloc() to fail and return a null pointer. The
offset argument will then serve as the destination address to the call to
memcpy(). The attacker can specify the
data_size arguments to provide the address and length of the address, respectively, that the attacker wishes to write into the memory referenced by
offset. The overall result is that the call to
memcpy() can be exploited by an attacker to overwrite an arbitrary memory location with an attacker-supplied address, typically resulting in arbitrary code execution.
Compliant Solution (Null Pointer Arithmetic)
This compliant solution ensures that the call to
Writing to out-of-range pointers or array subscripts can result in a buffer overflow and the execution of arbitrary code with the permissions of the vulnerable process. Reading from out-of-range pointers or array subscripts can result in unintended information disclosure.
Tainted buffer access
Pointer before beginning of object
A collection of warning classes that report uses of library functions prone to internal buffer overflows.
Could be configured to catch violations of this rule. The way to catch the noncompliant code example is to first hunt for example code that follows this pattern:
for (LPWSTR pwszTemp = pwszPath + 2; *pwszTemp != L'\\';
In particular, the iteration variable is a pointer, it gets incremented, and the loop condition does not set an upper bound on the pointer. Once this case is handled, ROSE can handle cases like the real noncompliant code example, which is effectively the same semantics, just different syntax
Can detect the access of memory past the end of a memory buffer/array
Can detect when the loop bound may become negative
Can detect the out-of-bound read/write to array allocated statically or dynamically
Can detect buffer overflows
|LDRA tool suite||9.7.1|
45 D, 47 S, 476 S, 489 S, 64 X, 66 X, 68 X, 69 X, 70 X, 71 X, 79 X
|Parasoft C/C++test||9.5||BD-PB-ARRAY||Partially implemented|
|Polyspace Bug Finder||R2016a|
Array index outside bounds during array access
Array index from unsecure source possibly outside array bounds
Pointer dereferenced outside its bounds
Offset is from an unsecure source and dereference may be out of bounds
Pointer from an unsecure source may be NULL or point to unknown memory
2840, 2841, 2842, 2843, 2844, 2930, 2931, 2932, 2933, 2934, 2950,
2820, 2821, 2822, 2823, 2824, 2840, 2841, 2842, 2843, 2844, 2930,
|Cppcheck||1.66||arrayIndexOutOfBounds, outOfBounds, negativeIndex, arrayIndexThenCheck, arrayIndexOutOfBoundsCond, possibleBufferAccessOutOfBounds|
Context sensitive analysis of array index, pointers, etc.
Array index out of bounds
Buffer overflow when calling various functions memset,strcpy,..
Warns about condition (a[i] == 0 && i < unknown_value) and recommends that (i < unknown_value && a[i] == 0) is used instead
Detects unsafe code when array is accessed before/after it is tested if the array index is out of bounds
CVE-2008-1517 results from a violation of this rule. Before Mac OSX version 10.5.7, the XNU kernel accessed an array at an unverified user-input index, allowing an attacker to execute arbitrary code by passing an index greater than the length of the array and therefore accessing outside memory [xorl 2009].
|ISO/IEC TR 24772:2013||Arithmetic Wrap-Around Error [FIF]|
Unchecked Array Indexing [XYZ]
|ISO/IEC TS 17961||Forming or using out-of-bounds pointers or array subscripts [invptr]|
CWE-119, Improper Restriction of Operations within the Bounds of a Memory Buffer
|MISRA C:2012||Rule 18.1 (required)|
|[Seacord 2013b]||Chapter 1, "Running with Scissors"|
|[Viega 2005]||Section 5.2.13, "Unchecked Array Indexing"|
|[xorl 2009 ]||"CVE-2008-1517: Apple Mac OS X (XNU) Missing Array Index Validation"|