July 2010 - sr-dev - kamailio.org

git:master: modules/tm: new implementation of load_contacts()/next_contacts ()
by Juha Heinanen 08 Aug '11

08 Aug '11

Module: sip-router Branch: master Commit: e973bbe5e7310861f77b17ce0afaf1cca35fe48a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e973bbe… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Fri May 28 15:38:41 2010 +0300 modules/tm: new implementation of load_contacts()/next_contacts() - Simpler implementation of load_contacts()/next_contacts() functions based on recent changes related to Request-URI handling. - Function next_contacts() does not anymore set any timers. Check contact_avp and call t_set_fr() before calling t_relay() instead. - Removed fr_timer_next module parameter, because it is not needed anymore. --- modules/tm/README | 255 ++++++++++++++------------------------ modules/tm/doc/functions.xml | 51 +++----- modules/tm/doc/params.xml | 73 ----------- modules/tm/doc/tm.xml | 15 +-- modules/tm/t_serial.c | 279 ++++++++++++------------------------------ modules/tm/tm.c | 1 - 6 files changed, 190 insertions(+), 484 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=e97…

2 16

preparing kamailio 3.0.3
by Daniel-Constantin Mierla 19 Aug '10

19 Aug '10

Hello, I am planning to release minor version 3.0.3 by beginning of next week. Please commit there what you think it needs to be fixed for this release. Thanks, Daniel -- Daniel-Constantin Mierla http://www.asipto.com/

2 4

Makefile bug fixes and Dragonfly BSD port
by Vincent Stemen 13 Aug '10

13 Aug '10

Hi. I have ported kamailio to Dragonfly BSD and fixed a couple of bugs in the build system in the process. I have 4 patches to the Makefiles which I created for kamailio-3.0.1, but have verified will apply to the current master branch. Two are bug fixes and the other 2 are to get it to compile and run on Dragonfly. I have attached all 4 patches to this email. There are comments in the top of each of the patches about what they do. All patches can be applied from the root of the source tree. It turns out that kamailio is not going to serve our needs at this time, so I don't have much more time to put into it. However, I am in hopes that the developers will utilize the patches for Dragonfly as well. I would like to see kamailio natively support Dragonfly BSD for the benefit of both communities and in case we ever need it ourselves in the future, since our environment is all Dragonfly BSD based. The patches primarily just make kamailio treat Dragonfly the same as FreeBSD. kamailio-3.0.1 compiled, installed, and ran successfully with these patches. However, after applying them to todays master branch from the git repository, make was successful, but "make install" errored with install -m 755 msilo.so /u1/falcon/ports/kamailio-20100724/package/kamailio-20100724/kamailio-20100724/usr/pkg/lib/ser/modules_s CC (gcc) [M nathelper.so] nathelper.o CC (gcc) [M nathelper.so] natping.o CC (gcc) [M nathelper.so] nhelpr_funcs.o CC (gcc) [M nathelper.so] rtpproxy_stream.o rtpproxy_stream.c: In function 'rtpproxy_stream': rtpproxy_stream.c:61: error: array type has incomplete element type rtpproxy_stream.c:61: warning: unused variable 'v' rtpproxy_stream.c: In function 'rtpproxy_stop_stream': rtpproxy_stream.c:151: error: array type has incomplete element type rtpproxy_stream.c:151: warning: unused variable 'v' gmake[1]: *** [rtpproxy_stream.o] Error 1 gmake: *** [install-modules_s] Error 1 I don't know if it is Dragonfly related or if it is just a temporary bug introduced in the current master branch. Additional notes: To successfully compile on Dragonfly BSD after applying the patches to kamailio-3.0.1 these are the commands I used. # export CFLAGS=-D__FreeBSD__ # gmake prefix=/usr/pkg # gmake prefix=/usr/pkg install Regards, Vince

2 3

3.0.2 Makefile issues continued - FreeBSD 6
by Olle E. Johansson 11 Aug '10

11 Aug '10

Installing on Linux went like a charm, no issues. However, there are some issues when building on FreeBSD. Some comments: - The xmlrpc module links to -lresolv, something that doesn't exist on FreeBSD and can be removed. - Libraries in /usr/local/lib was not found, but it seems like include files in /usr/local/include was found by some modules, but not all. Setting CPATH solved issues here. - There's no point in even trying to compile iptrtpproxy on systems that are not Linux, right? - There are many compiler warnings that should be looked at - In order to get INADDR_LOOPBACK, that is used in presence_dialoginfo and other modules I ended up copying the definition of INADDR_LOOPBACK into ip_addr.h... It's in netinet/in.h but must be disabled for some reason. Also getting this at make install (but not make modules), guessing it belongs to kamctl installation: MYSQLON=yes make -C ../../utils/kamctl/ install-modules "../../Makefile.defs", line 88: Missing dependency operator Error expanding embedded variable. gmake[1]: *** [install-mysql-scripts] Error 2 gmake[1]: Leaving directory `/s/usr-local/src/kamailio-3.0.2/modules/db_mysql' gmake: *** [install-modules] Error 1 I had to exclude all db modules in order to get around this. I guess someone with some more experience of the build system needs to look at this :-) Cheers, /O

