Skip to content

8209362: sun/security/ssl/SSLSocketImpl/ReuseAddr.java failed due to "BindException: Address already in use (Bind failed)"#803

Open
vieiro wants to merge 1 commit into
openjdk:pr/802from
vieiro:backports/JDK-8209362
Open

8209362: sun/security/ssl/SSLSocketImpl/ReuseAddr.java failed due to "BindException: Address already in use (Bind failed)"#803
vieiro wants to merge 1 commit into
openjdk:pr/802from
vieiro:backports/JDK-8209362

Conversation

@vieiro
Copy link
Copy Markdown
Contributor

@vieiro vieiro commented May 20, 2026

Backport of JDK-8209362 that updates the test test/sun/security/ssl/SSLSocketImpl/ReuseAddr.java to use the SSLSocketTemplate.java template.

Importantly, it also transforms private to protected void doClientSide() throws Exception in SSLSocketTemplate.java, thus paving the way for other future backports.

SSL tests pass in RHEL-8/gcc 8.5.0, with same failures as in previous PR #802 :

--------------------------------------------------
Test results: passed: 110; failed: 1; error: 4

Note that the number of tests has been increased to 110 from 109, obviously, and that the new test passes too:

TEST RESULT: Passed. Execution successful
--------------------------------------------------
Test results: passed: 1


Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue
  • JDK-8209362 needs maintainer approval

Integration blocker

 ⚠️ Dependency #802 must be integrated first

Issue

  • JDK-8209362: sun/security/ssl/SSLSocketImpl/ReuseAddr.java failed due to "BindException: Address already in use (Bind failed)" (Bug - P3)

Reviewers

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk8u-dev.git pull/803/head:pull/803
$ git checkout pull/803

Update a local copy of the PR:
$ git checkout pull/803
$ git pull https://git.openjdk.org/jdk8u-dev.git pull/803/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 803

View PR using the GUI difftool:
$ git pr show -t 803

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk8u-dev/pull/803.diff

Using Webrev

Link to Webrev Comment

@bridgekeeper
Copy link
Copy Markdown

bridgekeeper Bot commented May 20, 2026

👋 Welcome back avieiro! A progress list of the required criteria for merging this PR into pr/802 will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk
Copy link
Copy Markdown

openjdk Bot commented May 20, 2026

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

@openjdk openjdk Bot changed the title Backport 30ff292d64c6686eb1d8635738c4cedb836f65cf 8209362: sun/security/ssl/SSLSocketImpl/ReuseAddr.java failed due to "BindException: Address already in use (Bind failed)" May 20, 2026
@openjdk
Copy link
Copy Markdown

openjdk Bot commented May 20, 2026

This backport pull request has now been updated with issue and summary from the original commit.

@openjdk openjdk Bot added backport Port of a pull request already in a different code base rfr Pull request is ready for review labels May 20, 2026
@mlbridge
Copy link
Copy Markdown

mlbridge Bot commented May 20, 2026

Webrevs

Copy link
Copy Markdown
Contributor

@jerboaa jerboaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. Confirmed clean modulo path changes test/jdk => jdk/test from 11u to 8u.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Port of a pull request already in a different code base rfr Pull request is ready for review

Development

Successfully merging this pull request may close these issues.

2 participants