diff --git a/test/hotspot/jtreg/runtime/8176717/TestInheritFD.java b/test/hotspot/jtreg/runtime/8176717/TestInheritFD.java index fd71c9fd498..38d462ec9d3 100644 --- a/test/hotspot/jtreg/runtime/8176717/TestInheritFD.java +++ b/test/hotspot/jtreg/runtime/8176717/TestInheritFD.java @@ -3,6 +3,9 @@ import static java.lang.Long.parseLong; import static java.lang.System.getProperty; import static java.lang.management.ManagementFactory.getOperatingSystemMXBean; import static java.nio.file.Files.readAllBytes; +import static java.nio.file.Files.readSymbolicLink; +import static java.util.Arrays.stream; +import static java.util.stream.Collectors.joining; import static jdk.test.lib.process.ProcessTools.createJavaProcessBuilder; import java.io.File; @@ -37,6 +40,7 @@ import com.sun.management.UnixOperatingSystemMXBean; * @test TestInheritFD * @bug 8176717 8176809 * @summary a new process should not inherit open file descriptors + * @requires (os.family != "mac") * @library /test/lib * @modules java.base/jdk.internal.misc * java.management @@ -66,10 +70,11 @@ public class TestInheritFD { public static final String LEAKS_FD = "VM RESULT => LEAKS FD"; public static final String RETAINS_FD = "VM RESULT => RETAINS FD"; public static final String EXIT = "VM RESULT => VM EXIT"; + public static final String LOG_SUFFIX = ".strangelogsuffixthatcanbecheckedfor"; // first VM public static void main(String[] args) throws Exception { - String logPath = createTempFile("logging", ".log").getName(); + String logPath = createTempFile("logging", LOG_SUFFIX).getName(); File commFile = createTempFile("communication", ".txt"); ProcessBuilder pb = createJavaProcessBuilder( @@ -107,6 +112,8 @@ public class TestInheritFD { "" + (supportsUnixMXBean()?+unixNrFD():-1)); pb.inheritIO(); // in future, redirect information from third VM to first VM pb.start(); + + findOpenLogFile(); } } @@ -119,8 +126,13 @@ public class TestInheritFD { if(supportsUnixMXBean()){ long thisFDCount = unixNrFD(); - System.out.println("This VM FD-count (" + thisFDCount + ") should be strictly less than parent VM FD-count (" + parentFDCount + ") as log file should have been closed"); - System.out.println(thisFDCount "###" + f.getAbsolutePath() + " -> " + linkTarget(f)) + .collect(joining("\n"))); + + return stream(dir.listFiles()) + .map(TestInheritFD::linkTarget) + .filter(f -> f.getName().endsWith(LOG_SUFFIX)) + .findAny() + .isPresent(); + } + static void windows(File f, long parentPid) throws InterruptedException { System.out.println("waiting for pid: " + parentPid); ProcessHandle.of(parentPid).ifPresent(handle -> handle.onExit().join()); System.out.println("trying to rename file to the same name: " + f); - System.out.println(f.renameTo(f)?RETAINS_FD:LEAKS_FD); // this parts communicates a closed file descriptor by printing "CLOSED FD" + System.out.println(f.renameTo(f)?RETAINS_FD:LEAKS_FD); // this parts communicates a closed file descriptor by printing "VM RESULT => RETAINS FD" } } \ No newline at end of file