sr-dev November 2013

sr-dev@lists.kamailio.org

29 participants
222 discussions

[Redis module] Kamailio crashes in case of connection lost to redis server
by Tuan Viet Nguyen 28 Nov '13

28 Nov '13

Hello, I'll try to shut down the redis server to test the behavior of kamailio and it has crashed if a call is received and then cancelled. *1/The kamailio version is 4.0.4* *2/ Kamailio log * /usr/local/sbin/kamailio[25333]: ERROR: ndb_redis [redis_client.c:364]: redisc_exec(): Redis error: Server closed the connection /usr/local/sbin/kamailio[25361]: : <core> [pass_fd.c:293]: receive_fd(): ERROR: receive_fd: EOF on 13 /usr/local/sbin/kamailio[25328]: ALERT: <core> [main.c:788]: handle_sigs(): child process 25333 exited by a signal 11 /usr/local/sbin/kamailio[25328]: ALERT: <core> [main.c:791]: handle_sigs(): core was generated *3/ Interesting information in the core* #3 0x00007fc79412893d in redisvCommand (c=0x64657461, format=0x9 <Address 0x9 out of bounds>, ap=0x30, ap@entry=0x7fff0ff56aa8) at hiredis.c:1304 No locals. #4 0x00007fc794341713 in redisc_exec (srv=srv@entry=0x7fff0ff56be0, res=res@entry=0x7fff0ff56c00, cmd=cmd@entry=0x7fff0ff56bf0) at redis_client.c:368 rsrv = 0x7fc794565150 rpl = 0x7fc7946fab70 c = 0 '\000' ap = {{gp_offset = 48, fp_offset = 48, overflow_arg_area = 0x7fff0ff56bb0, reg_save_area = 0x7fff0ff56ac0}} __FUNCTION__ = "redisc_exec" #5 0x00007fc79433b781 in w_redis_cmd5 (msg=<optimized out>, ssrv=<optimized out>, scmd=<optimized out>, sargv1=<optimized out>, sargv2=0x7fc7946f7bf0 "p\243_\224\307\177", sres=0x7fc7946f7c50 " \253_\224\307\177") at ndb_redis_mod.c:250 s = {{s = 0x7fc7945fb300 "kamailio_redis", len = 14}, {s = 0x7fc7945f5f50 "PUBLISH %s %s", len = 13}, {s = 0x7fc7945fab20 "r", len = 1}} arg1 = {s = 0x7fc7945f5f80 "notification", len = 12} arg2 = { s = 0x7fc794551c60 "info XXX"..., len = 212} c1 = 0 '\000' c2 = 0 '\000' __FUNCTION__ = "w_redis_cmd5" I've found one of post that this issue has been fixed but it seems that it's always the case .. http://www.mail-archive.com/search?l=sr-users@lists.sip-router.org&q=subjec… Do you have any idea? Thank you

3 3

[tracker] Comment added: Crash in TCP Read on Kamailio 4.0.1
by sip-router 28 Nov '13

28 Nov '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#364 - Crash in TCP Read on Kamailio 4.0.1 User who did this - Vitaliy Aleksandrov (Vitaliy) ---------- Done. Now my kamailio is built with MEMDBG=1. Should I enable memlog or memdbg log levels to get error messages about memory ? ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=364#comment1181 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.

1 0

Assigning a null value from a $sht to a $var fails after upgrade on master
by Hugh Waite 27 Nov '13

27 Nov '13

