Request for Review: 6909089

Michael McMahon Michael.McMahon at Sun.COM
Thu Dec 10 08:21:28 PST 2009


Alan Bateman wrote:
> Christopher Hegarty -Sun Microsystems Ireland wrote:
>>
>> CR 6909089: Memory leak occurs by lack of free for read buffer in 
>> SocketInputStream#read()
>>
>> Webrev:
>>    http://cr.openjdk.java.net/~chegar/6909089/webrev.0/webrev/
>>
>> Bug description says it all.
>>
>> -Chris.
> Looks okay to me - I assume you've checked all other possible return 
> paths.
>
> -Alan.
Yes, it looks like all paths in socketRead0 are covered.

- Michael.



More information about the net-dev mailing list