Hello Glenn,

thanks for your contribution, please register your patch on the tracker. It will be reviewed and included in openser
http://sourceforge.net/tracker/?group_id=139143

See more comments inline.

On 07/19/07 22:25, Glenn Marmon wrote:
(Apologies if this is a duplicate message of one sent earlier, but after three hours it never showed up on the list, so I'm trying again from another email address)

Over the past weeks, I have been working on adding SCTP transport support for OpenSER 1.2.1. I've made a tarball with the necessary files, a README on using it, and a .diff showing the changes I've had to make to existing OpenSER core files:

http://oak.conncoll.edu/~gmarmon/openser-1.2.1-sctp-addon.tgz <http://oak.conncoll.edu/%7Egmarmon/openser-1.2.1-sctp-addon.tgz>

This code has been built and tested on a FreeBSD 6.2 system with the latest SCTP stack compiled into the kernel. I will soon be testing it on FreeBSD 7.0, which is slated for release this fall and supports SCTP natively.

A couple of caveats:

1. My testing of this implementation has been limited to the following scenario:
sipp (A) <--> openser (A) <----> openser (B) <--> sipp (B)
Where both openser A and B are version 1.2.1 with the SCTP add-on, and both sipp A and B are the nightly SipP build from July 12, 2007 (sipp.2007-07-12). The scenario is a simple one where sipp A runs the default client (uac), making calls to sipp B (uas). The SCTP communication can be seen in the message forwarding between the two openser servers. This scenario works for me, but it is the only one I have tested thus far.
Could you please check if the patch works for devel version (trunk) as well? Would be easier to review and integrate.

2. There is one major issue I can see at the moment: OpenSER with this add-on will crash whenever it is stopped. It only happens when the program quits, so its been more of just a nuisance: OpenSER is functional during its lifetime, it just leaves a large .core and the .pid file when it quits. Obviously, though, this is a serious problem, and I haven't yet tracked it down. Notes on it can be found in the readme of the openser-sctp package.
Have you investigated the core? What the backtrace says?

3. Obviously, changes had to be made to the OpenSER core code to support SCTP. I feel that most of them are clean, but there are a couple places where I'm less confident about whether or not the edit/fix I've made is the best way to do it. Specific notes on those can be found in the readme of the openser-sctp package.

I feel that SCTP functionality would be a valuable addition to OpenSER, and I will continue to work on this implementation. Any feedback would be greatly appreciated.
Indeed, including the SCPT will complete the transport layers in openser. You will get feedback from the reviewer and then a decision will be made for the maintenance of that part of the code.

Cheers,
Daniel

-Glenn Marmon
Connecticut College Class of 2009
[EMAIL PROTECTED] <mailto:[EMAIL PROTECTED]>
------------------------------------------------------------------------

_______________________________________________
Devel mailing list
[email protected]
http://openser.org/cgi-bin/mailman/listinfo/devel

_______________________________________________
Devel mailing list
[email protected]
http://openser.org/cgi-bin/mailman/listinfo/devel

Reply via email to