Datagram socket leak

Michael McMahon michael.x.mcmahon at oracle.com
Fri Sep 9 05:31:11 PDT 2011


On 08/09/11 21:40, Alan Bateman wrote:
> Michael McMahon wrote:
>> Sigh. Hopefully this is the last webrev.
>>
>> http://cr.openjdk.java.net/~michaelm/7085981/webrev.4/
>>
>> Socket.java changed from last one. If there's no objection
>> I'll push this version.
>>
>> - Michael.
> DatagramSocket(SocketAddress) creates an unbound socket if the 
> parameter is null. If I read the changes correctly then it will close 
> it for this case. I think the try/finally needs to be narrowed to just 
> the bind. Same thing in MulticastSocket.
>
> -Alan.
>
>
Yes, the regression tests picked that up as well. Well spotted.

http://cr.openjdk.java.net/~michaelm/7085981/webrev.5/

Thanks
Michael.







More information about the net-dev mailing list