> ### Description
> `Inet6Address/B6206527.java` test creates two instances of ServerSocket, both 
> of which are explicity bound to a Link-Local address. Neither of the 
> ServerSocket instances are explicitly closed meaning there is no guarantee 
> that their associated resources are freed. 
> 
> ### Fix
> Each ServerSocket is instantiated in a try-with-resources block. This ensures 
> that in both cases of success or failure within the try-with-resources block, 
> the sockets are always closed thanks to ServerSocket implementing Closeable. 
> The test is also now started in othervm mode as an added assurance of the 
> test's isolation in the event that resources are not freed.

Conor Cleary has updated the pull request incrementally with one additional 
commit since the last revision:

  Removed othervm argument

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3437/files
  - new: https://git.openjdk.java.net/jdk/pull/3437/files/2cbd69a6..42312d1b

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3437&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3437&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3437.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3437/head:pull/3437

PR: https://git.openjdk.java.net/jdk/pull/3437

Reply via email to