2 1

Re: [sr-dev] pua_dialoginfo deletes AVPs?
by Jasmin Schnatterbeck 08 Aug '10

08 Aug '10

Hi Henning, because of the fact, that the problem only appears, if the pua_dialoginfo module is enabled, I think the problem is located somewhere in the code tm hooks, dialog hooks or pua/pua_dialoginfo itself... I have attached the debug output of pua_dialoginfo which shows only, if the module is enabled... may it is useful in any way... Regards Jasmin > On Friday 04 June 2010, Jasmin Schnatterbeck wrote: > > onreply_avp_mode does not seem to be available in kamailio 3.0.2 (at > > least not in the module doc) > > I think this parameter is for avps in reply_route? or does it work for > > branch_route also? > > Hi Jasmin, > > ah, sorry. Its not available or necessary anymore in 3.0. In 1.5 it also > affected the branch_route, i think, but for the newer version this behaviour > is now the default. Then in your case its probably another problem.. > > Cheers, > > Henning

1 1

freezing for major release 3.1
by Daniel-Constantin Mierla 03 Aug '10

03 Aug '10

Hello, as discussed during our devel meeting, next major release 3.1 is scheduled for this autumn. Therefore the freezing should happen soon. Would be 2 weeks from now enough for everyone to push planned new features to repo? Cheers, Daniel -- Daniel-Constantin Mierla http://www.asipto.com/

6 9

INumWare: inum users conference
by Meftah Tayeb 02 Aug '10

02 Aug '10

hello folk you're invited to join the inum ware conference this conference is for inum users that frequantly use inum (+883 country code) (http://www.inum.net) you can join it by dialing +883510001288888 from any provider that provide INum termination (see http://www.inum.net) thank you! -- Meftah Tayeb algérie télécom SPA phone: +21321761805 phone (INUM): +883510001289101 mobile : +213660347746 mobile (INUM: +883510001289110 http://www.algerietelecom.dz

4 4

git:master: presence_dialoginfo(k): don't exeed array size in error case
by Daniel-Constantin Mierla 30 Jul '10

30 Jul '10

Module: sip-router Branch: master Commit: 48f3159cdcc9ef06114a6d8f853687dc1338a8f0 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=48f3159… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Jul 30 23:29:43 2010 +0200 presence_dialoginfo(k): don't exeed array size in error case - error handling could get to the size of array as index, resulting in out of bounds operation - patch by Marius-Ovidiu Bucur --- modules_k/presence_dialoginfo/notify_body.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules_k/presence_dialoginfo/notify_body.c b/modules_k/presence_dialoginfo/notify_body.c index 965cef0..efa3b45 100644 --- a/modules_k/presence_dialoginfo/notify_body.c +++ b/modules_k/presence_dialoginfo/notify_body.c @@ -264,7 +264,7 @@ str* agregate_xmls(str* pres_user, str* pres_domain, str** body_array, int n) error: if(xml_array!=NULL) { - for(i=0; i<=j; i++) + for(i=0; i<j; i++) { if(xml_array[i]!=NULL) xmlFreeDoc( xml_array[i]);

1 0

git:master: presence(k): set properly db updated flag
by Daniel-Constantin Mierla 30 Jul '10

30 Jul '10

Module: sip-router Branch: master Commit: b31c53abf1c3bafafb59d3fa6456135f1da6e6c9 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b31c53a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Jul 30 23:26:35 2010 +0200 presence(k): set properly db updated flag - patch by Marius-Ovidiu Bucur --- modules_k/presence/hash.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules_k/presence/hash.c b/modules_k/presence/hash.c index 874fe03..bc31b87 100644 --- a/modules_k/presence/hash.c +++ b/modules_k/presence/hash.c @@ -261,7 +261,7 @@ int insert_shtable(shtable_t htable,unsigned int hash_code, subs_t* subs) } new_rec->expires+= (int)time(NULL); - if(fallback2db==0) + if(fallback2db!=0) new_rec->db_flag= INSERTDB_FLAG; else new_rec->db_flag= NO_UPDATEDB_FLAG;

1 0

git:master: pua(k): use the outbound proxy on updates
by Daniel-Constantin Mierla 30 Jul '10

30 Jul '10

Module: sip-router Branch: master Commit: f39cb1976978f0860d41a05ed181e45e14c88c58 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f39cb19… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Jul 30 20:25:52 2010 +0200 pua(k): use the outbound proxy on updates - patch by Alex Hermann, SF#3035780 --- modules_k/pua/pua.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules_k/pua/pua.c b/modules_k/pua/pua.c index 3706200..bf73187 100644 --- a/modules_k/pua/pua.c +++ b/modules_k/pua/pua.c @@ -667,7 +667,7 @@ int update_pua(ua_pres_t* p, unsigned int hash_code) p->pres_uri, /* Request-URI */ p->pres_uri, /* To */ p->pres_uri, /* From */ - 0 /* Outbound proxy*/ + &outbound_proxy /* Outbound proxy*/ ); if(result< 0) {

1 0