[jdk17u-dev] RFR: 8289643: File descriptor leak with ProcessBuilder.startPipeline [v2]

[Rinku Kumar Maurya]

> clean backport to fix file descriptor leak with ProcessBuilder.startPipeline. When using ProcessBuilder.startPipeline the pipes (or pipe if only 2 ProcessBuilder instance used) created are never closed.
> 
> 
> with changes :-
> ==============================
> Test summary
> ==============================
>    TEST                                              TOTAL  PASS  FAIL ERROR   
>    jtreg:test/jdk/java/lang/ProcessBuilder/PipelineLeaksFD.java
>                                                          1     1     0     0   
> ==============================
> TEST SUCCESS
> 
> 
> without changes :-
> ==============================
> Test summary
> ==============================
>    TEST                                              TOTAL  PASS  FAIL ERROR   
>    jtreg:test/jdk/java/lang/ProcessBuilder/PipelineLeaksFD.java
>                                                          1     0     1     0 
> ==============================
> TEST FAILURE
> 
> with expected output :-
> test PipelineLeaksFD.checkForLeaks(java.util.ImmutableCollections$ListN at 44d1a216): failure
> java.lang.AssertionError: More or fewer pipes than expected
> ...

Rinku Kumar Maurya has updated the pull request incrementally with one additional commit since the last revision:

  Backport 226b8e680a82096b3d268703872d30182c10e326

-------------

Changes:
  - all: https://git.openjdk.org/jdk17u-dev/pull/2462/files
  - new: https://git.openjdk.org/jdk17u-dev/pull/2462/files/0603c6ed..3d18ffad

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=2462&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=2462&range=00-01

  Stats: 11 lines in 1 file changed: 7 ins; 1 del; 3 mod
  Patch: https://git.openjdk.org/jdk17u-dev/pull/2462.diff
  Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/2462/head:pull/2462

PR: https://git.openjdk.org/jdk17u-dev/pull/2462