RFR: 8289643: File descriptor leak with ProcessBuilder.startPipeline [v7]
Roger Riggs
rriggs at openjdk.org
Mon Jul 18 17:53:22 UTC 2022
On Fri, 8 Jul 2022 13:29:12 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Roger Riggs has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 10 additional commits since the last revision:
>>
>> - Remove diagnostics output from ProcessBuilder
>> - Merge branch '8289643-pipeline-start-leak' of https://github.com/RogerRiggs/jdk into 8289643-pipeline-start-leak
>> - Add DEBUG diagnostics to determine cause of GitHub Action test failures due to unexpected pipes
>> - Merge branch 'master' into 8289643-pipeline-start-leak
>> - Add DEBUG diagnostics to determine cause of GitHub Action test failures due to unexpected pipes
>> - Use DirectoryStream and handle IOExceptions
>> - remove debug output
>> - Add TWR for Files.walk of /proc
>> - Cleanup of PipelineLeaksFD test improving error messages and source cleanup.
>> - 8289643: File descriptor leak with ProcessBuilder.startPipeline
>
> test/jdk/java/lang/ProcessBuilder/PipelineLeaksFD.java line 101:
>
>> 99:
>> 100: Set<PipeRecord> pipesAfter = myPipes();
>> 101: if (!pipesBefore.equals(pipesAfter)) {
>
> Since `myPipes()` can return null, should there be a null check here for `pipesBefore`?
No longer returns null.
-------------
PR: https://git.openjdk.org/jdk/pull/9414
More information about the core-libs-dev
mailing list