Re: RFR8003991 (take two)

2012-11-27 Thread John Zavgren
inconvenience. John - Original Message - From: dmitry.samers...@oracle.com To: john.zavg...@oracle.com Cc: net-dev@openjdk.java.net Sent: Monday, November 26, 2012 11:54:58 PM GMT -05:00 US/Canada Eastern Subject: Re: RFR8003991 (take two) John, I guess free(loRoutes) should be here. -Dmitry On 20

Re: RFR8003991 (take two)

2012-11-27 Thread Chris Hegarty
On 11/27/2012 09:42 AM, Alan Bateman wrote: On 26/11/2012 23:19, John Zavgren wrote: Greetings: I'm posting the webrev image of a change that I made that eliminates a potential file descriptor leak. (The previous RFR referenced a webrev image with the right file name but the wrong content. 'sor

Re: RFR8003991 (take two)

2012-11-27 Thread Alan Bateman
On 26/11/2012 23:19, John Zavgren wrote: Greetings: I'm posting the webrev image of a change that I made that eliminates a potential file descriptor leak. (The previous RFR referenced a webrev image with the right file name but the wrong content. 'sorry about that.) http://cr.openjdk.java.net

Re: RFR8003991 (take two)

2012-11-26 Thread Dmitry Samersoff
John, I guess free(loRoutes) should be here. -Dmitry On 2012-11-27 03:19, John Zavgren wrote: > Greetings: > > I'm posting the webrev image of a change that I made that eliminates a > potential file descriptor leak. (The previous RFR referenced a webrev image > with the right file name but t

Re: RFR8003991 (take two)

2012-11-26 Thread Kurchi Hazra
Aren't we effectively calling fclose(NULL) here now? - Kurchi On 26.11.2012 15:19, John Zavgren wrote: Greetings: I'm posting the webrev image of a change that I made that eliminates a potential file descriptor leak. (The previous RFR referenced a webrev image with the right file name but th

RFR8003991 (take two)

2012-11-26 Thread John Zavgren
Greetings: I'm posting the webrev image of a change that I made that eliminates a potential file descriptor leak. (The previous RFR referenced a webrev image with the right file name but the wrong content. 'sorry about that.) http://cr.openjdk.java.net/~mullan/webrevs/8003991/webrev.00/ Thanks