The ISO/IEC 9899-1999 C standard function fopen() function is typically used to open an existing file or create a new one. However, The C11 version of the fopen() does not indicate if an existing file has been opened for writing or a new file has been created. This may function 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 example, an attempt is made to check whether a file exists before opening it for writing by trying to open the file for readingthe file referenced by file_name is opened for writing. This example is noncompliant if the programmer's intent was to create a new file, but the referenced file already exists.
| Code Block | ||||
|---|---|---|---|---|
| ||||
char *file_name; /* ... */ FILE *fp = fopen("foo.txt","r"); if (!fp) { /* file does not exist */ ; /* Initialize file_name */ fp = fopen("foo.txt",file_name, "w"); /* ... */ fclose(fp); } else { if (!fp) { /* fileHandle existserror */ fclose(fp); } /* ... */ |
| Wiki Markup |
|---|
However, this code suffers from a _Time of Check, Time of Use_ (or _TOCTOU_) vulnerability (see \[[Seacord 05|AA. C References#Seacord 05]\] Section 7.2). On a shared multitasking system there is a window of opportunity between the first call of {{fopen()}} and the second call for a malicious attacker to, for example, create a link with the given filename to an existing file, so that the existing file is overwritten by the second call of {{fopen()}} and the subsequent writing to the file. |
Non-Compliant Code Example: fopen_s() (ISO/IEC TR 24731-1)
The fopen_s() function defined in ISO/IEC TR 24731-1-2007 is designed to improve the security of the fopen() function. However, like fopen(), fopen_s() provides no mechanism to determine if an existing file has been opened for writing or a new file has been created. The code below contains the same TOCTOU race condition as in the first Non-Compliant Code Example.
| Code Block | ||
|---|---|---|
| ||
/* ... */
FILE *fptr;
errno_t res = fopen_s(&fptr,"foo.txt", "r");
if (res != 0) { /* file does not exist */
res = fopen_s(&fptr,"foo.txt", "w");
/* ... */
fclose(fptr);
} else {
fclose(fptr);
}
/* ... */
|
Compliant Solution: open() (POSIX)
Compliant Solution (fopen("x"), C11)
Starting in C11 a new mode suffix ("x") was added to the fopen() function which causes fopen() to return NULL if the file already exists or cannot be created [ISO/IEC 9899:2011].
| Code Block | ||||
|---|---|---|---|---|
| ||||
char *file_name;
FILE *fp;
/* Initialize file_name */
fp = fopen(file_name, "wx");
if (!fp) {
/* Handle error */
}
|
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 {{fopen()}} function does not indicate if an existing file has been opened for writing or a new file has been created. However, 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 such a mechanism. If the {{O_CREAT}} and {{O_EXCL}} flags are used together, the {{open()}} function fails when the file specified by {{file_name}} already exists.Wiki Markup
| Code Block | ||||
|---|---|---|---|---|
| ||||
char *file_name; int new_file_mode; /* ... Initialize 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(); see . IETF RFC 1813 Callaghan 95, in particular [Callaghan 1995] defines the EXCLUSIVE value to the mode argument of CREATE:
EXCLUSIVEspecifies 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.
...
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)
{{fdopen()}} \[[Open Group 04|AA. C References#Open Group 05]\] can be used in conjunction with {{open()}} to determine if a file is opened or created, and then associate a stream with the file descriptor.For code that operates on Wiki Markup 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 | ||||
|---|---|---|---|---|
| ||||
char *file_name; int new_file_mode; /* ... */ FILE *fp; int fd; /* Initialize 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 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 error */ } } |
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 a file other than the intended file accessed is the one that was intendedis not acted upon.
Recommendation | Severity | Likelihood |
|---|
Detectable | Repairable | Priority | Level |
|---|---|---|---|
FIO03- |
2 (medium)
2 (probable)
C | Medium | Probable | No | No | 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 | ||||||
| Polyspace Bug Finder |
| CERT C: Rec. FIO03-C | Checks for file not opened in exclusive mode |
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.19.3, "Files," and Section 7.19.4, "Operations on Files"
\[[ISO/IEC TR 24731-1-2007|AA. C References#SO/IEC TR 24731-1-2007]\] Section 6.5.2.1, "The fopen_s function"
\[[Open Group 04|AA. C References#Open Group 04]\]
\[[Seacord 05|AA. C References#Seacord 05]\] Chapter 7, "File I/O" |
...
| border | false |
|---|
Related Guidelines
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" |
| [Loosemore 2007] | Section 12.3, "Opening Streams" |
| [Seacord 2013] | Chapter 8, "File I/O" |
...
| Column | ||||
|---|---|---|---|---|
| ||||
FIO02-A. Canonicalize file names originating from untrusted sources 09. Input Output (FIO) FIO04-A. Detect and handle input and output errors |
...
| bgColor | red |
|---|---|
| width | 10% |
...
| bgColor | red |
|---|---|
| width | 10% |
...