On Sun, 27 Jul 2025 18:24:29 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Fix gcc warning about double close. forkedChildProcess() calls >> closeDescriptors(), which always closes STDERR_FILENO + 1 and STDERR_FILENO >> + 2. However, if it fails at some point after that, then >> forkedChildProcess() will attempt to close these same two files again. Fixed >> by making forkedChildProcess() no longer attempt to close these two files. >> >> I first verified the warning by configuring with >> --with-extra-cflags=-fanalyzer, and then verified the fix by doing the same. >> >> Tested by running jdi nsk test. I'll add some more CI testing. > > Chris Plummer 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 two additional > commits since the last revision: > > - Merge branch 'master' into 8361873_jdwp_close_fd > Merge > - fix gcc warning about double close() Marked as reviewed by mbaesken (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/26487#pullrequestreview-3062973047
