Common mistakes in creating format strings include
int
for width, precision, or the n
conversion specificationThe following are C99-compliant conversion specifications \[[ISO/IEC 9899:1999|AA. C References#ISO/IEC 9899-1999]\]. Using any other specification may result in [undefined behavior|BB. Definitions#undefined behavior]. |
d, i, o, u, x, X, f, F, e, E, g, G, a, A, c, s, p, n, % |
In addition, ISO/IEC 9945-2003 permits the following conversion specifications as synonyms for lc
and ls
:
C, S |
Only some of the conversion specifications can correctly take a length modifier. Using a length modifier on any specification other than the following may result in undefined behavior.
d, i, o, u, x, X, a, A, e, E, f, F, g, G, c, s |
Character class ranges must also be properly specified with a hyphen in between two printable characters. The two following lines are both properly specified. The first accepts any character from a to z, inclusive, while the second accepts anything that is not a to z, inclusive.
[a-z] [^a-z] |
Note that the range is in terms of character code values, and on an EBCDIC platform it will include some nonalphabetic codes. Consequently, the isalpha()
function should be used to verify the input.
Mismatches between arguments and conversion specifications may result in undefined behavior. Many compilers can diagnose type mismatches in formatted output function invocations.
const char *error_msg = "Resource not available to user."; int error_type = 3; /* ... */ printf("Error (type %s): %d\n", error_type, error_msg); |
This compliant solution ensures that the format arguments match their respective format specifications.
const char *error_msg = "Resource not available to user."; int error_type = 3; /* ... */ printf("Error (type %d): %s\n", error_type, error_msg); |
The width and precision arguments to printf()
format directives must be of type int
. According to C99:
A field width, or precision, or both, may be indicated by an asterisk ('*'). In this case an argument of type
int
supplies the field width or precision.
Passing them as any other type leads to undefined behavior. In this noncompliant code example, the width and precision are specified using parameters declared to be of size_t
type. These are unsigned types that may not be the same size as int
.
int print_int(int i, size_t width, size_t prec) { int n; n = printf("%*.*d", width, prec, i); return n; } |
In this compliant solution, the field width and precision arguments to printf()
format directives are of type int
.
int print_int(int i, int width, int prec) { int n; n = printf("%*.*d", width, prec, i); return n; } |
In most cases, incorrectly specified format strings will result in abnormal program termination.
Recommendation |
Severity |
Likelihood |
Remediation Cost |
Priority |
Level |
---|---|---|---|---|---|
FIO00-C |
high |
unlikely |
medium |
P6 |
L2 |
The LDRA tool suite V 7.6.0 can detect violations of this recommendation.
GCC Compiler can detect violations of this recommendation when the -Wformat
flag is used.
Search for vulnerabilities resulting from the violation of this rule on the CERT website.
This rule appears in the C++ Secure Coding Standard as FIO00-CPP. Take care when creating format strings.
\[[ISO/IEC 9899:1999|AA. C References#ISO/IEC 9899-1999]\] Section 7.19.6.1, "The {{fprintf}} function" \[[MITRE 07|AA. C References#MITRE 07]\] [CWE ID 686|http://cwe.mitre.org/data/definitions/686.html], "Function Call With Incorrect Argument Type" |
09. Input Output (FIO) 09. Input Output (FIO)