Re: Request for Review: 6909089

2009-12-10 Thread Michael McMahon
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 -

Re: Request for Review: 6909089

2009-12-10 Thread Christopher Hegarty -Sun Microsystems Ireland
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

Re: Request for Review: 6909089

2009-12-10 Thread Jean-Christophe Collet
I approve this change. On 12/10/09 16:35, 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. -Chr

Re: Request for Review: 6909089

2009-12-10 Thread Alan Bateman
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 che