RFR: 8175131: sun.rmi.transport.tcp.TCPChannel.createConnection close connection on timeout

[Mark Sheppard]

Hi Rob,
     updated change looks good and does the job (in jdk8 and jdk9) of 
eliminating the socket connection build up
regressions tests show no side effects.

noticed what appears to be a peculiarity in the test, in that the test's 
server object is exported on the same port as the
Registry! But that seems to be OK ... the end point mappings are set up 
fine and the end point listener
handles the incoming requests for RemoteObjectSample

regards
Mark

On 10/05/2017 21:22, Rob McKenna wrote:
> Good point, I was nervous that there may have been a reason for not
> closing the connection and completely overlooked the obvious.
>
> http://cr.openjdk.java.net/~robm/8175131/webrev.02/
>
> Thanks,
>
>      -Rob
>
> On 10/05/17 03:38, Roger Riggs wrote:
>> Hi Rob,
>>
>> If an exception is thrown in createConnection, it can't return the
>> connection that was created
>> and so it should close it.
>>
>> I would close the connection at line 298, before throwing the exception.
>>
>> Roger
>>
>> On 5/10/2017 3:25 PM, Rob McKenna wrote:
>>> Hi folks,
>>>
>>> Looking for a review of the following simple change:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8175131
>>> http://cr.openjdk.java.net/~robm/8175131/webrev.01/
>>>
>>> Note: I've deliberately limited the connection closure to the described
>>> case. If however you feel that it should be applied generally when an
>>> exception is thrown (in the if and / or else block) let me know!
>>>
>>>      -Rob
>>>