sr-dev November 2010

sr-dev@lists.kamailio.org

17 participants
108 discussions

[tracker] Assignee added: Assignment operators don't work
by sip-router 06 Jul '25

06 Jul '25

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A user has added themself to the list of users assigned to this task. FS#100 - Assignment operators don't work User who did this - Alex Hermann (axlh) http://sip-router.org/tracker/index.php?do=details&task_id=100 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.

3 2

git:master: modules_k/nathelper: add_contact_alias adds <> s around contact URI
by Juha Heinanen 07 Sep '11

07 Sep '11

Module: sip-router Branch: master Commit: 3ff9594a30eff2d18e6ea0595c5f82f878eaca1c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3ff9594… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Fri Nov 26 10:37:30 2010 +0200 modules_k/nathelper: add_contact_alias adds <>s around contact URI Make sure that Contact URI is surrounded by <> when adding ;alias parameter. Otherwise, ;alias parameter may be … [View More]interpreted as header parameter, because the syntax seems to be ambiguous. (cherry picked from commit f5ef15fee0c3bf09adf2506effa0895f72af9034) --- modules_k/nathelper/nathelper.c | 69 +++++++++++++++++++++++++++++++------- 1 files changed, 56 insertions(+), 13 deletions(-) diff --git a/modules_k/nathelper/nathelper.c b/modules_k/nathelper/nathelper.c index 51c1086..52c5e5f 100644 --- a/modules_k/nathelper/nathelper.c +++ b/modules_k/nathelper/nathelper.c @@ -786,7 +786,7 @@ add_contact_alias_f(struct sip_msg* msg, char* str1, char* str2) struct lump *anchor; struct sip_uri uri; struct ip_addr *ip; - char *param, *at, *port, *start; + char *lt, *gt, *param, *at, *port, *start; /* Do nothing if Contact header does not exist */ if (!msg->contact) { @@ -816,25 +816,63 @@ add_contact_alias_f(struct sip_msg* msg, char* str1, char* str2) return 2; } - /* Add alias param */ + /* Check if function has been called already */ if ((c->uri.s < msg->buf) || (c->uri.s > (msg->buf + msg->len))) { LM_ERR("you can't call alias_contact twice, check your config!\n"); return -1; } + + /* Check if Contact URI needs to be enclosed in <>s */ + lt = gt = param = NULL; + at = memchr(msg->contact->body.s, '<', msg->contact->body.len); + if (at == NULL) { + lt = (char*)pkg_malloc(1); + if (!lt) { + LM_ERR("no pkg memory left for lt sign\n"); + goto err; + } + *lt = '<'; + anchor = anchor_lump(msg, msg->contact->body.s - msg->buf, 0, 0); + if (anchor == NULL) { + LM_ERR("anchor_lump for beginning of contact body failed\n"); + goto err; + } + if (insert_new_lump_before(anchor, lt, 1, 0) == 0) { + LM_ERR("insert_new_lump_before for \"<\" failed\n"); + goto err; + } + gt = (char*)pkg_malloc(1); + if (!gt) { + LM_ERR("no pkg memory left for gt sign\n"); + goto err; + } + *gt = '>'; + anchor = anchor_lump(msg, msg->contact->body.s + + msg->contact->body.len - msg->buf, 0, 0); + if (anchor == NULL) { + LM_ERR("anchor_lump for end of contact body failed\n"); + goto err; + } + if (insert_new_lump_before(anchor, gt, 1, 0) == 0) { + LM_ERR("insert_new_lump_before for \">\" failed\n"); + goto err; + } + } + + /* Create ;alias param */ param_len = SALIAS_LEN + IP6_MAX_STR_SIZE + 1 /* ~ */ + 5 /* port */ + 1 /* ~ */ + 1 /* proto */; param = (char*)pkg_malloc(param_len); if (!param) { LM_ERR("no pkg memory left for alias param\n"); - return -1; + goto err; } at = param; append_str(at, SALIAS, SALIAS_LEN); ip_len = ip_addr2sbuf(&(msg->rcv.src_ip), at, param_len - SALIAS_LEN); if (ip_len <= 0) { - pkg_free(param); LM_ERR("failed to copy source ip\n"); - return -1; + goto err; } at = at + ip_len; append_chr(at, '~'); @@ -842,9 +880,8 @@ add_contact_alias_f(struct sip_msg* msg, char* str1, char* str2) append_str(at, port, len); append_chr(at, '~'); if ((msg->rcv.proto < PROTO_UDP) || (msg->rcv.proto > PROTO_SCTP)) { - pkg_free(param); LM_ERR("invalid transport protocol\n"); - return -1; + goto err; } append_chr(at, msg->rcv.proto + '0'); param_len = at - param; @@ -854,18 +891,24 @@ add_contact_alias_f(struct sip_msg* msg, char* str1, char* str2) } else { start = uri.host.s + uri.host.len; } + + /* Add ;alias param */ anchor = anchor_lump(msg, start - msg->buf, 0, 0); if (anchor == NULL) { - pkg_free(param); - LM_ERR("anchor_lump failed\n"); - return -1; + LM_ERR("anchor_lump for ;alias param failed\n"); + goto err; } if (insert_new_lump_after(anchor, param, param_len, 0) == 0) { - LM_ERR("insert_new_lump_after failed\n"); - pkg_free(param); - return -1; + LM_ERR("insert_new_lump_after for ;alias param failed\n"); + goto err; } return 1; + + err: + if (lt) pkg_free(lt); + if (gt) pkg_free(gt); + if (param) pkg_free(param); + return -1; } [View Less]

