sr-dev November 2013

sr-dev@lists.kamailio.org

29 participants
220 discussions

git:master: mohqueue: remove sql table call_time column's default value
by Timo Teras 08 Nov '13

08 Nov '13

Module: sip-router Branch: master Commit: 456c387e08ce40ea2c0f1e5e5f94b367ff9fa361 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=456c387… Author: Robert Boisvert <rdboisvert(a)gmail.com> Committer: Timo Teräs <timo.teras(a)iki.fi> Date: Tue Oct 29 18:10:25 2013 -0400 mohqueue: remove sql table call_time column's default value it is not supported in some database properly, and it is not really needed. --- lib/srdb1/schema/mohqcalls.xml | 1 - … [View More]

1 0

sca module warnings
by Daniel-Constantin Mierla 07 Nov '13

07 Nov '13

Hello, compiling sca module rises next warning: sca_call_info.c: In function ‘sca_call_info_update’: sca_call_info.c:1977:7: warning: the comparison will always evaluate as ‘true’ for the address of ‘call_info’ will never be NULL [-Waddress] sca_call_info.c:1984:14: warning: the comparison will always evaluate as ‘true’ for the address of ‘call_info’ will never be NULL [-Waddress] The lead to the next define, which does not seem right at first sight #define SCA_CALL_INFO_EMPTY( ci1 ) \ ((… [View More]

2 1

git:master: modules/sca: restore correct check for NULL in SCA_CALL_INFO_EMPTY.
by Andrew Mortensen 07 Nov '13

07 Nov '13

Module: sip-router Branch: master Commit: 04b457735f0bafa6ff214cb2d69afbe721d89bf1 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=04b4577… Author: Andrew Mortensen <admorten(a)isc.upenn.edu> Committer: Andrew Mortensen <admorten(a)isc.upenn.edu> Date: Thu Nov 7 14:52:29 2013 -0500 modules/sca: restore correct check for NULL in SCA_CALL_INFO_EMPTY. - should also suppress -Waddress error without relying on gcc _Pragma. --- modules/sca/… [View More]

1 0

[tracker] Task opened: modules/app_lua: sr.xavp.get() allow get all values of a name (Attachment added)
by sip-router 07 Nov '13

07 Nov '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Víctor Seva (linuxmaniac) Attached to Project - sip-router Summary - modules/app_lua: sr.xavp.get() allow get all values of a name Task Type - Improvement Category - Module Status - New Assigned To - Víctor Seva Operating System - All Severity - Low Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - Use value indx&… [View More]

1 0

[tracker] Comment added: Patch to Add s.urlencode.param
by sip-router 07 Nov '13

07 Nov '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#311 - Patch to Add s.urlencode.param User who did this - Josh (JoshE) ---------- Whoops. I picked this up accidentally when I generated the patch. That is for a whole different issue. I modified dispatch to return the attribute value to the column, which I use to store a value I want to use in my routing process. It's a simple patch. If anyone else wants t be able to use AVP_DSATTRS after a … [View More]

1 0

[tracker] Task opened: Crash in TCP Read on Kamailio 4.0.1 (Attachment added)
by sip-router 07 Nov '13

07 Nov '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Josh (JoshE) Attached to Project - sip-router Summary - Crash in TCP Read on Kamailio 4.0.1 Task Type - Bug Report Category - Core Status - Unconfirmed Assigned To - Operating System - All Severity - Critical Priority - Normal Reported Version - 4.0 Due in Version - Undecided Due Date - Undecided Details - I am seeing a crash in TCP Read using the HTTP module. Here's … [View More]

1 0

git:tmp/dialog_ng: tmp/dialog_ng: fixed bug in setting proper null flags for db storage
by Carlos Ruiz Diaz 06 Nov '13

06 Nov '13

