RFR8003991 (take two)

Chris Hegarty chris.hegarty at oracle.com
Tue Nov 27 02:02:08 PST 2012


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. 'sorry about that.)
>>
>> http://cr.openjdk.java.net/~mullan/webrevs/8003991/webrev.00/
>>
> John - this doesn't make sense, any chance this is also the wrong webrev?

Yes, it doesn't make sense to me either. I think you may be looking at 
out dated src, or confusing this with an issue you already fixed ;-)

JIRA issue 8003991: "Leaked File f at line 612 of 
jdk8-b62/src/jdk/src/solaris/native/java/net/net_util_md.c in function 
initLoopbackRoutes"

This issue was resolved by your previous patch for 8000203 [1]. The line 
number 612 matches the previously leaked file.

-Chris.

[1] http://hg.openjdk.java.net/jdk8/tl/jdk/rev/909676adaefd

>
> -Alan



More information about the net-dev mailing list