sr-dev May 2013

sr-dev@lists.kamailio.org

29 participants
318 discussions

unREGISTER not working on recent master (built Saturday evening) when usrloc db_mode = 3
by Peter Dunkley 20 May '13

20 May '13

This un-REGISTER: REGISTER sip:...domain... SIP/2.0 Via: SIP/2.0/TCP 10.244.155.159:5080;branch=z9hG4bKc449.b2edda67c9b4a8c76a60a96e2855f29f.0 Via: SIP/2.0/WSS 18vku8fabcfq.invalid;rport=50434;received=90.152.0.102;branch=z9hG4bK5064405 Max-Forwards: 16 To: <sip:...user...@...domain...> From: <sip:...user...@...domain...>;tag=ksmq223j0u Call-ID: pv465kef31j1l3qagh2bu7 CSeq: 84 REGISTER Contact: <sip:pufj2a59@18vku8fabcfq.invalid;transport=ws>;reg-id=1;+sip.instance="<urn:uuid:d098383b-5d6d-4816-9c6e-f1aa00b7cef5>";expires=0 Supported: path, outbound, gruu Content-Length: 0 Path: <sip:7n4oJa+O7yfY4QYK9JufAbtamABmxQI=@10.244.155.159:5080;transport=tcp;lr;ob> Results in this 200 OK (with 0 bindings): SIP/2.0 200 OK Via: SIP/2.0/TCP 10.244.155.159:5080;branch=z9hG4bKc449.b2edda67c9b4a8c76a60a96e2855f29f.0 Via: SIP/2.0/WSS 18vku8fabcfq.invalid;rport=50434;received=90.152.0.102;branch=z9hG4bK5064405 To: <sip:...user...@...domain...>;tag=8ab1eb0246bb313f703b656487ebc696.9519 From: <sip:...user...@...domain...>;tag=ksmq223j0u Call-ID: pv465kef31j1l3qagh2bu7 CSeq: 84 REGISTER Path: <sip:7n4oJa+O7yfY4QYK9JufAbtamABmxQI=@10.244.155.159:5080;transport=tcp;lr;ob> Supported: outbound Require: outbound Content-Length: 0 But with usrloc db_mode set to 3 the corresponding record remains in the database. When I set usrloc db_mode to 0 I can use "mi ul_dump" with kamcmd and see that the records are removed from the hash-table. It looks like some recent change to the way usrloc interacts with the database has broken something. Regards, Peter

3 12

git:master: modules/usrloc: fixed unregister bug in db_mode=3
by Juha Heinanen 20 May '13

20 May '13

