sr-dev October 2014

sr-dev@lists.kamailio.org

34 participants
381 discussions

[tracker] Task closed: crash in 4.1.6 with uac_replace_from
by sip-router 28 Oct '14

28 Oct '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#479 - crash in 4.1.6 with uac_replace_from User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Fixed Additional comments about closing: Fixed according to the comment. Re-open if still an issue. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=479 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] Task closed: kamailio -c complains about PKG_MEM size -M parameter
by sip-router 28 Oct '14

28 Oct '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#481 - kamailio -c complains about PKG_MEM size -M parameter User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Fixed Additional comments about closing: RPMs for 4.2 should have this fixed. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=481 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

an issue on terminating dialog for kamailio 4.2.x
by qw 28 Oct '14

28 Oct '14

hi, I install kamailio 4.2.x on centos 6.5 via 'yum'. One endpoint called another, where one dialog is created, and caller and callee can talk with each other. But when the caller hangup the phone(i.e. send bye to the callee), the callee can't receive the bye message, and the dialog can't be terminated normally. I have captured the packets via wireshark, and uploaded kamailio.cfg and the captured files as attachment. In the captured file, PRACK and BYE message can be sent to kamailio server successfully, but kamailio server doesn't forward the two messages to the callee. Why PRACK and BYE message can't be sent to the callee? Looking forward to your help! B.R. Andrew

1 0

git:master: textops: include sdp_helpr_funcs. h to get rid of compile warning
by Daniel-Constantin Mierla 27 Oct '14

27 Oct '14

Module: sip-router Branch: master Commit: 8608da35d224c70d2bd8cfc728f3d07e1ea7a3ce URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8608da3… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Sun Oct 26 16:32:13 2014 +0100 textops: include sdp_helpr_funcs.h to get rid of compile warning - needed for use of get_mixed_part_delimiter() - reported by Juha Heinanen --- modules/textops/textops.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/modules/textops/textops.c b/modules/textops/textops.c index 8bfef16..8e9bcb7 100644 --- a/modules/textops/textops.c +++ b/modules/textops/textops.c @@ -65,6 +65,7 @@ #include "../../parser/parse_content.h" #include "../../parser/parse_param.h" #include "../../parser/sdp/sdp.h" +#include "../../parser/sdp/sdp_helpr_funcs.h" #include "../../lib/kcore/parse_privacy.h" #include "../../msg_translator.h" #include "../../ut.h"

3 2

content-length mismatch for kamailio 4.2.x
by andrew 27 Oct '14

27 Oct '14

hi all, kamailio 4.2.x has been released, And I used two endpoint to test kamailio server. I install kamailio 4.2.x on centos 6.5 via 'yum'. One endpoint failed to call another, I captured packets via wireshark, and found that invite method is recognized as error sip signal because of content-length' mismatch. I examined the invite packets, and calculated content-length in invite signal. There is no mistake for the value of content-length. I guess it may be a bug in kamailio 4.2.x. But I don't know what causes kamailio to report the mismatch response. who can give me help? I have uploaded the captured packets as an attachment. Below are two snapshots from the captured packets. B.R. andrew

1 0

git:4.2: modules/tmx: downgraded info message to debug message
by Juha Heinanen 26 Oct '14

26 Oct '14

Module: sip-router Branch: 4.2 Commit: c46df63add89b5840759fef0137b110fe7c971f2 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=c46df63… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Sun Oct 26 19:11:28 2014 +0200 modules/tmx: downgraded info message to debug message - INFO: tmx [t_var.c:738]: pv_get_t_branch(): branch flags is [32768] (cherry picked from commit b3635962175dae2ba4d2df3e056248d08f0f6ad1) --- modules/tmx/t_var.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules/tmx/t_var.c b/modules/tmx/t_var.c index c95a55b..09fd673 100644 --- a/modules/tmx/t_var.c +++ b/modules/tmx/t_var.c @@ -735,7 +735,7 @@ int pv_get_t_branch(struct sip_msg *msg, pv_param_t *param, } res->ri = t->uac[branch].branch_flags; res->flags = PV_VAL_INT; - LM_INFO("branch flags is [%u]\n", res->ri); + LM_DBG("branch flags is [%u]\n", res->ri); break; default: LM_ERR("unsupported route_type %d\n", get_route_type());