Module: sip-router Branch: tmp/dialog_ng Commit: 93b43dbd267163e683ac3ee974d0059a4f83d859 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=93b43db… Author: Carlos Ruiz Diaz <carlos.ruizdiaz(a)gmail.com> Committer: Carlos Ruiz Diaz <carlos.ruizdiaz(a)gmail.com> Date: Wed Nov 6 13:06:22 2013 -0300 tmp/dialog_ng: fixed bug in setting proper null flags for db storage --- modules/dialog_ng/dlg_db_handler.c | 36 +++++++++++++++++++----------------- 1 … [View More]files changed, 19 insertions(+), 17 deletions(-) diff --git a/modules/dialog_ng/dlg_db_handler.c b/modules/dialog_ng/dlg_db_handler.c index 9fa06bf..bdc06ac 100644 --- a/modules/dialog_ng/dlg_db_handler.c +++ b/modules/dialog_ng/dlg_db_handler.c @@ -1020,25 +1020,25 @@ int update_dialog_dbinfo_unsafe(struct dlg_cell * cell) &sflags_column, &to_route_name_column, &to_route_index_column }; /* save all the current dialogs information*/ - VAL_TYPE(GET_FIELD_IDX(values, DLGI_ID_COL_IDX)) = DB1_INT; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_ID_COL_IDX)) = DB1_INT; VAL_TYPE(GET_FIELD_IDX(values, DLGI_HASH_ENTRY_COL_IDX))= DB1_INT; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_HASH_ID_COL_IDX)) = DB1_INT; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_HASH_ID_COL_IDX)) = DB1_INT; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_STATE_COL_IDX)) = DB1_INT; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_START_TIME_COL_IDX)) = DB1_INT; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_TIMEOUT_COL_IDX)) = DB1_INT; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_SFLAGS_COL_IDX)) = DB1_INT; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_STATE_COL_IDX)) = DB1_INT; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_START_TIME_COL_IDX)) = DB1_INT; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_TIMEOUT_COL_IDX)) = DB1_INT; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_SFLAGS_COL_IDX)) = DB1_INT; VAL_TYPE(GET_FIELD_IDX(values, DLGI_TOROUTE_INDEX_COL_IDX)) = DB1_INT; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_DID_COL_IDX)) = DB1_STR; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_CALLID_COL_IDX)) = DB1_STR; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_FROM_URI_COL_IDX)) = DB1_STR; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_FROM_TAG_COL_IDX)) = DB1_STR; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_CALLER_CSEQ_COL_IDX)) = DB1_STR; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_REQ_URI_COL_IDX)) = DB1_STR; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_DID_COL_IDX)) = DB1_STR; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_CALLID_COL_IDX)) = DB1_STR; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_FROM_URI_COL_IDX)) = DB1_STR; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_FROM_TAG_COL_IDX)) = DB1_STR; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_CALLER_CSEQ_COL_IDX)) = DB1_STR; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_REQ_URI_COL_IDX)) = DB1_STR; VAL_TYPE(GET_FIELD_IDX(values, DLGI_CALLER_ROUTESET_COL_IDX))= DB1_STR; VAL_TYPE(GET_FIELD_IDX(values, DLGI_CALLER_CONTACT_COL_IDX))= DB1_STR; - VAL_TYPE(GET_FIELD_IDX(values, DLGI_CALLER_SOCK_IDX)) = DB1_STR; + VAL_TYPE(GET_FIELD_IDX(values, DLGI_CALLER_SOCK_IDX)) = DB1_STR; VAL_TYPE(GET_FIELD_IDX(values, DLGI_TOROUTE_NAME_COL_IDX)) = DB1_STR; VAL_INT(GET_FIELD_IDX(values, DLGI_HASH_ENTRY_COL_IDX)) = cell->h_entry; @@ -1075,11 +1075,13 @@ int update_dialog_dbinfo_unsafe(struct dlg_cell * cell) SET_PROPER_NULL_FLAG(cell->toroute_name, values, DLGI_TOROUTE_NAME_COL_IDX); SET_PROPER_NULL_FLAG(cell->first_req_cseq, values, DLGI_CALLER_CSEQ_COL_IDX); - VAL_NULL(GET_FIELD_IDX(values, DLGI_HASH_ENTRY_COL_IDX))= 0; - VAL_NULL(GET_FIELD_IDX(values, DLGI_HASH_ID_COL_IDX)) = 0; - VAL_NULL(GET_FIELD_IDX(values, DLGI_STATE_COL_IDX)) = 0; + VAL_NULL(GET_FIELD_IDX(values, DLGI_HASH_ENTRY_COL_IDX)) = 0; + VAL_NULL(GET_FIELD_IDX(values, DLGI_HASH_ID_COL_IDX)) = 0; + VAL_NULL(GET_FIELD_IDX(values, DLGI_STATE_COL_IDX)) = 0; VAL_NULL(GET_FIELD_IDX(values, DLGI_START_TIME_COL_IDX)) = 0; - VAL_NULL(GET_FIELD_IDX(values, DLGI_TIMEOUT_COL_IDX)) = 0; + VAL_NULL(GET_FIELD_IDX(values, DLGI_TIMEOUT_COL_IDX)) = 0; + VAL_NULL(GET_FIELD_IDX(values, DLGI_CALLER_CONTACT_COL_IDX)) = 0; + VAL_NULL(GET_FIELD_IDX(values, DLGI_CALLER_SOCK_IDX)) = 0; if((dialog_dbf.insert(dialog_db_handle, insert_keys, values, DIALOG_IN_TABLE_COL_NO)) !=0){ LM_ERR("could not add another dialog_in to db\n"); [View Less]

1 0

git:coudot/escape-param: lib/kcore: make escape_param() conform to RFC3261
by Camille Oudot 06 Nov '13

06 Nov '13

Module: sip-router Branch: coudot/escape-param Commit: d2e964190cc4768a9968af3f699226980c169e07 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d2e9641… Author: Camille Oudot <camille.oudot(a)orange.com> Committer: Camille Oudot <camille.oudot(a)orange.com> Date: Wed Nov 6 16:28:44 2013 +0100 lib/kcore: make escape_param() conform to RFC3261 escape_param --- lib/kcore/strcommon.c | 72 ++++++++++++++++++++++++++++++------------------ 1 files … [View More]changed, 45 insertions(+), 27 deletions(-) diff --git a/lib/kcore/strcommon.c b/lib/kcore/strcommon.c index 1c4e9a2..3733cda 100644 --- a/lib/kcore/strcommon.c +++ b/lib/kcore/strcommon.c @@ -312,7 +312,11 @@ int unescape_param(str *sin, str *sout) /*! \brief * Escape all printable characters that are not valid in - * a param part of request uri: = | ; | , | | " | ? | & + * a param part of request uri + * no_need_to_escape = unreserved | param-unreserved + * unreserved = alphanum | mark + * mark = - | _ | . | ! | ~ | * | ' | ( | ) + * param-unreserved = [ | ] | / | : | & | + | $ */ int escape_param(str *sin, str *sout) { @@ -329,33 +333,47 @@ int escape_param(str *sin, str *sout) if (*p < 32 || *p > 126) { LM_ERR("invalid escaped character <%u>\n", (unsigned int)*p); return -1; - } - switch (*p) { - case ' ': - case '?': - case '&': - case '=': - case ',': - case ';': - case '"': - case '+': - *at++ = '%'; - x = (*p) >> 4; - if (x < 10) - { - *at++ = x + '0'; - } else { - *at++ = x - 10 + 'a'; - } - x = (*p) & 0x0f; - if (x < 10) { - *at = x + '0'; - } else { - *at = x - 10 + 'a'; + } else if (isdigit((int)*p) || ((*p >= 'A') && (*p <= 'Z')) || + ((*p >= 'a') && (*p <= 'z'))) { + + } else { + switch (*p) { + case '-': + case '_': + case '.': + case '!': + case '~': + case '*': + case '\'': + case '(': + case ')': + case '[': + case ']': + case '/': + case ':': + case '&': + case '+': + case '$': + *at = *p; + break; + default: + + *at++ = '%'; + x = (*p) >> 4; + if (x < 10) + { + *at++ = x + '0'; + } else { + *at++ = x - 10 + 'a'; + } + x = (*p) & 0x0f; + if (x < 10) { + *at = x + '0'; + } else { + *at = x - 10 + 'a'; + } + break; } - break; - default: - *at = *p; } at++; p++; [View Less]

1 0

Mitigation of unavailable rtpproxy
by Alex Balashov 06 Nov '13

06 Nov '13

Hello, (Sorry for cross-posting to -users and -dev; not really sure where this post belongs most.) A few days ago, I ran into an issue with a Kamailio server being somewhat unresponsive, during moderate call volume, on account of a down rtpproxy--the only rtpproxy in the set. This is rtpproxy classic, not ngcp-mediaproxy-ng. Rtpproxy was not actually engaged on any of the initial INVITEs going through the server; the server is configured to invoke it conditionally based on a setting,… [View More] and the setting was not set for any endpoints. rtpproxy_manage() was never called. However, I call unforce_rtp_proxy() unconditionally in my config when handling CANCELs, reasoning that it can't do any harm if rtpproxy_manage() was not called before[1]. Nevertheless, it seemed to be the case that this situation was clogging up SIP worker threads, because some SIP messages were definitely dropped. Periodic log messages about inability to reach the rtpproxy were echoed as well. This problem cleared up almost immediately when the rtpproxy instance was restored into service. This raised some questions in my mind about the relationship between rtpproxy management and SIP worker thread utilisation. I assume it was my indiscriminate unforce_rtp_proxy() calls that were actually clogging up the worker threads, right? If so, why? I figured that in the unforce_rtp_proxy() case, the rtpproxy module simply sends fire-and-forget UDP messages down the UDP control socket without any sort of blocking for acknowledgement, since in this case the call must be released on the rtpproxy side without doing any rewriting of SDP on the Kamailio side (unlike in the case where rtpproxy is engaged). Thus, there should be no need to wait for ports to substitute into the message. Or is the same response-wait mechanism used regardless, even in the unforce_rtp_proxy() case, for programmatic reasons? More broadly, is there any way that this scenario can be prevented? In other words, is there a way to work around an outage of all rtpproxies in the set without tying up workers, or at least tying them up less severely? Thanks! -- Alex [1] Is this a reasonable assumption? The reason I do this is that I don't see a way to find out if rtpproxy was engaged from the body of a CANCEL message. I do check for a ;proxy_media RR parameter when handling BYEs, but since a CANCEL is not an in-dialog request, I'm not sure what to do except to call unforce_rtp_proxy()/rtpproxy_manage() indiscriminately, without resorting to storing state in htable or other complications I don't want. -- Alex Balashov - Principal Evariste Systems LLC 235 E Ponce de Leon Ave Suite 106 Decatur, GA 30030 United States Tel: +1-678-954-0670 Web: http://www.evaristesys.com/, http://www.alexbalashov.com/ [View Less]

2 1

[tracker] Task opened: Errors in snmpstats KAMAILIO-TC (Attachment added)
by sip-router 06 Nov '13

06 Nov '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - i (takeshi) Attached to Project - sip-router Summary - Errors in snmpstats KAMAILIO-TC Task Type - Bug Report Category - Module Status - Unconfirmed Assigned To - Operating System - All Severity - Low Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - When I use snmpget to get snmpstats, I get this: [root@… [View More]LAB002185-FLIP-SERVER SIP-Trunking]# snmpget -v2c -c public 192.168.2.185 kamailioCurNumDialogs.0 2013-11-06 21:07:46 Expected "(" (_): At line 65 in /usr/share/snmp/mibs/KAMAILIO-TC 2013-11-06 21:07:46 Bad operator ((): At line 65 in /usr/share/snmp/mibs/KAMAILIO-TC 2013-11-06 21:07:46 Did not find 'KamailioSIPMethodIdentifier' in module KAMAILIO-TC (/usr/share/snmp/mibs/KAMAILIO-SIP-COMMON-MIB.txt) 2013-11-06 21:07:46 Did not find 'KamailioSIPEntityRole' in module KAMAILIO-TC (/usr/share/snmp/mibs/KAMAILIO-SIP-COMMON-MIB.txt) 2013-11-06 21:07:46 Did not find 'X731UsageState' in module KAMAILIO-TC (/usr/share/snmp/mibs/KAMAILIO-MIB.txt) 2013-11-06 21:07:46 Did not find 'X731AlarmStatus' in module KAMAILIO-TC (/usr/share/snmp/mibs/KAMAILIO-MIB.txt) 2013-11-06 21:07:46 Did not find 'X731AlarmState' in module KAMAILIO-TC (/usr/share/snmp/mibs/KAMAILIO-MIB.txt) KAMAILIO-MIB::kamailioCurNumDialogs.0 = Gauge32: 0 The command doesn't fail, as data is reported correctly, but the error messages are annoying. I compared, KAMAILIO-TC with the old OPENSER-TC, and found some issues. I have created a patch (see attachment) for commit: 0fbdb8cf7a7687d6ecc8049dfdcb1244abd726af After that, it works as expected: [root@LAB002185-FLIP-SERVER SIP-Trunking]# snmpget -v2c -c public 192.168.2.185 kamailioCurNumDialogs.0 KAMAILIO-MIB::kamailioCurNumDialogs.0 = Gauge32: 0 Obs: my only doubt is about the line for --sctp_tls(5). I am just commenting it to disable it (the problem is that the parser doesn't like the underscore in the word) but maybe it must be corrected differently. 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=363 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

← Newer
1
...
13
14
15
16
17
18
19
...
22
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev November 2013