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