Code Review Request for Bug #7123415

Brandon Passanisi brandon.passanisi at oracle.com
Wed Jan 4 10:28:00 PST 2012


Resending...

Hello net-dev.  I was wondering if somebody could please review the 
following proposed fix for 7123415:

    Webrev URL: http://cr.openjdk.java.net/~bpassani/7123415/0/webrev/
    <http://cr.openjdk.java.net/%7Ebpassani/7123415/0/webrev/>

The proposed fix is noted within the bug description:

    In src/solaris/native/java/net/net_util_md.c, there are instances of
    the network interface index being read in by a "%02x".  These
    instances are on lines 616 and 1110 of net_util_md.c and they appear
    involve code that obtains the default loopback interface index and
    default ipv6 interface index.  It has been shown by CR 7078386 that
    the index value can be more than two digits.  Therefore, these
    instances should likely be changed to "%08x" as instructed in CR
    7078386.

I'm not sure how to come up with a reproducible test case for this as 
there currently aren't any tests which expose this specific problem.  
This bug was found as part of the investigation for CR 7078386.  A 
reproducible test case would likely involve somehow setting the default 
loopback and ipv6 interfaces to a value more than 255, and I'm not sure 
if this can be done.  It is hoped that the approval given for the same 
type of fix pushed for CR 7078386 will allow the fix here.

Thanks.
-- 
Oracle <http://www.oracle.com>
Brandon Passanisi | Principle Member of Technical Staff

Green Oracle <http://www.oracle.com/commitment> Oracle is committed to 
developing practices and products that help protect the environment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/net-dev/attachments/20120104/4602936e/attachment.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/gif
Size: 658 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/net-dev/attachments/20120104/4602936e/attachment.gif 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: image/gif
Size: 356 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/net-dev/attachments/20120104/4602936e/attachment-0001.gif 


More information about the net-dev mailing list