Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

The fopen() function does not allow the programmer to explicitly specify file access permissions. In this non-compliant code example, if the call to fopen() creates a new file, the access permissions are implementation-defined.

Code Block
bgColor#FFCCCC
char *file_name;
FILE *fp;

/* initialize *fptrfile_name */

fp = fopen(file_name, "w");
if (!fptrfp){
  /* Handlehandle Error */
}

Implementation Details

...

The 'u' character can be thought of as standing for "umask," meaning that these are the same permissions that the file would have been created with had it been created by fopen().

Code Block
bgColor#ccccff

char *file_name;
FILE *fptr;fp;

/* initialize file_name */

errno_t res = fopen_s(&fptrfp, file_name, "w");
if (res != 0) {
  /* Handlehandle Error */
}

Non-Compliant Code Example: open() (POSIX)

Using the POSIX function open() to create a file, but failing to provide access permissions for that file, may cause the file to be created with overly permissive access permissions. This omission has been known to lead to vulnerabilities (for instance, CVE-2006-1174).

Code Block
bgColor#FFCCCC

char *file_name;
int fd;

/* access permissions are missinginitialize file_name */
int 
fd = open(file_name, O_CREAT | O_WRONLY); 
/* access permissions were missing */

if (fd == -1){
  /* Handlehandle Error */
}

Compliant Solution: open() (POSIX)

Access permissions for the newly created file should be specified in the third argument to open(). Again, the permissions are modified by the value of umask().

Code Block
bgColor#ccccff
char *file_name;
int file_access_permissions;

/* initialize file_name and file_access_permissions */

int fd = open(
  file_name, 
  O_CREAT | O_WRONLY, 
  file_access_permissions
);
if (fd == -1){
  /* Handlehandle Error */
}

Wiki Markup
John Viega and Matt Messier also provide the following advice \[[Viega 03|AA. C References#Viega 03]\]:

...