sr-dev July 2014

sr-dev@lists.kamailio.org

25 participants
140 discussions

git:master: usrloc: generate callid at startup and increase cseq for records added via mi

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 66c497fdf4ac1c3b889a7c3b50c3e5fed770cf0b URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=66c497f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Jul 4 13:46:35 2014 +0200 usrloc: generate callid at startup and increase cseq for records added via mi - should fix FS#448, reported by Savolainen Dmitri --- modules/usrloc/ul_mi.c | 27 ++++++++++++++++++++++++--- 1 files changed, 24 insertions(+), 3 deletions(-) diff --git a/modules/usrloc/ul_mi.c b/modules/usrloc/ul_mi.c index f36117c..6ec82a5 100644 --- a/modules/usrloc/ul_mi.c +++ b/modules/usrloc/ul_mi.c @@ -48,9 +48,9 @@ #include "usrloc.h" /*! CSEQ nr used */ -#define MI_UL_CSEQ 1 +static int MI_UL_CSEQ = 0; /*! call-id used for ul_add and ul_rm_contact */ -static str mi_ul_cid = str_init("dfjrewr12386fd6-343(a)kamailio.mi"); +static str mi_ul_cid = {0,0}; /* str_init("dfjrewr12386fd6-343(a)kamailio.mi");*/ /*! user agent used for ul_add */ static str mi_ul_ua = str_init("SIP Router MI Server"); /*! path used for ul_add and ul_rm_contact */ @@ -58,8 +58,27 @@ static str mi_ul_path = str_init("dummypath"); extern sruid_t _ul_sruid; +static char mi_ul_cid_buf[32]; + /************************ helper functions ****************************/ +static void set_mi_ul_cid(void) +{ + int i; + char charset[] = "0123456789" + "abcdefghijklmnopqrstuvwxyz" + "ABCDEFGHIJKLMNOPQRSTUVWXYZ"; + + if(mi_ul_cid.s!=NULL) return; + + for(i=0; i<19; i++) { + mi_ul_cid_buf[i] = charset[rand()%(sizeof(charset) - 1)]; + } + memcpy(mi_ul_cid_buf+i, "@kamailio.mi", sizeof("@kamailio.mi")); + mi_ul_cid.s = mi_ul_cid_buf; + mi_ul_cid.len = strlen(mi_ul_cid.s); +} + /*! * \brief Search a domain in the global domain list * \param table domain (table) name @@ -360,6 +379,7 @@ struct mi_root* mi_usrloc_rm_contact(struct mi_root *cmd, void *param) } contact = &node->next->next->value; + set_mi_ul_cid(); ret = get_ucontact( rec, contact, &mi_ul_cid, &mi_ul_path, MI_UL_CSEQ+1, &con); if (ret < 0) { unlock_udomain( dom, aor); @@ -566,6 +586,7 @@ struct mi_root* mi_usrloc_add(struct mi_root *cmd, void *param) lock_udomain( dom, aor); + set_mi_ul_cid(); n = get_urecord( dom, aor, &r); if ( n==1) { if (insert_urecord( dom, aor, &r) < 0) @@ -580,7 +601,7 @@ struct mi_root* mi_usrloc_add(struct mi_root *cmd, void *param) ci.callid = &mi_ul_cid; ci.user_agent = &mi_ul_ua; - ci.cseq = MI_UL_CSEQ; + ci.cseq = ++MI_UL_CSEQ; /* 0 expires means permanent contact */ if (ci.expires!=0) ci.expires += act_time;

9 years, 9 months

[tracker] Task closed: Update FSF address

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#447 - Update FSF address User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Implemented Additional comments about closing: Thanks, patch pushed to master branch. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=447 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

9 years, 9 months

git:master: all: updated FSF address in GPL text

by Daniel-Constantin Mierla

9 years, 9 months