1 0

git:master: modules/tmx: downgraded info message to debug message
by Juha Heinanen 26 Oct '14

26 Oct '14

Module: sip-router Branch: master Commit: b3635962175dae2ba4d2df3e056248d08f0f6ad1 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b363596… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Sun Oct 26 19:11:28 2014 +0200 modules/tmx: downgraded info message to debug message - INFO: tmx [t_var.c:738]: pv_get_t_branch(): branch flags is [32768] --- modules/tmx/t_var.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules/tmx/t_var.c b/modules/tmx/t_var.c index c95a55b..09fd673 100644 --- a/modules/tmx/t_var.c +++ b/modules/tmx/t_var.c @@ -735,7 +735,7 @@ int pv_get_t_branch(struct sip_msg *msg, pv_param_t *param, } res->ri = t->uac[branch].branch_flags; res->flags = PV_VAL_INT; - LM_INFO("branch flags is [%u]\n", res->ri); + LM_DBG("branch flags is [%u]\n", res->ri); break; default: LM_ERR("unsupported route_type %d\n", get_route_type());

1 0

git:4.2: textops: include sdp_helpr_funcs.h to get rid of compile warning
by Juha Heinanen 26 Oct '14

26 Oct '14

Module: sip-router Branch: 4.2 Commit: 8107f39a866817e12dcd5ef5a95dc562fcf121a2 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8107f39… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Sun Oct 26 16:32:13 2014 +0100 textops: include sdp_helpr_funcs.h to get rid of compile warning - needed for use of get_mixed_part_delimiter() - reported by Juha Heinanen (cherry picked from commit 8608da35d224c70d2bd8cfc728f3d07e1ea7a3ce) --- modules/textops/textops.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/modules/textops/textops.c b/modules/textops/textops.c index 8bfef16..8e9bcb7 100644 --- a/modules/textops/textops.c +++ b/modules/textops/textops.c @@ -65,6 +65,7 @@ #include "../../parser/parse_content.h" #include "../../parser/parse_param.h" #include "../../parser/sdp/sdp.h" +#include "../../parser/sdp/sdp_helpr_funcs.h" #include "../../lib/kcore/parse_privacy.h" #include "../../msg_translator.h" #include "../../ut.h"

1 0

please fix recent textops 4.2 commit
by Juha Heinanen 26 Oct '14

26 Oct '14

looks like "git:master: textops: fix append_body_part" commit e71c7f014f1217cb4431f2644e8c4c32cc6544f2 caused compiler warning CC (cc) [M textops.so] textops.o textops.c: In function 'append_multibody_helper': textops.c:1867:2: warning: implicit declaration of function 'get_mixed_part_delimiter' [-Wimplicit-function-declaration] please fix or revert the commit. i need to build 4.2 and don't want to do it until the warning is gone. -- juha

2 1

git:master: modules/pua: added outbound_proxy to struct publ_info
by Juha Heinanen 25 Oct '14

25 Oct '14

