Calling The C Standard, 7.23.9.3 paragraph 2 [ISO/IEC 9899:2024], defines the following behavior for fsetpos():
The
fsetposfunction sets thembstate_tobject (if any) and file position indicator for the stream pointed to bystreamaccording to the value of the object pointed to bypos, which shall be a value obtained from an earlier successful call to thefgetposfunction on a stream associated with the same file
...
.
Invoking the fsetpos() function with any other values for pos is undefined behavior 181.
Noncompliant Code Example
This noncompliant code example
Non-Compliant Code Example
The following non-compliant code attempts to read three values from a file and then set the cursor file position pointer back to the beginning of the file and return to the caller.:
| Code Block | ||||
|---|---|---|---|---|
| ||||
#include <stdio.h> enum { NO_FILE_POS_VALUES = 3 }; errno_t#include <string.h> int opener(FILE * file, int *width, int *height, int *data_offset) { int file_wrc; int file_h; int file_o; int rc; int offset = 0fpos_t offset; if (file == NULL) { return EINVAL; } if (fscanf(file, "%i %i %i", &file_w, &file_h, &file_o) != NO_FILE_POS_VALUES) { return EIO; }memset(&offset, 0, sizeof(offset)); if ((rcfile = fsetpos(file, &offset)) != 0 NULL) { return rc; } *width = file_w; *height = file_h; *data_offset = file_o; return 0; } int main(void) { int width; int height; int data_offset; FILE *file; /* ...return -1; } /* Read in data from file */ filerc = fopenfsetpos("myfile"file, "rb"&offset); if (opener(file, &width, &height, &data_offset)rc != 0 ) { return 0rc; } /* ... */return 0; } |
However, since only Only the return value of a getposan fgetpos() call is a valid argument to be used with setposfsetpos(), ; passing a specified int in instead may not work. It is possible that the position will be set to an arbitrary location in the filevalue of type fpos_t that was created in any other way is undefined behavior 181.
Compliant Solution
In this compliant solution, the initial file position indicator is stored by first calling fgetpos(), which is used to restore the state back to the beginning of the file in the later call to fsetpos().:
| Code Block | ||||
|---|---|---|---|---|
| ||||
#include <stdio.h> enum { NO_FILE_POS_VALUES = 3 }; errno_t#include <string.h> int opener(FILE * file, int *width, int *height, int *data_offset) { int file_w; int file_h; int file_o; int rc; fpos_t offset; if (file == NULL) { return EINVAL; return -1; } if ((rc = fgetpos(file, &offset)) != 0 ) { return rc; } if (fscanf(file, "%i %i %i", &file_w, &file_h, &file_o) != NO_FILE_POS_VALUES) { return EIO; } if ((rc = fsetpos(file, &offset)) != 0 ) { return rc; } *width = return file_wrc; *height = file_h; *data_offset = file_o; return 0; } int main(void) { int width; int height; int data_offset; FILE *file; /* ... /* Read in data from file */ filerc = fopenfsetpos("myfile"file, "rb"&offset); if (opener(file, &width, &height, &data_offset)rc != 0 ) { return return 0rc; } /* ... */return 0; } |
Risk Assessment
The misuse Misuse of the fsetpos() could move function can position a file stream read to a undesired location in the file. If this location held input from the user, the user would then gain control of the variables being read from position indicator to an unintended location in the file.
Rule | Severity | Likelihood |
|---|
Detectable | Repairable | Priority | Level |
|---|---|---|---|
FIO44-C |
2 (medium)
1 (unlikely)
2 (medium)
P4
Medium | Unlikely | No | No | P2 | L3 |
Automated Detection
Tool | Version | Checker | Description | ||||||
|---|---|---|---|---|---|---|---|---|---|
| CodeSonar |
| (customization) | Users can add a custom check for violations of this constraint. | ||||||
| Compass/ROSE | Can detect common violations of this rule. However, it cannot handle cases in which the value returned by | ||||||||
| Cppcheck Premium |
| premium-cert-fio44-c | |||||||
| Helix QAC |
| DF4841, DF4842, DF4843 | |||||||
| Klocwork |
| CERT.FSETPOS.VALUE | |||||||
| LDRA tool suite |
| 82 D | Fully implemented | ||||||
| Parasoft C/C++test |
| CERT_C-FIO44-a | Only use values for fsetpos() that are returned from fgetpos() | ||||||
| Polyspace Bug Finder |
| CERT C: Rule FIO44-C | Checks for invalid file position (rule partially covered) | ||||||
| PVS-Studio |
| V1035 |
Related Vulnerabilities
Search for vulnerabilities resulting from the violation of this rule on the CERT website.
References
...
Related Guidelines
Key here (explains table format and definitions)
Taxonomy | Taxonomy item | Relationship |
|---|---|---|
| ISO/IEC TS 17961:2013 | Using a value for fsetpos other than a value returned from fgetpos [xfilepos] | Prior to 2018-01-12: CERT: Unspecified Relationship |
Bibliography
| [ISO/IEC 9899:2024] | 7.23 |
...
| .9.3, |
...
"The fsetpos Function" |
...