[tracker] Task opened: usrloc driver error on query: Duplicate entry 'XXXXXXXXXXXX' for key 'ruid_idx'

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Savolainen Dmitri (snen) Attached to Project - sip-router Summary - usrloc driver error on query: Duplicate entry 'XXXXXXXXXXXX' for key 'ruid_idx' Task Type - Bug Report Category - usrloc (k) Status - Unconfirmed Assigned To - Operating System - Linux Severity - Medium Priority - Normal Reported Version - 4.1 Due in Version - Undecided Due Date - Undecided Details - getting error ERROR: db_mysql [km_dbase.c:122]: db_mysql_submit_query(): driver error on query: Duplicate entry 'XXXXXXXXXXXX' for key 'ruid_idx' Getting error expample usloc params: <code> modparam('usrloc', 'db_mode', 2) modparam("usrloc", "db_check_update", 1) modparam("usrloc", "db_ops_ruid", 1) modparam("usrloc", "timer_interval", 120) #default 60 modparam('usrloc', 'db_url', 'mysql:XXXXXXXXXXXXXXXXX') </code> now let's manipulate with user 1111 1. 1111 doesn't exist <code> mysql>select * from location where username='1111' \G Empty set (0.00 sec) </code> 2. add permanent location for 1111 ./kamctl ul add '1111(a)erinaco.ru' 'sip:1111@192.168.10.100' 3. 1111 is in DB after some seconds (up to "timer_interval") <code> mysql>select * from location where username='1111' \G *************************** 1. row *************************** id: 111625 ruid: ulcx-53b55b8f-46d1-1 username: 1111 domain: erinaco.ru contact: sip:1111@192.168.10.100 received: NULL path: NULL expires: 1970-01-01 03:00:00 q: 1.00 callid: dfjrewr12386fd6-343(a)kamailio.mi cseq: 1 last_modified: 1970-01-01 03:00:00 flags: 0 cflags: 0 user_agent: SIP Router MI Server socket: NULL methods: NULL instance: NULL reg_id: 0 </code> 4. let's execute this again ./kamctl ul add '1111(a)erinaco.ru' 'sip:1111@192.168.10.100' There is no any changes in db after "timer_interval" <code> mysql>select * from location where username='1111' \G *************************** 1. row *************************** id: 111626 ruid: ulcx-53b55f26-4895-1 username: 1111 domain: erinaco.ru contact: sip:1111@192.168.10.100 received: NULL path: NULL expires: 1970-01-01 03:00:00 q: 1.00 callid: dfjrewr12386fd6-343(a)kamailio.mi cseq: 1 last_modified: 1970-01-01 03:00:00 flags: 0 cflags: 0 user_agent: SIP Router MI Server socket: NULL methods: NULL instance: NULL reg_id: 0 1 row in set (0.00 sec) </code> Now, every "timer_interval" seconds in kamailio.log <code> [snen@sw4 sbin]# grep 'ulcx-53b55f26-4895-1' /var/log/kamailio.log Jul 3 17:52:22 sw4 ./kamailio[18576]: ERROR: db_mysql [km_dbase.c:122]: db_mysql_submit_query(): driver error on query: Duplicate entry 'ulcx-53b55f26-4895-1' for key 'ruid_idx' Jul 3 17:54:22 sw4 ./kamailio[18576]: ERROR: db_mysql [km_dbase.c:122]: db_mysql_submit_query(): driver error on query: Duplicate entry 'ulcx-53b55f26-4895-1' for key 'ruid_idx' Jul 3 17:56:22 sw4 ./kamailio[18576]: ERROR: db_mysql [km_dbase.c:122]: db_mysql_submit_query(): driver error on query: Duplicate entry 'ulcx-53b55f26-4895-1' for key 'ruid_idx' Jul 3 17:58:22 sw4 ./kamailio[18576]: ERROR: db_mysql [km_dbase.c:122]: db_mysql_submit_query(): driver error on query: Duplicate entry 'ulcx-53b55f26-4895-1' for key 'ruid_idx' </code> The reason is "modparam("usrloc", "db_check_update", 1)": affected_rows after UPDATE returns 0, and INSERT returns permanent error. Record was marked as CS_DIRTY and will be never CS_SYNC up to kamailio reboot or registration change(that may be never for permanent registrations) so we can see increasing the number of such records in the work process for some usage configuration. some fixes: * set last_modified to NOW() while updating (bad for simultaneous updates) * "INSERT IGNORE" as usrloc module param * do not perform DB changes, if there is no "in memory" changes (may be bad in external DB update case) * mark record as CS_SYNC after affected_rows 0 and INSERT error, so error will be only once in this patch INSERT error ignored after UPDATE and affected_rows 0 <code> diff --git a/modules/usrloc/ucontact.c b/modules/usrloc/ucontact.c index b18ce98..ad71b34 100644 --- a/modules/usrloc/ucontact.c +++ b/modules/usrloc/ucontact.c @@ -820,9 +820,8 @@ int db_update_ucontact_addr(ucontact_t* _c) /* supposed to be an UPDATE, but if affected rows is 0, then try * to do an INSERT */ if(ul_dbf.affected_rows(ul_dbh)==0) { - LM_DBG("affected rows by UPDATE was 0, doing an INSERT\n"); - if(db_insert_ucontact(_c)<0) - return -1; + LM_DBG("affected rows by UPDATE was 0, doing an INSERT without error check\n"); + db_insert_ucontact(_c); } } /* delete old db attrs and add the current list */ @@ -1006,9 +1005,8 @@ int db_update_ucontact_ruid(ucontact_t* _c) /* supposed to be an UPDATE, but if affected rows is 0, then try * to do an INSERT */ if(ul_dbf.affected_rows(ul_dbh)==0) { - LM_DBG("affected rows by UPDATE was 0, doing an INSERT\n"); - if(db_insert_ucontact(_c)<0) - return -1; + LM_DBG("affected rows by UPDATE was 0, doing an INSERT without error check\n"); + db_insert_ucontact(_c); } } @@ -1226,9 +1224,8 @@ int db_update_ucontact_instance(ucontact_t* _c) /* supposed to be an UPDATE, but if affected rows is 0, then try * to do an INSERT */ if(ul_dbf.affected_rows(ul_dbh)==0) { - LM_DBG("affected rows by UPDATE was 0, doing an INSERT\n"); - if(db_insert_ucontact(_c)<0) - return -1; + LM_DBG("affected rows by UPDATE was 0, doing an INSERT without error check\n"); + db_insert_ucontact(_c); } } </code> More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=448 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

