Skip to content

8347740: java/io/File/createTempFile/SpecialTempFile.java failing #794

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
sophia-guo wants to merge 2 commits into
base: master
Choose a base branch
from
Open

8347740: java/io/File/createTempFile/SpecialTempFile.java failing #794

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
df53e4a
Backport f2a9d26b2e409a7216d967ebb6b92726e8ed65c3
sophia-guo May 8, 2026
5857f18
Backport 8183344
sophia-guo May 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
55 changes: 32 additions & 23 deletions jdk/test/java/io/File/createTempFile/SpecialTempFile.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,17 @@

import java.io.File;
import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;

public class SpecialTempFile {

//
// If exceptionExpected == null, then any IOException thrown by
// File.createTempFile is ignored.
//
private static void test(String name, String[] prefix, String[] suffix,
boolean exceptionExpected) throws IOException
Boolean exceptionExpected) throws IOException
{
if (prefix == null || suffix == null
|| prefix.length != suffix.length)
Expand All @@ -45,46 +51,44 @@ private static void test(String name, String[] prefix, String[] suffix,
final String exceptionMsg = "Unable to create temporary file";
String[] dirs = { null, "." };

Path testPath = Paths.get(System.getProperty("test.dir", "."));
for (int i = 0; i < prefix.length; i++) {
boolean exceptionThrown = false;
File f = null;

for (String dir: dirs) {
Path tempDir = Files.createTempDirectory(testPath, dir);
System.out.println("In test " + name +
", creating temp file with prefix, " +
prefix[i] + ", suffix, " + suffix[i] +
", in dir, " + dir);
", in dir, " + tempDir);

try {
if (dir == null || dir.isEmpty())
f = File.createTempFile(prefix[i], suffix[i]);
else
f = File.createTempFile(prefix[i], suffix[i], new File(dir));
f = File.createTempFile(prefix[i], suffix[i],
tempDir.toFile());
} catch (IOException e) {
if (exceptionExpected) {
if (e.getMessage().startsWith(exceptionMsg))
exceptionThrown = true;
else
System.out.println("Wrong error message:" +
e.getMessage());
} else {
throw e;
if (exceptionExpected != null) {
if (exceptionExpected) {
if (e.getMessage().startsWith(exceptionMsg))
exceptionThrown = true;
else
System.out.println("Wrong error message:" +
e.getMessage());
} else {
throw e;
}

if (exceptionExpected && (!exceptionThrown || f != null))
throw new RuntimeException("IOException expected");
}
}

if (exceptionExpected && (!exceptionThrown || f != null))
throw new RuntimeException("IOException is expected");
}
}
}

public static void main(String[] args) throws Exception {
// Common test
final String name = "SpecialTempFile";
File f = new File(System.getProperty("java.io.tmpdir"), name);
if (!f.exists()) {
f.createNewFile();
}
String[] nulPre = { name + "\u0000" };
String[] nulSuf = { ".test" };
test("NulName", nulPre, nulSuf, true);
Expand All @@ -106,6 +110,11 @@ public static void main(String[] args) throws Exception {
// Test JDK-8013827
String[] resvPre = { "LPT1.package.zip", "com7.4.package.zip" };
String[] resvSuf = { ".temp", ".temp" };
test("ReservedName", resvPre, resvSuf, true);
System.out.println("OS name: " + System.getProperty("os.name") + "\n" +
"OS version: " + System.getProperty("os.version"));

// Here the test is for whether File.createTempFile hangs, so whether
// an exception is thrown is ignored: expectedException == null
test("ReservedName", resvPre, resvSuf, null);
}
}