On Wed, 17 Aug 2022 15:31:14 GMT, Roger Riggs <rri...@openjdk.org> wrote:
> Revised test to check and close all of the streams used in the pipeline. > > The test now checks the content length and closes each stdout and stderr > stream. > > The previous technique to redirect stderr to stdout was ineffective because > the fd/stream was still created in the parent process. Created > [8291986](https://bugs.openjdk.org/browse/JDK-8291986) > ProcessBuilder.redirectErrorStream(true) leaves error stream available to > investigate test/jdk/java/lang/ProcessBuilder/PipelineLeaksFD.java line 40: > 38: /* > 39: * @test > 40: * @bug 8289643 9291760 Hello Roger, there's a typo here in the new bug id. Should have been `8291760`. Other than that, the rest of the change looks fine to me. ------------- PR: https://git.openjdk.org/jdk/pull/9905