Module: sip-router Branch: master Commit: 2669197352bfc20bc916c00f74ac14380f8a62f8 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=2669197… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Mon May 20 21:03:30 2013 +0300 modules/usrloc: fixed unregister bug in db_mode=3 - Commit ce1d16ce1c8009918c294307de53f35378868b52 introduced unregister but that appeared in db_mode=3 when db_ops_ruid param is not set. - Reported by Peter Dunkley. --- modules/usrloc/ucontact.c | 42 ++++++++++++++++++++---------------------- 1 files changed, 20 insertions(+), 22 deletions(-) diff --git a/modules/usrloc/ucontact.c b/modules/usrloc/ucontact.c index 593bb1e..c8da8ec 100644 --- a/modules/usrloc/ucontact.c +++ b/modules/usrloc/ucontact.c @@ -1061,28 +1061,26 @@ int db_delete_ucontact_addr(ucontact_t* _c) vals[n].val.str_val = _c->callid; n++; - if (ul_xavp_contact_name.s) { - if (use_domain) { - keys[n] = &domain_col; - vals[n].type = DB1_STR; - vals[n].nul = 0; - dom = memchr(_c->aor->s, '@', _c->aor->len); - if (dom==0) { - vals[0].val.str_val.len = 0; - vals[n].val.str_val = *_c->aor; - } else { - vals[0].val.str_val.len = dom - _c->aor->s; - vals[n].val.str_val.s = dom + 1; - vals[n].val.str_val.len = _c->aor->s + - _c->aor->len - dom - 1; - } - uldb_delete_attrs(_c->domain, &vals[0].val.str_val, - &vals[n].val.str_val, &_c->ruid); - n++; - } else { - uldb_delete_attrs(_c->domain, &vals[0].val.str_val, - NULL, &_c->ruid); - } + if (use_domain) { + keys[n] = &domain_col; + vals[n].type = DB1_STR; + vals[n].nul = 0; + dom = memchr(_c->aor->s, '@', _c->aor->len); + if (dom==0) { + vals[0].val.str_val.len = 0; + vals[n].val.str_val = *_c->aor; + } else { + vals[0].val.str_val.len = dom - _c->aor->s; + vals[n].val.str_val.s = dom + 1; + vals[n].val.str_val.len = _c->aor->s + + _c->aor->len - dom - 1; + } + uldb_delete_attrs(_c->domain, &vals[0].val.str_val, + &vals[n].val.str_val, &_c->ruid); + n++; + } else { + uldb_delete_attrs(_c->domain, &vals[0].val.str_val, + NULL, &_c->ruid); } if (ul_dbf.use_table(ul_dbh, _c->domain) < 0) {

1 0

git:master: modules/rr: completely reverted after_strict() to its pre-outbound form
by Peter Dunkley 20 May '13

20 May '13

Module: sip-router Branch: master Commit: c9448d9657f0e5792072c6803643c8d9075d711a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=c9448d9… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Mon May 20 18:33:52 2013 +0100 modules/rr: completely reverted after_strict() to its pre-outbound form --- modules/rr/loose.c | 15 +++++---------- 1 files changed, 5 insertions(+), 10 deletions(-) diff --git a/modules/rr/loose.c b/modules/rr/loose.c index c1b669c..bf0080e 100644 --- a/modules/rr/loose.c +++ b/modules/rr/loose.c @@ -578,7 +578,6 @@ static inline int after_strict(struct sip_msg* _m) char* rem_off; str uri; struct socket_info *si; - int next_is_strict; hdr = _m->route; rt = (rr_t*)hdr->parsed; @@ -594,12 +593,9 @@ static inline int after_strict(struct sip_msg* _m) return RR_ERROR; } - next_is_strict = is_strict(&puri.params); - - if (enable_double_rr && is_2rr(&puri.params) && is_myself(&puri)) { + if ( enable_double_rr && is_2rr(&puri.params) && is_myself(&puri)) { /* double route may occure due different IP and port, so force as * send interface the one advertise in second Route */ - si = grep_sock_info( &puri.host, puri.port_no, puri.proto); if (si) { set_force_socket(_m, si); @@ -634,8 +630,7 @@ static inline int after_strict(struct sip_msg* _m) LM_ERR("failed to parse URI\n"); return RR_ERROR; } - next_is_strict = is_strict(&puri.params); - } + } /* set the hooks for the param * important note: RURI is already parsed by the above function, so @@ -643,7 +638,7 @@ static inline int after_strict(struct sip_msg* _m) routed_msg_id = _m->id; routed_params = _m->parsed_uri.params; - if (next_is_strict) { + if (is_strict(&puri.params)) { LM_DBG("Next hop: '%.*s' is strict router\n", uri.len, ZSW(uri.s)); /* Previous hop was a strict router and the next hop is strict * router too. There is no need to save R-URI again because it @@ -707,7 +702,7 @@ static inline int after_strict(struct sip_msg* _m) if (res < 0) { LM_ERR("searching for last Route URI failed\n"); return RR_ERROR; - } else if (res > 0) { + } else if (res > 0) { /* No remote target is an error */ return RR_ERROR; } @@ -731,7 +726,7 @@ static inline int after_strict(struct sip_msg* _m) if (prev) { rem_off = prev->nameaddr.name.s + prev->len; rem_len = rt->nameaddr.name.s + rt->len - rem_off; - } else { + } else { rem_off = hdr->name.s; rem_len = hdr->len; }

1 0

[tracker] Comment added: tm:branch-failure event_route is not executed on all branch failures
by sip-router 20 May '13

