On Tue, 13 Apr 2021 10:30:18 GMT, Conor Cleary <ccle...@openjdk.org> wrote:

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

Marked as reviewed by dfuchs (Reviewer).

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

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

Reply via email to