9 years, 9 months

[tracker] Task opened: Update FSF address (Attachment added)

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Anthony Messina (amessina) Attached to Project - sip-router Summary - Update FSF address Task Type - Bug Report Category - Core Status - Unconfirmed Assigned To - Operating System - All Severity - Very Low Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - The attached patch updates the FSF address and applies against the git master at 639fce79466baf77bbd707eb9cef6473dcb3eb7c One or more files have been attached. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=447 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

9 years, 9 months

Using t_suspend()/t_continue() multiple times on the same transaction

by Peter Dunkley

Hi, I am trying to use t_suspend()/t_continue() multiple times on the same transaction. Calling t_suspend() more than once works, but the second time I call t_continue() the transaction is killed and a 500 response is sent. It is the "if (branch == t->nr_of_outgoings)" check from the code fragment below (from t_suspend.c:t_continue()) that results in the transaction being killed - you can see the debug/error line I added to determine this in the fragment. Is using t_suspend()/t_continue() multiple times something that should work? Thanks, Peter if (t->uas.status < 200) { /* No final reply has been sent yet. * Check whether or not there is any pending branch. */ for ( branch = 0; branch < t->nr_of_outgoings; branch++ ) { if ((t->uac[branch].request.buffer != NULL) && (t->uac[branch].last_received < 200) ) break; } if (branch == t->nr_of_outgoings) { /* There is not any open branch so there is * no chance that a final response will be received. */ ret = 0; LM_ERR("branch == t->nr_of_outgoings\n"); goto kill_trans; } } -- Peter Dunkley Technical Director Crocodile RCS Ltd

9 years, 9 months

git:master: rtpengine: add support for TOS=... flag

by Richard Fuchs

Module: sip-router Branch: master Commit: 639fce79466baf77bbd707eb9cef6473dcb3eb7c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=639fce7… Author: Richard Fuchs <rfuchs(a)sipwise.com> Committer: Richard Fuchs <rfuchs(a)sipwise.com> Date: Wed Jul 2 14:53:36 2014 -0400 rtpengine: add support for TOS=... flag --- modules/rtpengine/doc/rtpengine_admin.xml | 8 ++++++++ modules/rtpengine/rtpengine.c | 2 ++ 2 files changed, 10 insertions(+), 0 deletions(-) diff --git a/modules/rtpengine/doc/rtpengine_admin.xml b/modules/rtpengine/doc/rtpengine_admin.xml index b9f62d4..7c35611 100644 --- a/modules/rtpengine/doc/rtpengine_admin.xml +++ b/modules/rtpengine/doc/rtpengine_admin.xml @@ -469,6 +469,14 @@ rtpengine_offer(); <emphasis>media-address=...</emphasis> - force a particular media address to be used in the &sdp; body. Address family is detected automatically. </para></listitem> + <listitem><para> + <emphasis>TOS=...</emphasis> - change the IP TOS value for all outgoing RTP + packets within the entire call in both directions. Only honoured in an + <quote>offer</quote>, ignored for an <quote>answer</quote>. Valid values are + 0 through 255, given in decimal. If this option is not specified, the TOS + value will revert to the default TOS (normally 184). A value of -1 may be used + to leave the currently used TOS unchanged. + </para></listitem> </itemizedlist> </listitem> </itemizedlist> diff --git a/modules/rtpengine/rtpengine.c b/modules/rtpengine/rtpengine.c index 334688b..ce2bc78 100644 --- a/modules/rtpengine/rtpengine.c +++ b/modules/rtpengine/rtpengine.c @@ -1188,6 +1188,8 @@ static int parse_flags(struct ng_flags_parse *ng_flags, struct sip_msg *msg, enu ng_flags->transport |= 0x100; ng_flags->transport &= ~0x002; } + else if (str_eq(&key, "TOS") && val.s) + bencode_dictionary_add_integer(ng_flags->dict, "TOS", atoi(val.s)); else goto generic; goto next;