Hello, We are having an issue with PVs after upgrading to the latest master (i.e. 4.1.0-pre1) which might be related to the comparison of $null and 0 In our previous version (compiled on 9th October) the following code worked. It is using the htable as a DB cache $var(ip_trusted) = $sht(trusted=>$var(srcip)); if($var(ip_trusted) == $null) { /* DB lookup */ $sht(trusted->$var(srcip)) = $var(ip_trusted); } After upgrading, the first line is not working as expected. The htable value is null (not assigned), but the $var(ip_trusted) value is assigned the value 0, so the check fails. Is this related to the pv comparison bugfix (http://git.sip-router.org/cgi-bin/gitweb.cgi?p=sip-router;a=commitdiff;h=38… Certainly something has changed since 9 Oct. Hugh -- Hugh Waite Principal Design Engineer Crocodile RCS Ltd.

2 2

[tracker] Task closed: location not updated correctly when using GRUUs
by sip-router 27 Nov '13

27 Nov '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#368 - location not updated correctly when using GRUUs User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Fixed More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=368 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.

1 0

[tracker] Comment added: uac_redirect not honoring q-value for max_total = 1
by sip-router 27 Nov '13

27 Nov '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#349 - uac_redirect not honoring q-value for max_total = 1 User who did this - Stefan Prelle (spr) ---------- The workaround with t_*_contacts is working (at least with 4.0.4). perfectly. Thank you for the hint. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=349#comment1180 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.

1 0

git:master: modules/ims_registrar_pcscf: fix log
by Camille Oudot 26 Nov '13

26 Nov '13

Module: sip-router Branch: master Commit: 0e711165f9e26b04110cadda9b372eae0387df41 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0e71116… Author: Camille Oudot <camille.oudot(a)orange.com> Committer: Camille Oudot <camille.oudot(a)orange.com> Date: Tue Nov 26 16:08:35 2013 +0100 modules/ims_registrar_pcscf: fix log --- modules/ims_registrar_pcscf/service_routes.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules/ims_registrar_pcscf/service_routes.c b/modules/ims_registrar_pcscf/service_routes.c index 8af77e2..ab57cd6 100644 --- a/modules/ims_registrar_pcscf/service_routes.c +++ b/modules/ims_registrar_pcscf/service_routes.c @@ -349,7 +349,7 @@ int force_service_routes(struct sip_msg* _m, udomain_t* _d) { } LM_DBG("Setting dst_uri to <%.*s> \n", c->service_routes[0].len, - c->service_routes[i].s); + c->service_routes[0].s); if (set_dst_uri(_m, &c->service_routes[0]) !=0 ) { LM_ERR("Error setting new dst uri\n");

1 0

git:4.1: rtpproxy-ng: remove trailing double \r\n from multipart SDP
by Richard Fuchs 26 Nov '13

26 Nov '13

Module: sip-router Branch: 4.1 Commit: 0b85493f3c1bc7c6cbd48f39aea9d3ecc135d4ff URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0b85493… Author: Richard Fuchs <rfuchs(a)sipwise.com> Committer: Richard Fuchs <rfuchs(a)sipwise.com> Date: Tue Nov 26 10:36:18 2013 -0500 rtpproxy-ng: remove trailing double \r\n from multipart SDP --- modules/rtpproxy-ng/rtpproxy_funcs.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/modules/rtpproxy-ng/rtpproxy_funcs.c b/modules/rtpproxy-ng/rtpproxy_funcs.c index 1c283e5..3f7079e 100644 --- a/modules/rtpproxy-ng/rtpproxy_funcs.c +++ b/modules/rtpproxy-ng/rtpproxy_funcs.c @@ -257,7 +257,10 @@ int extract_body(struct sip_msg *msg, str *body ) } /* end of while */ if(c==1) { - while(rest && (*rest=='\r' || *rest=='\n')) rest++; + if (rest < p2 && *rest == '\r') rest++; + if (rest < p2 && *rest == '\n') rest++; + if (rest < p2 && p2[-1] == '\n') p2--; + if (rest < p2 && p2[-1] == '\r') p2--; body->s = rest; body->len = p2-rest; goto done;

1 0

git:master: rtpproxy-ng: remove trailing double \r\n from multipart SDP
by Richard Fuchs 26 Nov '13

26 Nov '13

Module: sip-router Branch: master Commit: dd3ce694280e9748740680a19837b530f48b5bc5 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=dd3ce69… Author: Richard Fuchs <rfuchs(a)sipwise.com> Committer: Richard Fuchs <rfuchs(a)sipwise.com> Date: Tue Nov 26 10:36:18 2013 -0500 rtpproxy-ng: remove trailing double \r\n from multipart SDP --- modules/rtpproxy-ng/rtpproxy_funcs.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/modules/rtpproxy-ng/rtpproxy_funcs.c b/modules/rtpproxy-ng/rtpproxy_funcs.c index 1c283e5..3f7079e 100644 --- a/modules/rtpproxy-ng/rtpproxy_funcs.c +++ b/modules/rtpproxy-ng/rtpproxy_funcs.c @@ -257,7 +257,10 @@ int extract_body(struct sip_msg *msg, str *body ) } /* end of while */ if(c==1) { - while(rest && (*rest=='\r' || *rest=='\n')) rest++; + if (rest < p2 && *rest == '\r') rest++; + if (rest < p2 && *rest == '\n') rest++; + if (rest < p2 && p2[-1] == '\n') p2--; + if (rest < p2 && p2[-1] == '\r') p2--; body->s = rest; body->len = p2-rest; goto done;

1 0

git:luismartingil/ndb_cassandra: ndb_cassandra: Initial commit of the module
by Luis Martin Gil 25 Nov '13

25 Nov '13

Module: sip-router Branch: luismartingil/ndb_cassandra Commit: 5caf33ceaea8449adfe134fa784c10b2d0f5d068 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=5caf33c… Author: Luis Martin Gil <martingil.luis(a)gmail.com> Committer: Luis Martin Gil <martingil.luis(a)gmail.com> Date: Mon Nov 25 18:20:02 2013 -0500 ndb_cassandra: Initial commit of the module - Simply adding all the files of the new module. --- modules/ndb_cassandra/.gitignore | 1 + modules/ndb_cassandra/Cassandra.cpp |11061 ++++++++++++++++++++ modules/ndb_cassandra/Cassandra.h | 4685 +++++++++ modules/ndb_cassandra/Makefile | 44 + modules/ndb_cassandra/README | 162 + modules/ndb_cassandra/cassandra_constants.cpp | 18 + modules/ndb_cassandra/cassandra_constants.h | 24 + modules/ndb_cassandra/cassandra_types.cpp | 3263 ++++++ modules/ndb_cassandra/cassandra_types.h | 1999 ++++ modules/ndb_cassandra/doc/Makefile | 4 + modules/ndb_cassandra/doc/ndb_cassandra.xml | 37 + modules/ndb_cassandra/doc/ndb_cassandra_admin.xml | 171 + modules/ndb_cassandra/ndb_cassandra.c | 301 + modules/ndb_cassandra/test-cpp/Cassandra.cpp | 1 + modules/ndb_cassandra/test-cpp/Cassandra.h | 1 + modules/ndb_cassandra/test-cpp/Makefile | 33 + .../ndb_cassandra/test-cpp/cassandra_constants.cpp | 1 + .../ndb_cassandra/test-cpp/cassandra_constants.h | 1 + modules/ndb_cassandra/test-cpp/cassandra_types.cpp | 1 + modules/ndb_cassandra/test-cpp/cassandra_types.h | 1 + modules/ndb_cassandra/test-cpp/test.c | 42 + modules/ndb_cassandra/test-cpp/thrift_wrapper.cpp | 1 + modules/ndb_cassandra/test-cpp/thrift_wrapper.h | 1 + modules/ndb_cassandra/thrift_wrapper.cpp | 129 + modules/ndb_cassandra/thrift_wrapper.h | 18 + 25 files changed, 22000 insertions(+), 0 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=5ca…

1 0

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

25 Nov '13

Module: sip-router Branch: coudot/escape-param DELETED 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 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++;

1 0

← Newer
1
2
3
4
5
6
7
8
...
23
Older →

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev November 2013