20 May '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#296 - tm:branch-failure event_route is not executed on all branch failures User who did this - Juha Heinanen (jh) ---------- i don't use append_branch and i don't have corex module in my config and thus cannot test with it. i tested a simpler case, where only $ru and $du are set similar to this: request_route { xlog("===== main route block\n"); $ru = "sip:foo@bar.com"; $du = "sip:127.0.0.1:9;transport=tcp"; t_on_failure("x"); t_on_branch_failure("x"); set_forward_no_connect(); t_relay(); $var(rc) = $rc; xlog("===== t relay return code: $rc\n"); exit; } and got: May 20 19:21:56 siika /usr/sbin/sip-proxy[20734]: INFO: activating failure and branch failure routes May 20 19:21:56 siika /usr/sbin/sip-proxy[20734]: ERROR: <core> [tcp_main.c:2781]: tcpconn_1st_send(): connect 127.0.0.1:9 failed (RST) Connection refused May 20 19:21:56 siika /usr/sbin/sip-proxy[20734]: ERROR: <core> [tcp_main.c:2791]: tcpconn_1st_send(): 127.0.0.1:9: connect & send for 0x7ff2f7b63030 failed: Connection refused (111) May 20 19:21:56 siika /usr/sbin/sip-proxy[20734]: ERROR: tm [../../forward.h:216]: msg_send(): msg_send: ERROR: tcp_send failed May 20 19:21:56 siika /usr/sbin/sip-proxy[20734]: ERROR: tm [t_fwd.c:1592]: t_send_branch(): ERROR: t_send_branch: sending request on branch 0 failed May 20 19:21:56 siika /usr/sbin/sip-proxy[20734]: ERROR: tm [t_fwd.c:1595]: t_send_branch(): putting branch 0 on hold May 20 19:21:56 siika /usr/sbin/sip-proxy[20734]: INFO: t_relay() failed can you test with the above simpler config and see what you get? ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=296#comment881 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: tm:branch-failure event_route is not executed on all branch failures
by sip-router 20 May '13

20 May '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#296 - tm:branch-failure event_route is not executed on all branch failures User who did this - Daniel-Constantin Mierla (miconda) ---------- I tested again - t_relay() returns 1. I have two invalid branches -- here is my config: <code> request_route { xlog("===== main route block\n"); t_on_failure("x"); t_on_branch_failure("x"); set_forward_no_connect(); # t_relay_to("tcp:127.0.0.1:9"); $du = "sip:127.0.0.1:9;transport=tcp"; append_branch(); $du = "sip:127.0.0.1:10;transport=tcp"; t_relay(); $var(rc) = $rc; xlog("===== t relay return code: $rc\n"); exit; } failure_route[x] { xlog("==== failure route block\n"); exit; } event_route[tm:branch-failure:x] { xlog("==== branch failure route block\n"); } </code> And the logs I get: 1(72046) ERROR: tm [../../forward.h:216]: msg_send(): msg_send: ERROR: tcp_send failed 1(72046) DEBUG: tm [t_fwd.c:1573]: t_send_branch(): t_send_branch: send to 127.0.0.1:10 (2) failed 1(72046) ERROR: tm [t_fwd.c:1592]: t_send_branch(): ERROR: t_send_branch: sending request on branch 0 failed 1(72046) ERROR: tm [t_fwd.c:1595]: t_send_branch(): putting branch 0 on hold 1(72046) ERROR: tm [../../forward.h:216]: msg_send(): msg_send: ERROR: tcp_send failed 1(72046) DEBUG: tm [t_fwd.c:1573]: t_send_branch(): t_send_branch: send to 127.0.0.1:9 (2) failed 1(72046) ERROR: tm [t_fwd.c:1592]: t_send_branch(): ERROR: t_send_branch: sending request on branch 1 failed 1(72046) ERROR: tm [t_fwd.c:1595]: t_send_branch(): putting branch 1 on hold 1(72046) DEBUG: tm [t_funcs.c:384]: t_relay_to(): SER: new transaction fwd'ed 1(72046) ERROR: <script>: ===== t relay return code: 1 1(72046) DEBUG: <core> [usr_avp.c:644]: destroy_avp_list(): DEBUG:destroy_avp_list: destroying list 0x0 1(72046) DEBUG: <core> [usr_avp.c:644]: destroy_avp_list(): DEBUG:destroy_avp_list: destroying list 0x0 1(72046) DEBUG: <core> [usr_avp.c:644]: destroy_avp_list(): DEBUG:destroy_avp_list: destroying list 0x0 1(72046) DEBUG: <core> [usr_avp.c:644]: destroy_avp_list(): DEBUG:destroy_avp_list: destroying list 0x0 1(72046) DEBUG: <core> [usr_avp.c:644]: destroy_avp_list(): DEBUG:destroy_avp_list: destroying list 0x0 1(72046) DEBUG: <core> [usr_avp.c:644]: destroy_avp_list(): DEBUG:destroy_avp_list: destroying list 0x0 1(72046) DEBUG: <core> [xavp.c:448]: xavp_destroy_list(): destroying xavp list 0x0 1(72046) DEBUG: <core> [receive.c:293]: receive_msg(): receive_msg: cleaning up ... 3(72048) DEBUG: tm [t_reply.c:1289]: t_should_relay_response(): ->>>>>>>>> T_code=100, new_code=408 3(72048) DEBUG: tm [t_reply.c:1806]: relay_reply(): DEBUG: relay_reply: branch=0, save=1, relay=-1 icode=908 3(72048) DEBUG: tm [t_reply.c:1289]: t_should_relay_response(): ->>>>>>>>> T_code=100, new_code=408 3(72048) ERROR: <script>: ==== failure route block 3(72048) DEBUG: tm [t_reply.c:1806]: relay_reply(): DEBUG: relay_reply: branch=1, save=0, relay=0 icode=908 3(72048) DEBUG: <core> [msg_translator.c:204]: check_via_address(): check_via_address(127.0.0.1, 192.168.178.8, 0) ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=296#comment880 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: uac: Added ability to set Call-Id through $uac_req(callid).
by Konstantin Mosesov 20 May '13