3 13

git:master: modules/tm: new implementation of load_contacts()/next_contacts ()
by Juha Heinanen 08 Aug '11

08 Aug '11

Module: sip-router Branch: master Commit: e973bbe5e7310861f77b17ce0afaf1cca35fe48a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e973bbe… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Fri May 28 15:38:41 2010 +0300 modules/tm: new implementation of load_contacts()/next_contacts() - Simpler implementation of load_contacts()/next_contacts() functions based on recent changes related to Request-URI handling. … [View More]

2 16

lockup("location") doesn't find a "sip" registration if RURI scheme is "sips"
by Iñaki Baz Castillo 31 May '11

31 May '11

Hi, "sip:alice@domain.org" is registered in Kamailio. If the proxy receives a request for "sips:alice@domain.org" and invokes "lookup(location)" the function doesn't retrieve the registration of alice. But it should: RFC 5630 section 3.3: When used as the Request-URI of a request, the SIPS scheme signifies that each hop over which the request is forwarded, until the request reaches the SIP entity responsible for the domain portion of the Request-URI, must be secured with … [View More]

4 9

question about module function arguments
by Juha Heinanen 22 Apr '11

22 Apr '11

i understood that module functions can now have an arbitrary number of string, int and pseudo var arguments, where string arguments can possibly also contain pseudo variables. some time ago i asked an example on how this is done, but didn't get any reply. so i'll try again. for example, allow_trusted function is now defined like this: {"allow_trusted", (cmd_function)allow_trusted_2, 2, fixup_pvar_pvar, fixup_free_pvar_pvar, REQUEST_ROUTE | FAILURE_ROUTE}, how can i add a third … [View More]

3 17

Moving from 1.3.4
by Andrew O. Zhukov 07 Dec '10

07 Dec '10

