Request for Review: 6909089

Christopher Hegarty -Sun Microsystems Ireland Christopher.Hegarty at Sun.COM
Thu Dec 10 07:46:11 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.

Thanks for the review Alan. Yes I checked all possible return paths and 
this is the only offender.

-Chris.

> 
> -Alan.



More information about the net-dev mailing list