Module: sip-router Branch: master Commit: b0d089b685abc51b68b25e90c6021b888cabf051 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b0d089b… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Sat Oct 25 18:46:25 2014 +0300 modules/pua: added outbound_proxy to struct publ_info --- modules/pua/README | 25 ++++++++++++++----------- modules/pua/doc/pua_admin.xml | 3 ++- modules/pua/doc/pua_devel.xml | 2 ++ modules/pua/hash.h | 2 +- modules/pua/send_publish.c | 20 ++++++++++++++++++-- modules/pua/send_publish.h | 1 + 6 files changed, 38 insertions(+), 15 deletions(-) diff --git a/modules/pua/README b/modules/pua/README index c8c7d8a..ed64a3b 100644 --- a/modules/pua/README +++ b/modules/pua/README @@ -8,7 +8,7 @@ Edited by Anca-Maria Vamanu - Copyright � 2006 Voice Sistem SRL + Copyright (c) 2006 Voice Sistem SRL __________________________________________________________________ Table of Contents @@ -186,7 +186,7 @@ modparam("pua", "hash_size", 11) Database url. - Default value is ">mysql://openser:openserrw@localhost/openser". + Default value is ">mysql://kamailio:kamailiorw@localhost/kamailio". Example 1.2. Set db_url parameter ... @@ -243,7 +243,8 @@ modparam("pua", "update_period", 100) 3.7. outbound_proxy (str) - SIP URI of outbound proxy to be used when sending PUBLISH requests. + SIP URI of outbound proxy to be used when sending PUBLISH requests if + no outbound proxy is given in outbound_proxy field of struct publ_info. By default, no outbound proxy has been defined. @@ -334,7 +335,7 @@ modparam("pua", "fetch_rows", 1000) 4.1. pua_update_contact() -4.1. pua_update_contact() +4.1. pua_update_contact() The remote target can be updated by the Contact of a subsequent in dialog request. In the PUA watcher case (sending a SUBSCRIBE messages), @@ -359,7 +360,7 @@ if(method=="NOTIFY") 5.1. pua_cleanup -5.1. pua_cleanup +5.1. pua_cleanup Manually triggers the cleanup functions for the pua table. Useful if you have set update_period to zero or less. @@ -394,7 +395,7 @@ Chapter 2. Developer Guide The module provides the following functions that can be used by other Kamailio modules. -1. bind_pua(pua_api_t* api) +1. bind_pua(pua_api_t* api) This function binds the pua modules and fills the structure with the two exported functions. @@ -410,7 +411,7 @@ typedef struct pua_api { } pua_api_t; ... -2. send_publish +2. send_publish Field type: ... @@ -445,12 +446,14 @@ typedef struct publ_info for that event)*/ str* etag; /* (optional) the value of the etag the request should match */ + str* outbound_proxy;/* outbound_proxy to use when sending the + Publish request */ str* extra_headers /* (optional) extra_headers that should be added to Publish msg*/ }publ_info_t; ... -3. send_subscribe +3. send_subscribe Field type: ... @@ -489,7 +492,7 @@ typedef struct subs_info }subs_info_t; ... -4. is_dialog +4. is_dialog Field type: ... @@ -508,7 +511,7 @@ typedef int (*query_dialog_t)(ua_pres_t* presentity); } ... -5. register_puacb +5. register_puacb Field type: ... @@ -537,7 +540,7 @@ typedef int (*register_puacb_t)(int types, pua_cb f, void* param ); typedef int (pua_cb)(ua_pres_t* hentity, struct sip_msg*); ... -6. add_event +6. add_event Field type: ... diff --git a/modules/pua/doc/pua_admin.xml b/modules/pua/doc/pua_admin.xml index f692786..e7b242d 100644 --- a/modules/pua/doc/pua_admin.xml +++ b/modules/pua/doc/pua_admin.xml @@ -197,7 +197,8 @@ modparam("pua", "update_period", 100) <title><varname>outbound_proxy</varname> (str)</title> <para> SIP URI of outbound proxy to be used when sending - PUBLISH requests. + PUBLISH requests if no outbound proxy is given in + outbound_proxy field of struct publ_info. </para> <para> <emphasis>By default, no outbound proxy has been defined. diff --git a/modules/pua/doc/pua_devel.xml b/modules/pua/doc/pua_devel.xml index 75014b4..d36c8d1 100644 --- a/modules/pua/doc/pua_devel.xml +++ b/modules/pua/doc/pua_devel.xml @@ -86,6 +86,8 @@ typedef struct publ_info for that event)*/ str* etag; /* (optional) the value of the etag the request should match */ + str* outbound_proxy;/* outbound_proxy to use when sending the + Publish request */ str* extra_headers /* (optional) extra_headers that should be added to Publish msg*/ }publ_info_t; diff --git a/modules/pua/hash.h b/modules/pua/hash.h index af32d13..161939b 100644 --- a/modules/pua/hash.h +++ b/modules/pua/hash.h @@ -81,6 +81,7 @@ typedef struct ua_pres{ void* cb_param; struct ua_pres* next; int ua_flag; + str* outbound_proxy; /* croc-not sored in db_only mode */ /* publish */ str etag; @@ -96,7 +97,6 @@ typedef struct ua_pres{ int cseq; int version; /* int watcher_count; croc-nolonger used!! */ - str* outbound_proxy; /* croc-not sored in db_only mode */ str* extra_headers; str record_route; str remote_contact; diff --git a/modules/pua/send_publish.c b/modules/pua/send_publish.c index da74ab7..3ddd5e7 100644 --- a/modules/pua/send_publish.c +++ b/modules/pua/send_publish.c @@ -274,6 +274,7 @@ void publ_cback_func(struct cell *t, int type, struct tmcb_params *ps) publ.content_type= hentity->content_type; publ.id= hentity->id; publ.extra_headers= hentity->extra_headers; + publ.outbound_proxy = hentity->outbound_proxy; publ.cb_param= hentity->cb_param; if (dbmode == PUA_DB_ONLY && pua_dbf.end_transaction) @@ -692,8 +693,9 @@ send_publish: result= tmb.t_request(&uac_r, publ->pres_uri, /*! Request-URI */ publ->pres_uri, /*! To */ - publ->pres_uri, /*! From */ - &outbound_proxy /*! Outbound proxy*/ + publ->pres_uri, /*! From */ + publ->outbound_proxy? + publ->outbound_proxy:&outbound_proxy /*! Outbound proxy*/ ); if(result< 0) @@ -757,6 +759,9 @@ ua_pres_t* publish_cbparam(publ_info_t* publ,str* body,str* tuple_id, size= sizeof(ua_pres_t)+ sizeof(str)+ (publ->pres_uri->len+ + publ->content_type.len+ publ->id.len+ 1)*sizeof(char); + + if(publ->outbound_proxy) + size+= sizeof(str)+ publ->outbound_proxy->len* sizeof(char); if(body && body->s && body->len) size+= sizeof(str)+ body->len* sizeof(char); if(publ->etag) @@ -822,6 +827,16 @@ ua_pres_t* publish_cbparam(publ_info_t* publ,str* body,str* tuple_id, cb_param->extra_headers->len= publ->extra_headers->len; size+= publ->extra_headers->len; } + if(publ->outbound_proxy) + { + cb_param->outbound_proxy = (str*)((char*)cb_param + size); + size += sizeof(str); + cb_param->outbound_proxy->s = (char*)cb_param + size; + memcpy(cb_param->outbound_proxy->s, publ->outbound_proxy->s, + publ->outbound_proxy->len); + cb_param->outbound_proxy->len = publ->outbound_proxy->len; + size+= publ->outbound_proxy->len; + } if(publ->content_type.s && publ->content_type.len) { @@ -837,6 +852,7 @@ ua_pres_t* publish_cbparam(publ_info_t* publ,str* body,str* tuple_id, cb_param->tuple_id.len= tuple_id->len; size+= tuple_id->len; } + cb_param->event= publ->event; cb_param->flag|= publ->source_flag; cb_param->cb_param= publ->cb_param; diff --git a/modules/pua/send_publish.h b/modules/pua/send_publish.h index ef81ee9..667d7aa 100644 --- a/modules/pua/send_publish.h +++ b/modules/pua/send_publish.h @@ -44,6 +44,7 @@ typedef struct publ_info str content_type; /*! the content_type of the body if present(optional if the same as the default value for that event) */ str* etag; + str* outbound_proxy; str* extra_headers; void* cb_param; /*! the parameter for the function to be called on the callback for the received reply; it must be allocated in share memory;

1 0

← Newer
1
...
4
5
6
7
8
9
10
...
39
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev October 2014