20 May '13

Module: sip-router Branch: master Commit: 121c5a49f81effdac42977d716332215d6a95e4f URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=121c5a4… Author: Konstantin Mosesov <ez(a)voipgroup.org.ua> Committer: Konstantin Mosesov <ez(a)voipgroup.org.ua> Date: Fri May 17 21:14:29 2013 +0300 uac: Added ability to set Call-Id through $uac_req(callid). --- modules/tm/uac.c | 5 ++++- modules/tm/uac.h | 1 + modules/uac/README | 1 + modules/uac/doc/uac_admin.xml | 1 + modules/uac/uac_send.c | 27 +++++++++++++++++++++++++++ 5 files changed, 34 insertions(+), 1 deletions(-) diff --git a/modules/tm/uac.c b/modules/tm/uac.c index c1f8256..38ead71 100644 --- a/modules/tm/uac.c +++ b/modules/tm/uac.c @@ -762,7 +762,10 @@ int request(uac_req_t *uac_r, str* ruri, str* to, str* from, str *next_hop) if (check_params(uac_r, to, from) < 0) goto err; - generate_callid(&callid); + if (uac_r->callid == NULL || uac_r->callid->len <= 0) + generate_callid(&callid); + else + callid = *uac_r->callid; generate_fromtag(&fromtag, &callid); if (new_dlg_uac(&callid, &fromtag, DEFAULT_CSEQ, from, to, &dialog) < 0) { diff --git a/modules/tm/uac.h b/modules/tm/uac.h index 619d43e..69376b4 100644 --- a/modules/tm/uac.h +++ b/modules/tm/uac.h @@ -58,6 +58,7 @@ typedef struct uac_req { int cb_flags; transaction_cb *cb; void *cbp; + str *callid; } uac_req_t; /* macro for setting the values of uac_req_t struct */ diff --git a/modules/uac/README b/modules/uac/README index cd5cf5c..fc3846d 100644 --- a/modules/uac/README +++ b/modules/uac/README @@ -605,6 +605,7 @@ $uac_req(method)="OPTIONS"; $uac_req(ruri)="sip:kamailio.org"; $uac_req(furi)="sip:kamailio.org"; $uac_req(turi)="sip:kamailio.org"; +$uac_req(callid)=$(mb{s.md5}); uac_req_send(); ... diff --git a/modules/uac/doc/uac_admin.xml b/modules/uac/doc/uac_admin.xml index ec86950..d3a3c54 100644 --- a/modules/uac/doc/uac_admin.xml +++ b/modules/uac/doc/uac_admin.xml @@ -713,6 +713,7 @@ $uac_req(method)="OPTIONS"; $uac_req(ruri)="sip:kamailio.org"; $uac_req(furi)="sip:kamailio.org"; $uac_req(turi)="sip:kamailio.org"; +$uac_req(callid)=$(mb{s.md5}); uac_req_send(); ... </programlisting> diff --git a/modules/uac/uac_send.c b/modules/uac/uac_send.c index 50bff25..91ad56c 100644 --- a/modules/uac/uac_send.c +++ b/modules/uac/uac_send.c @@ -61,6 +61,8 @@ typedef struct _uac_send_info { char b_apasswd[64]; str s_apasswd; unsigned int onreply; + char b_callid[128]; + str s_callid; } uac_send_info_t; static struct _uac_send_info _uac_req; @@ -84,6 +86,7 @@ uac_send_info_t *uac_send_info_clone(uac_send_info_t *ur) tp->s_ouri.s = tp->b_ouri; tp->s_auser.s = tp->b_auser; tp->s_apasswd.s = tp->b_apasswd; + tp->s_callid.s = tp->b_callid; return tp; } @@ -134,6 +137,10 @@ int pv_get_uac_req(struct sip_msg *msg, pv_param_t *param, if(_uac_req.s_apasswd.len<=0) return pv_get_null(msg, param, res); return pv_get_strval(msg, param, res, &_uac_req.s_apasswd); + case 11: + if(_uac_req.s_callid.len<=0) + return pv_get_null(msg, param, res); + return pv_get_strval(msg, param, res, &_uac_req.s_callid); default: return pv_get_uintval(msg, param, res, _uac_req.flags); } @@ -160,6 +167,7 @@ int pv_set_uac_req(struct sip_msg* msg, pv_param_t *param, _uac_req.s_body.len = 0; _uac_req.s_method.len = 0; _uac_req.onreply = 0; + _uac_req.s_callid.len = 0; } break; case 1: @@ -360,6 +368,21 @@ int pv_set_uac_req(struct sip_msg* msg, pv_param_t *param, _uac_req.s_apasswd.s[val->rs.len] = '\0'; _uac_req.s_apasswd.len = val->rs.len; break; + case 11: + if(val==NULL) + { + _uac_req.s_callid.len = 0; + return 0; + } + if(!(val->flags&PV_VAL_STR)) + { + LM_ERR("Invalid value type\n"); + return -1; + } + memcpy(_uac_req.s_callid.s, val->rs.s, val->rs.len); + _uac_req.s_callid.s[val->rs.len] = '\0'; + _uac_req.s_callid.len = val->rs.len; + break; } return 0; } @@ -399,6 +422,8 @@ int pv_parse_uac_req_name(pv_spec_p sp, str *in) case 6: if(strncmp(in->s, "method", 6)==0) sp->pvp.pvn.u.isname.name.n = 7; + else if(strncmp(in->s, "callid", 6)==0) + sp->pvp.pvn.u.isname.name.n = 11; else goto error; break; case 7: @@ -439,6 +464,7 @@ void uac_req_init(void) _uac_req.s_method.s = _uac_req.b_method; _uac_req.s_auser.s = _uac_req.b_auser; _uac_req.s_apasswd.s = _uac_req.b_apasswd; + _uac_req.s_callid.s = _uac_req.b_callid; return; } @@ -612,6 +638,7 @@ int uac_req_send(struct sip_msg *msg, char *s1, char *s2) /* Callback parameter */ uac_r.cbp = (void*)tp; } + uac_r.callid = (_uac_req.s_callid.len <= 0) ? NULL : &_uac_req.s_callid; ret = tmb.t_request(&uac_r, /* UAC Req */ &_uac_req.s_ruri, /* Request-URI */ (_uac_req.s_turi.len<=0)?&_uac_req.s_ruri:&_uac_req.s_turi, /* To */

2 1

[tracker] Comment added: tm:branch-failure event_route is not executed on all branch failures
by sip-router 20 May '13

20 May '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#296 - tm:branch-failure event_route is not executed on all branch failures User who did this - Juha Heinanen (jh) ---------- i added xlog info after t_relay() and got: May 20 09:08:02 siika /usr/sbin/sip-proxy[15173]: INFO: activating contact_failure failure route May 20 09:08:02 siika /usr/sbin/sip-proxy[15173]: INFO: activating contact branch_failure route May 20 09:08:02 siika /usr/sbin/sip-proxy[15173]: INFO: Routing INVITE <sip:jh@192.98.102.10:5044;transport=tcp> to contact via <sip:192.98.102.10:43560;transport=TCP> May 20 09:08:02 siika /usr/sbin/sip-proxy[15173]: ERROR: tm [../../forward.h:216]: msg_send(): msg_send: ERROR: tcp_send failed May 20 09:08:02 siika /usr/sbin/sip-proxy[15173]: ERROR: tm [t_fwd.c:1592]: t_send_branch(): ERROR: t_send_branch: sending request on branch 0 failed May 20 09:08:02 siika /usr/sbin/sip-proxy[15173]: ERROR: tm [t_fwd.c:1595]: t_send_branch(): putting branch 0 on hold May 20 09:08:02 siika /usr/sbin/sip-proxy[15173]: INFO: t_relay() failed that explains why 408 comes immediately, because i send it from script if t_relay() fails and there are no more contacts left. what i understood from this commit. t_relay() should not anymore fail if failure and branch failure routes are set when tcp destination is dead, but it does. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=296#comment879 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: tm:branch-failure event_route is not executed on all branch failures
by sip-router 20 May '13

20 May '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#296 - tm:branch-failure event_route is not executed on all branch failures User who did this - Juha Heinanen (jh) ---------- i re-build kamailio from latest master and still in my test branch and normal failure route is not executed by t_relay() to dead tcp destination. i get May 20 08:05:21 siika /usr/sbin/sip-proxy[14415]: INFO: activating contact_failure failure route May 20 08:05:21 siika /usr/sbin/sip-proxy[14415]: INFO: activating contact branch_failure route May 20 08:05:21 siika /usr/sbin/sip-proxy[14415]: INFO: Routing INVITE <sip:jh@192.98.102.10:5044;transport=tcp> to contact via <sip:192.98.102.10:43368;transport=TCP> May 20 08:05:21 siika /usr/sbin/sip-proxy[14415]: ERROR: tm [../../forward.h:216]: msg_send(): msg_send: ERROR: tcp_send failed May 20 08:05:21 siika /usr/sbin/sip-proxy[14415]: ERROR: tm [t_fwd.c:1592]: t_send_branch(): ERROR: t_send_branch: sending request on branch 0 failed May 20 08:05:21 siika /usr/sbin/sip-proxy[14415]: ERROR: tm [t_fwd.c:1595]: t_send_branch(): putting branch 0 on hold the branch is put on hold, but never released from there. using the same config and same destination that registered over udp and then died, both branch and normal failure routes do get executed after fr_timer fires. in tcp case, calling party gets 408 immediately, i.e., kamailio does not wait for fr_timer to fire. i don't know what the difference could be. i call t_on_failure(), t_on_branch() and "if (t_relay()) exit;" in route block and t_on_branch_failure() in branch route block. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=296#comment878 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/websocket: Improved module_loaded() check for nathelper and outbound so it only fires when the SIP subprotocol is enabled
by Peter Dunkley 19 May '13

19 May '13

Module: sip-router Branch: master Commit: f0ae598a3e63e62d12a143c6cd42b698ad1f2eed URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f0ae598… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Sun May 19 23:56:06 2013 +0100 modules/websocket: Improved module_loaded() check for nathelper and outbound so it only fires when the SIP subprotocol is enabled --- modules/websocket/ws_mod.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/modules/websocket/ws_mod.c b/modules/websocket/ws_mod.c index ea1e23a..ec5e07c 100644 --- a/modules/websocket/ws_mod.c +++ b/modules/websocket/ws_mod.c @@ -278,7 +278,9 @@ static int mod_init(void) return -1; } - if (!module_loaded("nathelper") && !module_loaded("outbound")) + if (((ws_sub_protocols & SUB_PROTOCOL_SIP) == SUB_PROTOCOL_SIP) + && !module_loaded("nathelper") + && !module_loaded("outbound")) { LM_WARN("neither \"nathelper\" nor \"outbound\" modules are" " loaded. At least one of these is required for correct"

1 0

Kamailio segmentation fault when using dispatcher pings
by Peter Dunkley 19 May '13

19 May '13

Hello, This occurs on code built from master as of last night (18 May). #0 crcitt_string_array (dst=0x7fb402426e21 "", src=0x7fffc1b901f0, size=<value optimized out>) at crc.c:239 #1 0x00007fb4021f5e45 in generate_fromtag (tag=0x7fffc1b901e0, callid=<value optimized out>) at uac.c:142 #2 0x00007fb4021fb17e in request (uac_r=0x7fffc1b90290, ruri=0x7fb3fd6166e0, to=0x7fb3fd6166e0, from=<value optimized out>, next_hop=0x7fb3ff822a80) at uac.c:769 #3 0x00007fb3ff6042ce in ds_check_timer (ticks=<value optimized out>, param=<value optimized out>) at dispatch.c:2430 #4 0x000000000053b437 in compat_old_handler (ti=<value optimized out>, tl=<value optimized out>, data=<value optimized out>) at timer.c:1017 #5 0x000000000053d753 in slow_timer_main () at timer.c:1151 #6 0x000000000046b045 in main_loop () at main.c:1674 #7 0x000000000046cf0c in main (argc=<value optimized out>, argv=<value optimized out>) at main.c:2547 Regards, Peter -- Peter Dunkley Technical Director Crocodile RCS Ltd

2 2

← 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 May 2013