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

[Rinku Kumar Maurya]

On Mon, 13 May 2024 11:40:09 GMT, Rinku Kumar Maurya <duke at openjdk.org> wrote:

> 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
> ...

This pull request has now been integrated.

Changeset: d1e31f6f
Author:    Rinku Kumar Maurya <rkmaurya at amazon.com>
Committer: Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.org/jdk17u-dev/commit/d1e31f6f6a455572c2d5b6f118ce54fa040e0e82
Stats:     150 lines in 2 files changed: 149 ins; 0 del; 1 mod

8289643: File descriptor leak with ProcessBuilder.startPipeline
8290885: java/lang/ProcessBuilder/PipelineLeaksFD.java fail: More or fewer pipes than expected

Reviewed-by: shade
Backport-of: 620c8a045f92126c2552347b9f369405ab2d6d36

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

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