The C99 {{The C Wiki Markup fopen()
}} function is used to open an existing file or create a new one \[[ISO/IEC 9899:1999|AA. C References#ISO/IEC 9899-1999]\]. However, {{fopen()}} does not indicate if an existing file has been opened for writing or a new file has been created. This may lead to a program overwriting or accessing an unintended file.
For examples on how to just check for the existence of a file without actually opening it, please see FIO10-A. Take care when using the rename() function.
Non-Compliant Code Example: fopen()
. The C11 version of the fopen()
and fopen_s()
functions provides a mode flag, x
, that provides the mechanism needed to determine if the file that is to be opened exists. Not using this mode flag can lead to a program overwriting or accessing an unintended file.
Noncompliant Code Example (fopen()
)
In this noncompliant code In this non-compliant coding example, the file referenced by file_name
is opened for writing. This example is non-compliant, however, noncompliant if the programmer's intent was to create a new file, but the reference referenced file already exists.
Code Block | ||||
---|---|---|---|---|
| ||||
char *file_name; FILE *fp; /* initializeInitialize file_name */ fp = fopen(file_name, "w"); if (!fp) { /* handleHandle error */ } |
...
Noncompliant Code Example
...
(fopen_s()
...
, C11 Annex K)
The C11 Annex K fopen_s()
function defined in ISO/IEC TR 24731-1:2007 is designed to improve the security of the fopen()
function. However, like Like the fopen()
function, fopen_s()
provides no a mechanism to determine if an existing file has been opened for writing or a new file has been createdwhether the file exists. See below for use of the exclusive mode flag.
Code Block | ||||
---|---|---|---|---|
| ||||
char *file_name; FILE *fp; /* initializeInitialize file_name */ errno_t res = fopen_s(&fp, file_name, "w"); if (res != 0) { /* handleHandle error */ } |
Compliant Solution
...
(fopen_s()
, C11 Annex K)
The C Standard provides a new flag to address this problem. Subclause 7.21.5.3, paragraph 5 [ISO/IEC 9899:2011], states:
Opening a file with exclusive mode ('x' as the last character in the mode argument) fails if the file already exists or cannot be created. Otherwise, the file is created with exclusive (also known as non-shared) access to the extent that the underlying system supports exclusive access.
This option is also provided by the GNU C library [Loosemore 2007].
This compliant solution uses the x
mode character to instruct fopen_s()
to fail rather than open an existing file:
Code Block | ||||
---|---|---|---|---|
| ||||
char *file_name;
/* Initialize file_name */
FILE *fp;
errno_t res = fopen_s(&fp, file_name, "wx");
if (res != 0) {
/* Handle error */
}
|
Use of this option allows for the easy remediation of legacy code. However, note that Microsoft Visual Studio 2012 and earlier do not support the x
mode character [MSDN].
Compliant Solution (open()
, POSIX)
The open()
function, as defined in the Standard for Information Technology—Portable Operating System Interface (POSIX®), Base Specifications, Issue 7 [IEEE Std 1003.1:2013], is available on many platforms and provides finer control than fopen()
. In particular, open()
accepts the O_CREAT
and O_EXCL
flags. When used together, these flags instruct the open()
function to fail if the file specified by file_name
already The {{open()}} function as defined in the Open Group Base Specifications Issue 6 \[[Open Group 04|AA. C References#Open Group 04]\] is available on many platforms and provides the control that {{fopen()}} does not provide. If the {{O_CREAT}} and {{O_EXCL}} flags are used together, the {{open()}} function fails if the file specified by {{file_name}} already exists. Wiki Markup
Code Block | ||||
---|---|---|---|---|
| ||||
char *file_name; int new_file_mode; /* initializeInitialize file_name and new_file_mode */ int fd = open(file_name, O_CREAT | O_EXCL | O_WRONLY, new_file_mode); if (fd == -1) { /* Handle Errorerror */ } |
...
Care should be observed taken when using {{O_EXCL
}} with remote file systems as because it does not work with NFS version 2. NFS version 3 added support for {{O_EXCL
}} mode in {{open()
}}. IETF RFC 1813 [Callaghan 1995] defines the {{EXCLUSIVE
}} value to the {{mode
}} argument of {{CREATE}} \[[Callaghan 95|AA. C References#Callaghan 95]\]. CREATE
:
EXCLUSIVE
specifies that the server is to follow exclusive creation semantics, using the verifier to ensure exclusive creation of the target. No attributes may be provided in this case, since the server may use the target file metadata to store the createverf3 verifier.
Compliant Solution: fopen()
(GNU)
Wiki Markup |
---|
Section 12.3 of the GNU C Library says: \[[Loosemore 07|AA. C References#Loosemore 07]\] |
The GNU C library defines one additional character for use in
opentype
: the character 'x
' insists on creating a new fileâ”if a filefilename
already exists,fopen
fails rather than opening it. If you use 'x
' you are guaranteed that you will not clobber an existing file. This is equivalent to theO_EXCL
option to theopen
function.
Code Block | ||
---|---|---|
| ||
char *file_name;
/* initialize file_name */
FILE *fp = fopen(file_name, "wx");
if (!fp) {
/* Handle Error */
}
|
Use of this non portable extension can allow for easy remediation of legacy code.
Compliant Solution: fdopen()
(POSIX)
Wiki Markup |
---|
For code that operates on {{FILE}} pointers and not file descriptors, the POSIX {{fdopen()}} function can be used to associate an open stream with the file descriptor returned by {{open()}}, as shown in this compliant solution \[[Open Group 04|AA. C References#Open Group 04]\]. |
For examples of how to check for the existence of a file without opening it, see recommendation FIO10-C. Take care when using the rename() function.
Compliant Solution (fdopen()
, POSIX)
For code that operates on FILE
pointers and not file descriptors, the POSIX fdopen()
function can be used to associate an open stream with the file descriptor returned by open()
, as shown in this compliant solution [IEEE Std 1003.1:2013]:
Code Block | ||||
---|---|---|---|---|
| ||||
Code Block | ||||
| ||||
char *file_name; int new_file_mode; FILE *fp; int fd; /* initializeInitialize file_name and new_file_mode */ fd = open(file_name, O_CREAT | O_EXCL | O_WRONLY, new_file_mode); if (fd == -1) { /* Handle Errorerror */ } fp = fdopen(fd, "w"); if (fp == NULL) { /* Handle error */ } |
Compliant Solution (Windows)
The Win32 API CreateFile()
allows a programmer to create or open a file depending on the flags passed in. Passing in the CREATE_NEW
flag ensures the call fails if the file already exists. This compliant solution demonstrates how to open a file for reading and writing without sharing access to the file such that the call fails if the file already exists.
Code Block | ||||
---|---|---|---|---|
| ||||
TCHAR *file_name; HANDLE hFile = CreateFile(file_name, GENERIC_READ | GENERIC_WRITE, 0, 0, CREATE_NEW, FILE_ATTRIBUTE_NORMAL, 0); if (INVALID_HANDLE_VALUE == hFile) { DWORD err = GetLastError(); if (ERROR_FILE_EXISTS == err) { /* Handle file exists error */ } else { /* Handle other Errorerror */ } } |
Risk Assessment
The ability to determine if whether an existing file has been opened or a new file has been created provides greater assurance that the intended file is accessed, or perhaps more importantly, a file other than the intended file is not acted upon.
Recommendation | Severity | Likelihood | Remediation Cost | Priority | Level |
---|---|---|---|---|---|
FIO03- |
C |
Medium |
Probable |
High | P4 | L3 |
Automated Detection
Tool | Version | Checker | Description | ||||||
---|---|---|---|---|---|---|---|---|---|
Coverity | 6.5 | OPEN_ARGS | Fully implemented | ||||||
Helix QAC |
| C5012 | |||||||
LDRA tool suite |
| 44 S | Enhanced Enforcement |
Related Vulnerabilities
Search for vulnerabilities resulting from the violation of this rule on the CERT website.
References
...
Related Guidelines
SEI CERT C++ Coding Standard | VOID FIO03-CPP. Do not make assumptions about fopen() and file creation |
ISO/IEC TR 24731-1:2007 | Section 6.5.2.1, "The fopen_s Function" |
Bibliography
[Callaghan 1995] | IETF RFC 1813 NFS Version 3 Protocol Specification |
[IEEE Std 1003.1:2013] | System Interfaces: open |
[ISO/IEC 9899:2011] | Subclause 7.21.5.3, "The fopen Function"Subclause K.3.5.2.1, |
...
"The |
...
fopen_s |
...
Function" | |
[Loosemore 2007] | Section 12.3, |
...
"Opening Streams" | |
[Seacord 2013] | Chapter 8, "File I/O" |
...
Streams"|http://www.gnu.org/software/libc/manual/html_node/Opening-Streams.html] \[[Open Group 04|AA. C References#Open Group 04]\] \[[Seacord 05a|AA. C References#Seacord 05]\] Chapter 7, "File I/O"FIO02-A. Canonicalize path names originating from untrusted sources 09. Input Output (FIO) FIO04-A. Detect and handle input and output errors