Request for Review 6855335
Christopher Hegarty - Sun Microsystems Ireland
Christopher.Hegarty at Sun.COM
Mon Jun 29 03:11:06 PDT 2009
Hi Michael, Jessie,
I need a code review for some SCTP changes that were found during testing.
CR 6855335:
Several changes in the SCTP implementation.
Webrev:
http://cr.openjdk.java.net/~chegar/6855335/webrev.00/webrev/
1) SctpMultiChannel.branch returns null.
branch was not implemented in the first push of sctp. It is now.
2) SCTP_PRIMARY_ADDR not working correctly for both SctpChannel and
SctpMultiChannel
Fixed native implementation.
3) SctpChannel.connect(SocketAddress,int,int) throws
"java.net.SocketException: Invalid argument" on a connected channel.
Need to check channel state before setting options.
4) SendFailedNotification/Shutdown notification is null sometimes with
SctpMultiChannel.
Fall through in switch statement that implements the notification
handler for SctpMutliChannel which leads to some handlers being
invoked when actually a different notification was received.
5) SendFailedNotification sometimes has a null SocketAddress or
ByteBuffer and invalid errorCode and streamNumber.
Incorrect parameter listing when creating SendFailedNotification from
native.
6) SctpServerChannel.getAllLocalAddresses returns null for unbound
channel.
Now returns empty set as specified.
7) getRemoteAddresses throws "java.net.SocketException: Invalid
argument" for shutdown association.
Fixed.
8) Add some new testcases as well as update existing ones to cover some
of the above changes.
Thanks,
-Chris.
More information about the net-dev
mailing list