sr-dev December 2013

sr-dev@lists.kamailio.org

30 participants
318 discussions

[tracker] Task opened: SegFault in parse_avp_db
by sip-router 11 Dec '13

11 Dec '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Kevin Scott Adams (TheGrandWazoo) Attached to Project - sip-router Summary - SegFault in parse_avp_db Task Type - Bug Report Category - Module Status - Unconfirmed Assigned To - Operating System - Linux Severity - Critical Priority - Normal Reported Version - 4.1 Due in Version - Undecided Due Date - Undecided Details - Upgraded to 4.1 to fix a SegFault with t_load_contacts and now have this SegFault on startup. Core was generated by `kamailio'. Program terminated with signal 11, Segmentation fault. #0 0x00002b359c9b4d9d in parse_avp_db (s=0x2b35985f2788 "*", dbp=0x2b35986963c8, allow_scheme=1) at avpops_parse.c:132 132 dbp->a.u.sval->pvp.pvn.u.isname.type |= (flags<<8)&0xff00; (gdb) bt full #0 0x00002b359c9b4d9d in parse_avp_db (s=0x2b35985f2788 "*", dbp=0x2b35986963c8, allow_scheme=1) at avpops_parse.c:132 ul = 47509190304664 tmp = {s = 0x2b35985f2788 "*", len = -1739042800} s0 = {s = 0x2b359c9b9eea "fixup_db_avp", len = -1667522637} have_scheme = 0 '\000' p0 = 0x0 flags = 0 __FUNCTION__ = "parse_avp_db" #1 0x00002b359c99e731 in fixup_db_avp (param=0x2b35985f21b8, param_no=2, allow_scheme=1) at avpops.c:295 sp = 0x2b3598696348 dbp = 0x2b35986963c8 flags = 0 s = {s = 0x2b35985f2788 "*", len = 3} p = 0x0 __FUNCTION__ = "fixup_db_avp" #2 0x00002b359c99e8c0 in fixup_db_load_avp (param=0x2b35985f21b8, param_no=2) at avpops.c:309 No locals. #3 0x00000000004b1f0d in fix_actions (a=0x2b35985f2148) at route.c:975 t = 0x2b35985f2148 p = 0x5f8c59 tmp = 0x2b35985ec7c8 "SETPAI with fU as: $fU" tmp_p = 0x2b35985f2788 ret = 0 i = 1 cmd = 0x2b35985d5b98 s = {s = 0x2b3598696660 "\310\307^\230\065+", len = 0} he = 0x2b35985ec7c8 ip = {af = 48, len = 48, u = {addrl = {140733577136352, 140733577136144}, addr32 = {383748320, 32767, 383748112, 32767}, addr16 = {35040, 5855, 32767, 0, 34832, 5855, 32767, 0}, addr = "\340\210\337\026\377\177\000\000\020\210\337\026\377\177\000"}} si = 0x2b3500000016 lval = 0x2b359b6d853d rve = 0x2b3598693700 err_rve = 0x324b84cf08 rve_type = 32767 err_type = 383748352 expected_type = RV_NONE rv = 0x10 rve_param_no = 0 __FUNCTION__ = "fix_actions" #4 0x00000000004cc862 in fix_rval (rv=0x2b35985f2870) at rvalue.c:2883 __FUNCTION__ = "fix_rval" #5 0x00000000004d4c70 in fix_rval_expr (p=0x2b35985f2868) at rvalue.c:3804 rve = 0x2b35985f2868 ret = 32767 __FUNCTION__ = "fix_rval_expr" #6 0x00000000004aff2d in fix_actions (a=0x2b35985f1528) at route.c:750 t = 0x2b35985f3f08 p = 0x4cc862 tmp = 0x5f8c50 "fix_rval" tmp_p = 0x2b35985ec7c8 ret = 0 i = 2 cmd = 0x2b35985d1b90 ---Type <return> to continue, or q <return> to quit--- s = {s = 0x5d8e18 "DEBUG", len = 383750288} he = 0x2b359b6d853d ip = {af = 16, len = 0, u = {addrl = {6131224, 4294967300}, addr32 = {6131224, 0, 4, 1}, addr16 = {36376, 93, 0, 0, 4, 0, 1, 0}, addr = "\030\216]\000\000\000\000\000\004\000\000\000\001\000\000"}} si = 0x5d8e18 lval = 0x7fff16df8b90 rve = 0x2b35985f2868 err_rve = 0x0 rve_type = RV_INT err_type = 2607646013 expected_type = 11061 rv = 0x7fff16df8cf4 rve_param_no = 0 __FUNCTION__ = "fix_actions" #7 0x00000000004aff6c in fix_actions (a=0x2b35985f0598) at route.c:754 t = 0x2b35985f4048 p = 0x0 tmp = 0x7fff16df8fb0 "" tmp_p = 0x2b35985ec518 ret = 0 i = 2 cmd = 0x2b35985d1b90 s = {s = 0x2b35985e0b58 "Route", len = 5} he = 0x7fff16df8d80 ip = {af = 383749552, len = 32767, u = {addrl = {5533488, 17179869188}, addr32 = {5533488, 0, 4, 4}, addr16 = { 28464, 84, 0, 0, 4, 0, 4, 0}, addr = "0oT\000\000\000\000\000\004\000\000\000\004\000\000"}} si = 0x2b359dad3518 lval = 0x2b35985e25d0 rve = 0x2b35985f08e0 err_rve = 0x0 rve_type = RV_INT err_type = 2648027208 expected_type = RV_NONE rv = 0x2b35985e5a50 rve_param_no = 0 __FUNCTION__ = "fix_actions" #8 0x00000000004b9efb in fix_rl (rt=0x939b80) at route.c:2162 i = 1 ret = 0 #9 0x00000000004b9f39 in fix_rls () at route.c:2178 ret = 4985 #10 0x0000000000470507 in main (argc=1, argv=0x7fff16df9098) at main.c:2522 cfg_stream = 0xefc9010 c = -1 r = 50 tmp = 0x7fff16df8f87 "" tmp_len = 32767 port = 383750022 proto = 0 options = 0x5e2318 ":f:cm:M:dVIhEeb:l:L:n:vKrRDTN:W:w:t:u:g:P:G:SQ:O:a:A:" ret = -1 seed = 3011491440 rfd = 4 debug_save = 0 ---Type <return> to continue, or q <return> to quit--- debug_flag = 0 dont_fork_cnt = 0 n_lst = 0x0 p = 0x324b61cbc0 "" __FUNCTION__ = "main" Thanks More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=381 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: fixes for memory leak in TLS module
by sip-router 11 Dec '13

