Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8211844: [aix] ProcessBuilder: Piping between created processes does not work.
authorsgroeger
Fri, 07 Dec 2018 14:35:57 +0100
changeset 52943 c9fb47668dbe
parent 52942 746602d9682f
child 52944 d75110673dc9
8211844: [aix] ProcessBuilder: Piping between created processes does not work. Reviewed-by: cjplummer, simonis, goetz
src/java.base/unix/classes/java/lang/ProcessImpl.java file | annotate | diff | comparison | revisions
test/jdk/ProblemList.txt file | annotate | diff | comparison | revisions 
--- a/src/java.base/unix/classes/java/lang/ProcessImpl.java	Tue Dec 11 09:36:49 2018 -0500
+++ b/src/java.base/unix/classes/java/lang/ProcessImpl.java	Fri Dec 07 14:35:57 2018 +0100
@@ -412,7 +412,7 @@
                         new BufferedOutputStream(
                             new FileOutputStream(newFileDescriptor(fds[0])));
 
-                stdout = (fds[1] == -1) ?
+                stdout = (fds[1] == -1 || forceNullOutputStream) ?
                          ProcessBuilder.NullInputStream.INSTANCE :
                          new BufferedInputStream(
                              stdout_inner_stream =
@@ -446,7 +446,7 @@
                         ProcessBuilder.NullOutputStream.INSTANCE :
                         new ProcessPipeOutputStream(fds[0]);
 
-                stdout = (fds[1] == -1) ?
+                stdout = (fds[1] == -1 || forceNullOutputStream) ?
                          ProcessBuilder.NullInputStream.INSTANCE :
                          new DeferredCloseProcessPipeInputStream(fds[1]);
 
--- a/test/jdk/ProblemList.txt	Tue Dec 11 09:36:49 2018 -0500
+++ b/test/jdk/ProblemList.txt	Fri Dec 07 14:35:57 2018 +0100
@@ -515,7 +515,6 @@
 # jdk_lang
 
 java/lang/StringCoding/CheckEncodings.sh                        7008363 generic-all
-java/lang/ProcessBuilder/PipelineTest.java                      8211844 aix-ppc64
 java/lang/ProcessHandle/InfoTest.java                           8211847 aix-ppc64
 
 ############################################################################
jdk-sandbox