Description

The application creates temporary withouth properly restricting their privileges or access modes, allowing an attacker to craft new attack vectors

Impact

- Get access to the created temporary files. - Tamper data contained in the created temporary files.

Recommendation

Create the temporary files in a different directory than the default provided by the Operative system and ensure it has 0600 permission mask.

Threat

Authenticated attacker with local access to the server.

Expected Remediation Time

⏱️ 60 minutes.

Details

This java method has the following signature:

public static File createTempFile(
  // The prefix string defines the files name;
  // must be at least three characters long
  String prefix,
  // The suffix string defines the file's extension;
  // if null the suffix ".tmp" will be used
  String suffix,
  // For default temporary-file directory null is to passed....
Show more

The first two arguments do not affect security of the created file.

In Linux, files and directories are different entities, they have their own permissions and are isolated from other files and directories.

Protections you apply to the directory do not affect the files inside of it. For example, if I protect the directory I can prevent an attacker from executing the ls command, but nothing impedes the attacker from executing cat directory/file.

To prevent access to the file we must protect the file, not the directory.

Additionally, we make sure that permissions are set atomically. If we do not, the following situation can happen:

At moment A we File.createTempFile() a file.

At moment B we add secure permissions to the file.

After moment B the file is secured. However, between moment A and moment B the file has insecure permissions and an attacker had enough opportunity to get control over it.

Vulnerable implementation

The java.io.File.createTempFile method creates files with write permissions in groups and other:

import java.io.File;

public class Test {
  public static void main(String[] args){
    try {
      System.out.println(File.createTempFile("xxx", null));
    catch (Exception e) {}...
Show more

Secure implementation

Use java.nio.file.Files.createTempFile.

Use the attrs argument (an optional list of file attributes to set atomically when creating the file).

Requirements

Fixes