11 Dec '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#380 - fixes for memory leak in TLS module User who did this - Ding Ma (mading087) ---------- License of the patch is BSD ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=380#comment1218 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

Separate timeout for unconfirmed dialogs?
by Alex Balashov 11 Dec '13

11 Dec '13

Hi, I am curious as to whether anyone thinks it's a bad idea to write a separate timeout parameter to 'dialog' for established-unconfirmed (200 OK, no ACK). It is usually desirable to terminate these far earlier than the default_timeout. -- 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/

1 0

git:master: pkg/kamailio/deb: removing lucid
by Victor Seva 11 Dec '13

11 Dec '13

Module: sip-router Branch: master Commit: a44d5311b583bc457b44550870dfbd4ea3fbcea4 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a44d531… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: Wed Dec 11 14:11:19 2013 +0100 pkg/kamailio/deb: removing lucid --- pkg/kamailio/deb/lucid/changelog | 90 ----- pkg/kamailio/deb/lucid/compat | 1 - pkg/kamailio/deb/lucid/control | 459 ------------------------- pkg/kamailio/deb/lucid/copyright | 27 -- pkg/kamailio/deb/lucid/kamailio.README.Debian | 38 -- pkg/kamailio/deb/lucid/kamailio.default | 33 -- pkg/kamailio/deb/lucid/kamailio.dirs | 2 - pkg/kamailio/deb/lucid/kamailio.examples | 2 - pkg/kamailio/deb/lucid/kamailio.init | 240 ------------- pkg/kamailio/deb/lucid/kamailio.postinst | 48 --- pkg/kamailio/deb/lucid/rules | 228 ------------ 11 files changed, 0 insertions(+), 1168 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=a44…

1 0

git:master: pkg/kamailio/deb: use gcj-* instead default-* one
by Victor Seva 11 Dec '13

11 Dec '13

