ABV.UNICODE.SELF_MAP
Buffer overflow-array index out of bounds in failed mapping function
ABV.UNICODE.SELF_MAP checks for buffer overrun conditions caused in failed MultiByteToWideChar and WideCharToMultiByte mapping functions. Typically, the checker detects an error condition when the source and destination pointers of the mapping function are the same.
For more information on the MultiByteToWideChar and WideCharToMultiByte mapping functions, see the MSDN website.
Vulnerability and risk
Using these mapping functions incorrectly can compromise the security of an application by causing a buffer overflow. To avoid this potential condition, it's important to specify a buffer size that is appropriate for the data type the buffer receives.
Vulnerable code example
int
foo(void *src, int size) {
WideCharToMultiByte(CP_ACP, 0, (wchar_t)src, -1, src, size, 0, 0);
MultiByteToWideChar(CP_ACP, 0, src, -1, src, 30);
}
Klocwork produces a message for both lines 3 and 4 in this example, indicating that the source and destination pointers must not be the same. The checker has found failed WideCharToMultiByte and MultiByteToWideChar mapping functions that have caused a buffer overrun condition.
Related checkers
External guidance
- CERT ARR00-C: Understand how arrays work
- CERT ARR30-C: Do not form or use out-of-bounds pointers or array subscripts
- CERT ENV01-C: Do not make assumptions about the size of an environment variable
- CWE-119: Improper Restriction of Operations within the Bounds of a Memory Buffer
- CWE-124: Buffer Underwrite ('Buffer Underflow')
- CWE-125: Out-of-bounds Read
- CWE-176: Improper Handling of Unicode Encoding
- CWE-787: Out-of-bounds Write
- CWE-806: Buffer Access Using Size of Source Buffer
- Security considerations: International features
- STIG-ID:APP3590.1 Application is vulnerable to buffer overflows
Security training
Application security training materials provided by Secure Code Warrior.
Extension
This checker can be extended through the Klocwork knowledge base. See Tuning C/C++ analysis for more information.