Creating temporary files using insecure methods exposes the application to race conditions on filenames: a malicious user can try to create a file with a predictable name before the application does. A successful attack can result in other files being accessed, modified, corrupted or deleted. This risk is even higher if the application run with elevated permissions.

In the past, it has led to the following vulnerabilities:

• CVE-2014-1858
• CVE-2014-1932

Path.GetTempFileName() generates predictable file names and is inherently unreliable and insecure. Additionally, the method will raise an IOException if it is used to create more than 65535 files without deleting previous temporary files.

Recommended Secure Coding Practices

Out of the box, .NET is missing secure-by-design APIs to create temporary files. To overcome this, one of the following options can be used:

• Use a dedicated sub-folder with tightly controlled permissions
• Created temporary files in a publicly writable folder and make sure:
  • Generated filename is unpredictable
  • File is readable and writable only by the creating user ID
  • File descriptor is not inherited by child processes
  • File is destroyed as soon as it is closed

Noncompliant Code Example

Dim TempPath = Path.GetTempFileName() 'Noncompliant

Using Writer As New StreamWriter(TempPath)
    Writer.WriteLine("content")
End Using

Compliant Solution

Dim RandomPath = Path.Combine(Path.GetTempPath(), Path.GetRandomFileName())

Using FileStream As new FileStream(RandomPath, FileMode.CreateNew, FileAccess.Write, FileShare.None, 4096, FileOptions.DeleteOnClose)
    Using Writer As New StreamWriter(FileStream)
        Writer.WriteLine("content")
    End Using
End Using

See

• OWASP Top 10 2021 Category A1 - Broken Access Control
• OWASP Top 10 2017 Category A9 - Using Components with Known Vulnerabilities
• MITRE, CWE-377 - Insecure Temporary File
• MITRE, CWE-379 - Creation of Temporary File in Directory with Incorrect Permissions
• OWASP, Insecure Temporary File