Need your suggestion. I'm just comletely satisfied of openser 1.3.4 fuction set. However, it's going down too recently(3-5 times per month) in the last half year. Some of gdb output applyed. Can you suggest me the best stabile version for relocation. Thanks in advance. -------------------------- core.27986 Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 0x0000000000443315 … [View More]in pv_get_authattr (msg=0x2aaaae6685a8, param=0x763958, res=0x7fff701d7860) at pvar.c:1319 1319 res->rs.s =((auth_body_t*)(hdr->parsed))->digest.username.user.s; ------------------ core.29065: Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 0x000000000048e679 in fm_free (qm=0x7389e0, p=<value optimized out>) at mem/f_malloc.c:125 125 if (frag->size <= (*f)->size) break; --------------- core.29063: Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 fm_status (qm=0x7389e0) at mem/f_malloc.c:513 513 size+=f->size,f=f->u.nxt_free,i++,j++){ -------------------------- core.27985: Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 fm_status (qm=0x7389e0) at mem/f_malloc.c:513 513 size+=f->size,f=f->u.nxt_free,i++,j++){ (gdb) print i $1 = 383 (gdb) print j $2 = 1 (gdb) print f $3 = (struct fm_frag *) 0x6366373364393531 (gdb) print size $4 = 3631135185266580216 (gdb) print f->u.nxt_free Cannot access memory at address 0x6366373364393539 (gdb) print f->size Cannot access memory at address 0x6366373364393531 ------------------------------------------ core.29790: Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 fm_malloc (qm=0x7389e0, size=<value optimized out>) at mem/f_malloc.c:267 267 if ((*f)->size>=size) goto found; (gdb) print size $1 = <value optimized out> (gdb) print *f $2 = (struct fm_frag *) 0x12 ----------------------------------------- core.2156: Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 fm_status (qm=0x7389e0) at mem/f_malloc.c:513 513 size+=f->size,f=f->u.nxt_free,i++,j++){ (gdb) print size $1 = 4268969821966590315 (gdb) print f $2 = (struct fm_frag *) 0x373365383d676174 (gdb) print i $3 = 0 (gdb) print j $4 = 1 ---------------------------------------------------------------------------- core.2157: Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 sip_msg_cloner (org_msg=0x770a50, sip_msg_len=0x7fff9430d31c) at sip_msg.c:327 327 for(prm=via->param_lst;prm;prm=prm->next) ------------------------------------------------------- core.32225 Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 0x00002aaaacbf78fb in acc_log_request (rq=0x2aaaae4fd0e8) at acc.c:223 223 memcpy(p, val_arr[i].s, val_arr[i].len); (gdb) print i $1 = 8 (gdb) print val_arr[i].len $2 = 6 (gdb) print val_arr[i].s $3 = 0x5e2c34 <Address 0x5e2c34 out of bounds> (gdb) print p $4 = 0x0 ---------------------------- core.21832 Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 0x00002aaaacbf78fb in acc_log_request (rq=0x2aaaae72baa0) at acc.c:223 223 memcpy(p, val_arr[i].s, val_arr[i].len); (gdb) print p $1 = 0x0 (gdb) print i $2 = 8 (gdb) print val_arr[i].s $3 = 0x636084 <Address 0x636084 out of bounds> (gdb) print val_arr[i].len $4 = 6 ------------------- core.24808: Core was generated by `openser -P /var/run/openser/openser.pid -m 32 -u openser -g openser -w /var/run'. Program terminated with signal 11, Segmentation fault. #0 fm_status (qm=0x7389e0) at mem/f_malloc.c:513 513 size+=f->size,f=f->u.nxt_free,i++,j++){ (gdb) print size $1 = 8317892 (gdb) print f $2 = (struct fm_frag *) 0xc (gdb) print i $3 = 367 (gdb) print j $4 = 1 (gdb) q -- Andrew O. Zhukov Telegroup Ukraine Technical director. Phone 380-44-2308228 Cell 380-67-4017256 Fax 380-44-2386027 [View Less]

3 2

Releasing Kamailio 3.1.1
by Daniel-Constantin Mierla 02 Dec '10

02 Dec '10

Hello, there are almost 2 months since release of Kamailio 3.1.0, so it is time for 3.1.1 as there were several important fixes done. I propose Thursday, next week, by that time we should sort out most of the issues reported lately. Other opinions? Cheers, Daniel -- Daniel-Constantin Mierla Kamailio (OpenSER) Advanced Training Jan 24-26, 2011, Irvine, CA, USA http://www.asipto.com

1 2

git:kamailio_3.0: drouting: do not call init child for PROC_INIT
by Daniel-Constantin Mierla 30 Nov '10

30 Nov '10

Module: sip-router Branch: kamailio_3.0 Commit: 405466638c9cc6231614979f2e89e500d2a7a2fc URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=4054666… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Nov 30 23:16:19 2010 +0100 drouting: do not call init child for PROC_INIT - avoid opening a database connection before forking - reported by several people in context of Postgress async … [View More]

1 0

git:3.1: drouting: do not call init child for PROC_INIT
by Daniel-Constantin Mierla 30 Nov '10

30 Nov '10

Module: sip-router Branch: 3.1 Commit: 1e204f4bbcc84271f509838182539476577fc77a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1e204f4… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Nov 30 23:16:19 2010 +0100 drouting: do not call init child for PROC_INIT - avoid opening a database connection before forking - reported by several people in context of Postgress async lib … [View More]

1 0

git:master: drouting: do not call init child for PROC_INIT
by Daniel-Constantin Mierla 30 Nov '10

30 Nov '10

Module: sip-router Branch: master Commit: 907936449b2543966a5b3b2fe821d406c2293799 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9079364… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Nov 30 23:16:19 2010 +0100 drouting: do not call init child for PROC_INIT - avoid opening a database connection before forking - reported by several people in context of Postgress async lib … [View More]

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev November 2010