9 years, 9 months

rtpengine crashes

by Alex Balashov

I'm using the latest rtpengine master and still getting crashes once every 1-2 days on a high call volume installation: (gdb) where #0 0x000000304e232925 in raise () from /lib64/libc.so.6 #1 0x000000304e234105 in abort () from /lib64/libc.so.6 #2 0x000000304e22ba4e in __assert_fail_base () from /lib64/libc.so.6 #3 0x000000304e22bb10 in __assert_fail () from /lib64/libc.so.6 #4 0x0000000000412d50 in stream_packet (fd=<value optimized out>, p= 0x7f1dcc1b1030, u=<value optimized out>) at call.c:547 #5 stream_fd_readable (fd=<value optimized out>, p=0x7f1dcc1b1030, u=<value optimized out>) at call.c:819 #6 0x000000000040b4ce in poller_poll (p=0x10d3750, timeout=<value optimized out>) at poller.c:354 #7 0x000000000040722d in poller_loop (d=0x10d3750) at main.c:542 #8 0x000000000040bb5f in thread_detach_func (d=<value optimized out>) at aux.c:160 #9 0x000000304e6079d1 in start_thread () from /lib64/libpthread.so.0 #10 0x000000304e2e8b6d in clone () from /lib64/libc.so.6 Unfortunately, this packet capture is not very insightful. What's the easiest way to build rtpengine with debug symbols so that some of these values can be filled in? -- Alex Balashov - Principal Evariste Systems LLC Tel: +1-678-954-0670 Web: http://www.evaristesys.com/, http://www.alexbalashov.com/ Please be kind to the English language: http://www.entrepreneur.com/article/232906

9 years, 9 months

git:master: tm: run reply callbacks under lock

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: da9d56be28e050dd0cb4aed50efcbda043a3e5cf URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=da9d56b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jul 1 16:10:20 2014 +0200 tm: run reply callbacks under lock - callback functions can modify the request from transaction (e.g., acc for parsing headers needed for accounting) --- modules/tm/t_reply.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/modules/tm/t_reply.c b/modules/tm/t_reply.c index 8e435cb..fb217cb 100644 --- a/modules/tm/t_reply.c +++ b/modules/tm/t_reply.c @@ -1998,8 +1998,10 @@ enum rps relay_reply( struct cell *t, struct sip_msg *p_msg, int branch, if (likely(uas_rb->dst.send_sock && SEND_PR_BUFFER( uas_rb, buf, res_len ) >= 0)){ if (unlikely(!totag_retr && has_tran_tmcbs(t, TMCB_RESPONSE_OUT))){ + LOCK_REPLIES( t ); run_trans_callbacks_with_buf( TMCB_RESPONSE_OUT, uas_rb, t->uas.request, relayed_msg, relayed_code); + UNLOCK_REPLIES( t ); } if (unlikely(has_tran_tmcbs(t, TMCB_RESPONSE_SENT))){ INIT_TMCB_ONSEND_PARAMS(onsend_params, t->uas.request, @@ -2007,7 +2009,9 @@ enum rps relay_reply( struct cell *t, struct sip_msg *p_msg, int branch, res_len, (relayed_msg==FAKED_REPLY)?TMCB_LOCAL_F:0, uas_rb->branch, relayed_code); + LOCK_REPLIES( t ); run_trans_callbacks_off_params(TMCB_RESPONSE_SENT, t, &onsend_params); + UNLOCK_REPLIES( t ); } } else if (unlikely(uas_rb->dst.send_sock == 0)) ERR("no resolved dst to send reply to\n");

9 years, 9 months

Kamailio Development Workshop, July 9-10, 2014, Paris, France

by Daniel-Constantin Mierla

Hello, I want to announce the next Kamailio Development Workshop, to take place in Paris, France, during July 9-10, 2014. This event is the next in its series targeting to show the internals of Kamailio and enable more people to become developers as well as let users of Kamailio to get more knowledge about the design of the application which can have relevant impact in operating deployment. Be aware that it is not a workshop about Kamailio installation and administration. Its content is about writing C code to extend Kamailio. More details can be found at: - http://www.kamailio.org/w/2014/06/kamailio-development-workshop-july-9-10-2… Similar to the past events, we are considering to have a social networking event in the evening of July 9 (most probably a dinner meetup at a nice place in Paris), which is open for everyone, each participant taking care of own expenses. Cheers, Daniel -- Daniel-Constantin Mierla - http://www.asipto.com http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda

9 years, 9 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev July 2014