Module: sip-router Branch: master Commit: 4d4ab8becd7d806f8cae43149aab1e73bbe54969 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=4d4ab8b… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: Wed Dec 11 13:48:36 2013 +0100 pkg/kamailio/deb: use gcj-* instead default-* one --- pkg/kamailio/deb/debian/control | 4 ++-- pkg/kamailio/deb/jessie/control | 4 ++-- pkg/kamailio/deb/wheezy/control | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/pkg/kamailio/deb/debian/control b/pkg/kamailio/deb/debian/control index 8271807..6d8c31b 100644 --- a/pkg/kamailio/deb/debian/control +++ b/pkg/kamailio/deb/debian/control @@ -4,7 +4,7 @@ Priority: optional Maintainer: Jon Bonilla <manwe(a)aholab.ehu.es> Build-Depends: bison, debhelper (>= 5), - default-jdk, + gcj-jdk, docbook-xml, dpatch, dpkg-dev (>= 1.13.19), @@ -494,7 +494,7 @@ Description: authentication using ephemeral credentials module for Kamailio Package: kamailio-java-modules Architecture: any Depends: kamailio (= ${Source-Version}), - default-jre, + gcj-jre, ${shlibs:Depends} Description: contains the app_java module Kamailio is a very fast and flexible SIP (RFC3261) diff --git a/pkg/kamailio/deb/jessie/control b/pkg/kamailio/deb/jessie/control index 8271807..6d8c31b 100644 --- a/pkg/kamailio/deb/jessie/control +++ b/pkg/kamailio/deb/jessie/control @@ -4,7 +4,7 @@ Priority: optional Maintainer: Jon Bonilla <manwe(a)aholab.ehu.es> Build-Depends: bison, debhelper (>= 5), - default-jdk, + gcj-jdk, docbook-xml, dpatch, dpkg-dev (>= 1.13.19), @@ -494,7 +494,7 @@ Description: authentication using ephemeral credentials module for Kamailio Package: kamailio-java-modules Architecture: any Depends: kamailio (= ${Source-Version}), - default-jre, + gcj-jre, ${shlibs:Depends} Description: contains the app_java module Kamailio is a very fast and flexible SIP (RFC3261) diff --git a/pkg/kamailio/deb/wheezy/control b/pkg/kamailio/deb/wheezy/control index 319dcb3..70e7da8 100644 --- a/pkg/kamailio/deb/wheezy/control +++ b/pkg/kamailio/deb/wheezy/control @@ -4,7 +4,7 @@ Priority: optional Maintainer: Jon Bonilla <manwe(a)aholab.ehu.es> Build-Depends: bison, debhelper (>= 5), - default-jdk, + gcj-jdk, docbook-xml, dpatch, dpkg-dev (>= 1.13.19), @@ -478,7 +478,7 @@ Description: authentication using ephemeral credentials module for Kamailio Package: kamailio-java-modules Architecture: any Depends: kamailio (= ${Source-Version}), - default-jre, + gcj-jre, ${shlibs:Depends} Description: contains the app_java module Kamailio is a very fast and flexible SIP (RFC3261)

1 0

git:master: lib/ims: Replaced custom getter for P-Asserted-Identity with the Parser from Core
by Carsten Bock 11 Dec '13

11 Dec '13

Module: sip-router Branch: master Commit: af1734cea213b2a07c9d725fff158cd4899725d1 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=af1734c… Author: Carsten Bock <carsten(a)ng-voice.com> Committer: Carsten Bock <carsten(a)ng-voice.com> Date: Wed Dec 11 13:47:57 2013 +0100 lib/ims: Replaced custom getter for P-Asserted-Identity with the Parser from Core --- lib/ims/ims_getters.c | 45 +++++++-------------------------------------- 1 files changed, 7 insertions(+), 38 deletions(-) diff --git a/lib/ims/ims_getters.c b/lib/ims/ims_getters.c index 8fce196..a41cd31 100644 --- a/lib/ims/ims_getters.c +++ b/lib/ims/ims_getters.c @@ -56,6 +56,7 @@ #include "../../parser/parse_from.h" #include "../../parser/parse_content.h" #include "ims_getters.h" +#include "../../parser/parse_ppi_pai.h" /** @@ -389,45 +390,13 @@ str s_asserted_identity={"P-Asserted-Identity",19}; */ str cscf_get_asserted_identity(struct sip_msg *msg) { - name_addr_t id; - struct hdr_field *h; - rr_t *r; - memset(&id,0,sizeof(name_addr_t)); - if (!msg) return id.uri; - if (parse_headers(msg, HDR_EOH_F, 0)<0) { - return id.uri; - } - h = msg->headers; - while(h) - { - if (h->name.len == s_asserted_identity.len && - strncasecmp(h->name.s,s_asserted_identity.s,s_asserted_identity.len)==0) - { - if (parse_rr(h)<0){ - //This might be an old client - LM_CRIT("WARN:cscf_get_asserted_identity: P-Asserted-Identity header must contain a Nameaddr!!! Fix the client!\n"); - id.name.s = h->body.s; - id.name.len = 0; - id.len = h->body.len; - id.uri = h->body; - while(id.uri.len && (id.uri.s[0]==' ' || id.uri.s[0]=='\t' || id.uri.s[0]=='<')){ - id.uri.s = id.uri.s+1; - id.uri.len --; - } - while(id.uri.len && (id.uri.s[id.uri.len-1]==' ' || id.uri.s[id.uri.len-1]=='\t' || id.uri.s[id.uri.len-1]=='>')){ - id.uri.len--; - } - return id.uri; - } - r = (rr_t*) h->parsed; - id = r->nameaddr; - free_rr(&r); - h->parsed=r; - return id.uri; - } - h = h->next; + str uri = {0,0}; + if (!msg) return uri; + if((parse_pai_header(msg) == 0) && (msg->pai) && (msg->pai->parsed)) { + to_body_t *pai = get_pai(msg)->id; + return pai->uri; } - return id.uri; + return uri; } static str phone_context_s={";phone-context=",15};

