RFR: 8364312: debug agent should set FD_CLOEXEC flag rather than explicitly closing every open file [v4]
Kevin Walls
kevinw at openjdk.org
Wed Aug 6 13:50:04 UTC 2025
On Wed, 6 Aug 2025 12:14:50 GMT, Guanqiang Han <ghan at openjdk.org> wrote:
>> Create a new function that marks all file descriptors found in /proc/self/fd with the FD_CLOEXEC flag to ensure they are automatically closed upon execution of a new program via exec().
>
> Guanqiang Han 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 six additional commits since the last revision:
>
> - update some comments
> - Merge remote-tracking branch 'upstream/master' into 8364312
> - a small fix
> - Merge remote-tracking branch 'upstream/master' into 8364312
> - Update exec_md.c
>
> correct an compilation error
> - 8364312: debug agent should set FD_CLOEXEC flag rather than explicitly closing every open file
>
> Create a new function that marks all file descriptors found in /proc/self/fd with the FD_CLOEXEC flag to ensure they are automatically closed upon execution of a new program via exec().
Thanks for the updates, looks good.
-------------
Marked as reviewed by kevinw (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/26568#pullrequestreview-3092673233
More information about the serviceability-dev
mailing list