Variadic functions can accept a variable number of arguments , but they are problematic. Variadic functions define an implicit contract between the function writer and the function user that allows the function to determine the number of arguments passed in any particular invocation. Failure to exercise care when invoking a variadic function to ensure that it knows when to stop processing arguments enforce this contract may result in undefined behavior. See undefined behavior 141 of Appendix J of the C Standard.
Argument Processing
In the following code example, the variadic function {{ Wiki Markup average()
}} is used to determine the average value of its passed integer arguments \[[Seacord 05c|AA. C References#Seacord 05c]\]. The function processes arguments until it finds one with a value of {{\-1}}. calculates the average value of the positive integer arguments passed to the function [Seacord 2013]. The function processes arguments until it encounters an argument with the value of va_eol
(-1
).
Code Block |
---|
enum { va_eol = -1 };
unsigned |
Code Block |
int average(int first, ...) { unsigned size_tint count = 0; unsigned int sum = 0; int i = first; va_list markerargs; va_start(markerargs, first); while (i != -1va_eol) { sum += i; count++; i = va_arg(markerargs, int); } va_end(markerargs); return(count ? (sum / count) : 0); } |
Note that va_start()
must be called to initialize the argument list and that va_end()
must always be called when finished with a variable argument list.
...
Noncompliant Code Example
In this non-compliant noncompliant code example, the average()
function above is called as follows:
Code Block | ||||
---|---|---|---|---|
| ||||
int avg = average(1, 4, 6, 4, 1);
|
The omission of the -1
va_eol
terminating value means that the function will continue to process values from the stack until it encounters a -1
va_eol
by coincidence or an error occurs.
Compliant Solution
The following call maintains This compliant solution enforces the contract by adding a -1 va_eol
as the last final argument.:
Code Block | ||||
---|---|---|---|---|
| ||||
int avg = average(1, 4, 6, 4, 1, -1va_eol); |
...
Noncompliant Code Example
Another common mistake is to use more format conversion specifiers than supplied arguments. This results in undefined behavior, which could end up pulling extra values off the stack and unintentionally exposing data. The following example illustrates a case of this, as shown in this noncompliant code example:
Code Block | ||||
---|---|---|---|---|
| ||||
const char *error_msg = "Resource not available to user.";
/* ... */
printf("Error (%s): %s", error_msg);
|
This code results in nonexistent arguments being processed by the function, potentially leaking information about the process.
Compliant Solution
The following code This compliant solution matches the number of format specifiers with the number of variable arguments.:
Code Block | ||||
---|---|---|---|---|
| ||||
const char *error_msg = "Resource not available to user.";
/* ... */
printf("Error: %s", error_msg);
|
Argument List Caveats
C99 C functions that accept the variadic primitive va_list
as an argument pose an additional risk. Calls to vfprintf()
, vfscanf()
, vprintf()
, vscanf()
, vsnprintf()
, vsprintf()
, and vsscanf()
use the va_arg()
macro, invalidating the parameterized va_list
. Consequently, once a va_list
is passed as an argument to any of these functions, it cannot be used again without a call to va_end()
followed by a call to va_start()
.
Risk Assessment
Incorrectly using a variadic function can result in abnormal program termination or unintended information disclosure.
Recommendation | Severity | Likelihood | Remediation Cost | Priority | Level |
---|---|---|---|---|---|
DCL10- |
2 (medium)
2 (probable)
2 (medium)
C | High | Probable | High | P6 | L2 |
Related Vulnerabilities
Search for vulnerabilities resulting from the violation of this rule on the CERT website.
References
Wiki Markup |
---|
\[[ISO/IEC 9899-1999|AA. C References#ISO/IEC 9899-1999]\] Section 7.15, "Variable arguments," and Section 7.19.6.8, "The {{vfprintf}} function"
\[[Seacord 05c|AA. C References#Seacord 05c]\] |
Automated Detection
Tool | Version | Checker | Description | ||||||
---|---|---|---|---|---|---|---|---|---|
Astrée |
| Supported, but no explicit checker | |||||||
Helix QAC |
| C0185, C0184 | |||||||
Klocwork |
| SV.FMT_STR.PRINT_PARAMS_WRONGNUM.FEW SV.FMT_STR.PRINT_PARAMS_WRONGNUM.MANY SV.FMT_STR.SCAN_PARAMS_WRONGNUM.FEW SV.FMT_STR.SCAN_PARAMS_WRONGNUM.MANY | |||||||
LDRA tool suite |
| 41 S | Enhanced Enforcement | ||||||
Parasoft C/C++test |
| CERT_C-DCL10-a | The number of format specifiers in the format string and the number of corresponding arguments in the invocation of a string formatting function should be equal | ||||||
PC-lint Plus |
| 558, 719 | Assistance provided: reports issues involving format strings | ||||||
Polyspace Bug Finder |
| Checks for format string specifiers and arguments mismatch (rec. partially covered) |
Related Guidelines
ISO/IEC TR 24772:2013 | Subprogram Signature Mismatch [OTR] |
MISRA C:2012 | Rule 17.1 (required) |
MITRE CWE | CWE-628, Function call with incorrectly specified arguments |
Bibliography
[Seacord 2013] | Chapter 6, "Formatted Output" |
...
DCL09-A. Declare functions that return an errno with a return type of errno_t 02. Declarations and Initialization (DCL) DCL11-A. Understand the type issues associated with variadic functions