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 04 Jul '14

04 Jul '14

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 | … [View More]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; [View Less]

1 0

[tracker] Task closed: Update FSF address
by sip-router 04 Jul '14

04 Jul '14

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 … [View More]

1 0

git:master: all: updated FSF address in GPL text
by Daniel-Constantin Mierla 04 Jul '14

04 Jul '14

1 0

[tracker] Task opened: usrloc driver error on query: Duplicate entry 'XXXXXXXXXXXX' for key 'ruid_idx'
by sip-router 04 Jul '14

04 Jul '14

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 - … [View More]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. [View Less]

1 0

[tracker] Task opened: Update FSF address (Attachment added)
by sip-router 04 Jul '14

04 Jul '14

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 … [View More]

1 0

Using t_suspend()/t_continue() multiple times on the same transaction
by Peter Dunkley 02 Jul '14

02 Jul '14

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()… [View More]

5 12

git:master: rtpengine: add support for TOS=... flag
by Richard Fuchs 02 Jul '14

02 Jul '14

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 … [View More]

1 0

rtpengine crashes
by Alex Balashov 02 Jul '14

02 Jul '14

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 … [View More]

3 22

git:master: tm: run reply callbacks under lock
by Daniel-Constantin Mierla 01 Jul '14

01 Jul '14

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) --- … [View More]

1 0

Kamailio Development Workshop, July 9-10, 2014, Paris, France
by Daniel-Constantin Mierla 01 Jul '14

01 Jul '14

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 … [View More]

1 2

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev July 2014