1 0

pua and pua_reginfo modules
by Richard Good 11 Dec '13

11 Dec '13

Hi I am re-using the pua_reginfo module to implement P-CSCF subscription to Registration-State information at the S-CSCF. For the first SUBSCRIBE everything works fine and as per RFC 3680 the SUBSCRIBE has the presentity URI as request-URI, e.g. SUBSCRIBE sip:bob@biloxi.example.com SIP/2.0 The S-CSCF returns 200 OK with contact-header e.g.: Contact: sip:server19.example.com >From RFC 3261: 12.1.2 UAC Behavior ........... The remote target MUST be set to the URI from the Contact header field of the response. So any subsequent SUBSCRIBE has its remote target (and hence request-URI) changed to the contact returned in the 200 OK. So any subsequent SUBSCRIBE requests have request-URI: SUBSCRIBE *sip:scscf.example.com:6060 <http://scscf.example.com:6060>* SIP/2.0 Currently our S-CSCF implementation uses the request-URI to look up the presentity URI so any subsequent SUBSCRIBE requests fail. >From RFC 3265: 3.1.4.2. Refreshing of Subscriptions ..... The handling for such a request is the same as for the initial creation of a subscription except as described below. Does anyone know what subsequent SUBSCRIBE requests should have in their request-URI? The contact returned in the first 200 OK (the S-CSCF contact) as per RFC 3261 or the same as the initial SUBSCRIBE as per RFC 3265 with the presentity URI in the request-URI? Regards Richard. This email is subject to the disclaimer of Smile Communications at http://www.smilecoms.com/disclaimer

2 2

$var null confusion
by Juha Heinanen 11 Dec '13

11 Dec '13

the fact that $var cannot hold value seems to be a constant cause of confusion. opensips version 1.9.0 has this: $var() script variables have now an explicit value of NULL, so values can be (a) integer, (b) string or (c) NULL. The NULL value can be tested in an "if" conditions i don't know how it is implemented, but might be a good idea for kamailio too. -- juha

2 1

git:vseva/test DELETED: sorry for the noise.
by Victor Seva 11 Dec '13

11 Dec '13

Module: sip-router Branch: vseva/test DELETED Commit: 881a8c747f629a449c11450535b8b99e2d4ca9b5 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=881a8c7… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: Wed Dec 11 10:35:16 2013 +0100 sorry for the noise. --- README.github | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/README.github b/README.github index 63a5b71..54705e5 100644 --- a/README.github +++ b/README.github @@ -4,4 +4,4 @@ testing github mirroring sync. ..... This seems to work. -.. \ No newline at end of file +... \ No newline at end of file

1 0

git:vseva/test: sorry for the noise.
by Victor Seva 11 Dec '13

11 Dec '13

Module: sip-router Branch: vseva/test Commit: 881a8c747f629a449c11450535b8b99e2d4ca9b5 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=881a8c7… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: Wed Dec 11 10:35:16 2013 +0100 sorry for the noise. --- README.github | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/README.github b/README.github index 63a5b71..54705e5 100644 --- a/README.github +++ b/README.github @@ -4,4 +4,4 @@ testing github mirroring sync. ..... This seems to work. -.. \ No newline at end of file +... \ No newline at end of file

1 0

← Newer
1
...
12
13
14
15
16
17
18
...
32
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev December 2013