infernap12/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2026-02-15 04:45:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

8059034: ProcessTools.startProcess() might leak processes

Browse Source 
Reviewed-by: sla, miauno
This commit is contained in:
Jaroslav Bachorik 2014-09-25 12:51:43 +02:00
parent c0f7675bb7
commit 22ced78f0a
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
jdk/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java
View File

@ -36,12 +36,10 @@ import java.util.concurrent.CountDownLatch;
import java.util.Map;
import java.util.concurrent.ExecutionException;
import java.util.concurrent.Future;
import java.util.concurrent.Phaser;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.TimeoutException;
import java.util.function.Predicate;
import java.util.function.Consumer;
import java.util.stream.Collector;
import java.util.stream.Collectors;
import sun.management.VMManagement;
@ -177,6 +175,11 @@ public final class ProcessTools {
for(Map.Entry<Thread, StackTraceElement[]> s : Thread.getAllStackTraces().entrySet()) {
printStack(s.getKey(), s.getValue());
}
if (p.isAlive()) {
p.destroyForcibly();
}
stdoutTask.cancel(true);
stderrTask.cancel(true);
throw e;