Request for Review 6855335

Michael McMahon Michael.McMahon at Sun.COM
Mon Jun 29 03:32:37 PDT 2009


Christopher Hegarty - Sun Microsystems Ireland wrote:
> 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.
>
SctpChannelImpl.java: Do you need to establish any state other than 
remoteAddresses?
What about localAddresses for instance?

- Michael.




More information about the net-dev mailing list