sr-dev January 2012

sr-dev@lists.kamailio.org

25 participants
167 discussions

[tracker] Comment added: rtpproxy timeout functionality via dlg_terminate_dlg mi command
by sip-router 31 Jan '12

31 Jan '12

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#196 - rtpproxy timeout functionality via dlg_terminate_dlg mi command User who did this - Daniel-Constantin Mierla (miconda) ---------- Patch to rtpproxy module was applied with condition on U command and to_tag.len>0. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=196#comment494 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: rtpproxy: send timeout socket only for U command
by Daniel-Constantin Mierla 31 Jan '12

31 Jan '12

Module: sip-router Branch: master Commit: 7426c76dfb6750b344e19a632c16374c162f5090 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=7426c76… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jan 31 15:21:46 2012 +0100 rtpproxy: send timeout socket only for U command - optional parameters are not allowed for other commands - patch by Nick Altmann (nikbyte), FS#196 --- modules/rtpproxy/rtpproxy.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules/rtpproxy/rtpproxy.c b/modules/rtpproxy/rtpproxy.c index 93019cd..f72df2a 100644 --- a/modules/rtpproxy/rtpproxy.c +++ b/modules/rtpproxy/rtpproxy.c @@ -2221,7 +2221,7 @@ force_rtp_proxy(struct sip_msg* msg, char* str1, char* str2, int offer, int forc } else { v[3].iov_len = 0; } - if (to_tag.len > 0) { + if (opts.s.s[0] == 'U' && to_tag.len > 0) { iovec_param_count = 18; if (timeout_socket_str.len > 0) { iovec_param_count = 20;

1 0

git:3.2: pkg/kamailio/rpm: version set to 3.2.2 for rpm specs
by Daniel-Constantin Mierla 31 Jan '12

31 Jan '12

Module: sip-router Branch: 3.2 Commit: 8786cb88cdd05b28d48b899b327f5f56b84da8a1 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8786cb8… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jan 31 15:04:11 2012 +0100 pkg/kamailio/rpm: version set to 3.2.2 for rpm specs --- pkg/kamailio/rpm/kamailio.spec-4.1 | 2 +- pkg/kamailio/rpm/kamailio.spec.CenOS | 2 +- pkg/kamailio/rpm/kamailio.spec.SuSE | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/kamailio/rpm/kamailio.spec-4.1 b/pkg/kamailio/rpm/kamailio.spec-4.1 index c513722..f836061 100644 --- a/pkg/kamailio/rpm/kamailio.spec-4.1 +++ b/pkg/kamailio/rpm/kamailio.spec-4.1 @@ -1,5 +1,5 @@ %define name kamailio -%define ver 3.2.1 +%define ver 3.2.2 %define rel 0 %define EXCLUDED_MODULES mysql jabber cpl-c avp_radius auth_radius group_radius uri_radius pa postgres osp tlsops unixodbc diff --git a/pkg/kamailio/rpm/kamailio.spec.CenOS b/pkg/kamailio/rpm/kamailio.spec.CenOS index 835fcee..c749441 100644 --- a/pkg/kamailio/rpm/kamailio.spec.CenOS +++ b/pkg/kamailio/rpm/kamailio.spec.CenOS @@ -1,5 +1,5 @@ %define name kamailio -%define ver 3.2.1 +%define ver 3.2.2 %define rel 0 %define _sharedir %{_prefix}/share diff --git a/pkg/kamailio/rpm/kamailio.spec.SuSE b/pkg/kamailio/rpm/kamailio.spec.SuSE index 3f3ba52..ac59041 100644 --- a/pkg/kamailio/rpm/kamailio.spec.SuSE +++ b/pkg/kamailio/rpm/kamailio.spec.SuSE @@ -1,5 +1,5 @@ %define name kamailio -%define ver 3.2.1 +%define ver 3.2.2 %define rel 0 %define EXCLUDED_MODULES mysql jabber cpl-c auth_radius misc_radius peering postgress pa unixodbc osp tlsops

1 0

git:3.2: pkg/kamailio/deb: version set to 3.2.2 for deb control files
by Daniel-Constantin Mierla 31 Jan '12

31 Jan '12

Module: sip-router Branch: 3.2 Commit: e362cd5aebf88f7de1b2641454b4da95cdf9019f URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e362cd5… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jan 31 15:01:49 2012 +0100 pkg/kamailio/deb: version set to 3.2.2 for deb control files --- pkg/kamailio/deb/debian/changelog | 6 ++++++ pkg/kamailio/deb/lenny/changelog | 6 ++++++ pkg/kamailio/deb/lucid/changelog | 6 ++++++ pkg/kamailio/deb/squeeze/changelog | 6 ++++++ pkg/kamailio/deb/wheezy/changelog | 6 ++++++ 5 files changed, 30 insertions(+), 0 deletions(-) diff --git a/pkg/kamailio/deb/debian/changelog b/pkg/kamailio/deb/debian/changelog index b1a2ee8..81c2151 100644 --- a/pkg/kamailio/deb/debian/changelog +++ b/pkg/kamailio/deb/debian/changelog @@ -1,3 +1,9 @@ +kamailio (3.2.2) unstable; urgency=low + + * update to 3.2.2 from upstream + + -- Daniel-Constantin Mierla <miconda(a)gmail.com> Thu, 31 Jan 2012 14:29:30 +0100 + kamailio (3.2.1) unstable; urgency=low * update to 3.2.1 from upstream diff --git a/pkg/kamailio/deb/lenny/changelog b/pkg/kamailio/deb/lenny/changelog index b1a2ee8..81c2151 100644 --- a/pkg/kamailio/deb/lenny/changelog +++ b/pkg/kamailio/deb/lenny/changelog @@ -1,3 +1,9 @@ +kamailio (3.2.2) unstable; urgency=low + + * update to 3.2.2 from upstream + + -- Daniel-Constantin Mierla <miconda(a)gmail.com> Thu, 31 Jan 2012 14:29:30 +0100 + kamailio (3.2.1) unstable; urgency=low * update to 3.2.1 from upstream diff --git a/pkg/kamailio/deb/lucid/changelog b/pkg/kamailio/deb/lucid/changelog index b1a2ee8..81c2151 100644 --- a/pkg/kamailio/deb/lucid/changelog +++ b/pkg/kamailio/deb/lucid/changelog @@ -1,3 +1,9 @@ +kamailio (3.2.2) unstable; urgency=low + + * update to 3.2.2 from upstream + + -- Daniel-Constantin Mierla <miconda(a)gmail.com> Thu, 31 Jan 2012 14:29:30 +0100 + kamailio (3.2.1) unstable; urgency=low * update to 3.2.1 from upstream diff --git a/pkg/kamailio/deb/squeeze/changelog b/pkg/kamailio/deb/squeeze/changelog index b1a2ee8..81c2151 100644 --- a/pkg/kamailio/deb/squeeze/changelog +++ b/pkg/kamailio/deb/squeeze/changelog @@ -1,3 +1,9 @@ +kamailio (3.2.2) unstable; urgency=low + + * update to 3.2.2 from upstream + + -- Daniel-Constantin Mierla <miconda(a)gmail.com> Thu, 31 Jan 2012 14:29:30 +0100 + kamailio (3.2.1) unstable; urgency=low * update to 3.2.1 from upstream diff --git a/pkg/kamailio/deb/wheezy/changelog b/pkg/kamailio/deb/wheezy/changelog index b1a2ee8..81c2151 100644 --- a/pkg/kamailio/deb/wheezy/changelog +++ b/pkg/kamailio/deb/wheezy/changelog @@ -1,3 +1,9 @@ +kamailio (3.2.2) unstable; urgency=low + + * update to 3.2.2 from upstream + + -- Daniel-Constantin Mierla <miconda(a)gmail.com> Thu, 31 Jan 2012 14:29:30 +0100 + kamailio (3.2.1) unstable; urgency=low * update to 3.2.1 from upstream

1 0

git:3.2: modules/pua Fixed memory leaks in limbxml2
by Daniel-Constantin Mierla 31 Jan '12

31 Jan '12

Module: sip-router Branch: 3.2 Commit: 77d3b19501d7d0e885f18900021fd9d797230448 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=77d3b19… Author: Anca Vamanu <anca.vamanu(a)1and1.ro> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Jan 26 15:27:21 2012 +0200 modules/pua Fixed memory leaks in limbxml2 (cherry picked from commit 68295d124ee2d36f3e8d31eac5a76c4d7c89f5b8) --- modules_k/pua/send_publish.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules_k/pua/send_publish.c b/modules_k/pua/send_publish.c index 3523cbc..75a28a8 100644 --- a/modules_k/pua/send_publish.c +++ b/modules_k/pua/send_publish.c @@ -571,7 +571,7 @@ send_publish: if( body && ret_code) { if(body->s) - free(body->s); + xmlFree(body->s); pkg_free(body); } if(etag.s) @@ -595,7 +595,7 @@ error: if(body&& ret_code) { if(body->s) - free(body->s); + xmlFree(body->s); pkg_free(body); } if(str_hdr)

1 0

git:3.2: utils: free xcap_tree for xcap_auth_status()
by Daniel-Constantin Mierla 31 Jan '12

31 Jan '12

Module: sip-router Branch: 3.2 Commit: 1d8f4787a4d06fabb93f5c834fedfdf07a63a8f8 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1d8f478… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jan 31 12:57:30 2012 +0100 utils: free xcap_tree for xcap_auth_status() - patch by Laura Testi (cherry picked from commit 7dd4828530a70df212cedd298418c1a9facd0326) --- modules/utils/xcap_auth.c | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/modules/utils/xcap_auth.c b/modules/utils/xcap_auth.c index a262684..eb87e7e 100644 --- a/modules/utils/xcap_auth.c +++ b/modules/utils/xcap_auth.c @@ -280,13 +280,16 @@ int pres_watcher_allowed(subs_t* subs) } node= get_rule_node(subs, xcap_tree); - if (node== NULL) + if (node== NULL) { + xmlFreeDoc(xcap_tree); return 0; + } /* process actions */ actions_node = xmlNodeGetChildByName(node, "actions"); if (actions_node == NULL) { LM_DBG("actions_node NULL\n"); + xmlFreeDoc(xcap_tree); return 0; } LM_DBG("actions_node->name= %s\n", actions_node->name); @@ -294,6 +297,7 @@ int pres_watcher_allowed(subs_t* subs) sub_handling_node = xmlNodeGetChildByName(actions_node, "sub-handling"); if (sub_handling_node== NULL) { LM_DBG("sub_handling_node NULL\n"); + xmlFreeDoc(xcap_tree); return 0; } sub_handling = (char*)xmlNodeGetContent(sub_handling_node); @@ -302,6 +306,7 @@ int pres_watcher_allowed(subs_t* subs) if (sub_handling == NULL) { LM_ERR("Couldn't get sub-handling content\n"); + xmlFreeDoc(xcap_tree); return -1; } if (strncmp((char*)sub_handling, "block", 5) == 0) { @@ -324,10 +329,12 @@ int pres_watcher_allowed(subs_t* subs) } else { LM_ERR("unknown subscription handling action\n"); + xmlFreeDoc(xcap_tree); xmlFree(sub_handling); return -1; } + xmlFreeDoc(xcap_tree); xmlFree(sub_handling); return 0;

1 0

git:3.2: modules/pua_xmpp Fixed some memory leaks in libxml on error casses
by Daniel-Constantin Mierla 31 Jan '12

31 Jan '12

Module: sip-router Branch: 3.2 Commit: 74d067429282febe82a8066a2d92e3765cbdd1f4 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=74d0674… Author: Anca Vamanu <anca.vamanu(a)1and1.ro> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Jan 26 15:14:43 2012 +0200 modules/pua_xmpp Fixed some memory leaks in libxml on error casses (cherry picked from commit 3c3cc57f6d4938270efbb89426d80a994be1aaa7) --- modules_k/pua_xmpp/xmpp2simple.c | 17 +++++++++-------- 1 files changed, 9 insertions(+), 8 deletions(-) diff --git a/modules_k/pua_xmpp/xmpp2simple.c b/modules_k/pua_xmpp/xmpp2simple.c index d0bdc1e..f2ae0d1 100644 --- a/modules_k/pua_xmpp/xmpp2simple.c +++ b/modules_k/pua_xmpp/xmpp2simple.c @@ -370,18 +370,19 @@ int build_publish(xmlNodePtr pres_node, int expires) if(resource== NULL) { LM_ERR("no more memory\n"); + xmlFree(uri); return -1; } strcpy(resource, slash+1); slash= '\0'; } pres_uri= euri_xmpp_sip(uri); + xmlFree(uri); if(pres_uri== NULL) { LM_ERR("while encoding xmpp-sip uri\n"); - goto error; - } - xmlFree(uri); + goto error; + } uri_str.s= pres_uri; uri_str.len= strlen(pres_uri); @@ -460,12 +461,12 @@ int presence_subscribe(xmlNodePtr pres_node, int expires,int flag) return -1; } to_uri= duri_xmpp_sip(uri); + xmlFree(uri); if(to_uri== NULL) { LM_ERR("while decoding xmpp--sip uri\n"); goto error; } - xmlFree(uri); to_uri_str.s= to_uri; to_uri_str.len= strlen(to_uri); @@ -476,15 +477,15 @@ int presence_subscribe(xmlNodePtr pres_node, int expires,int flag) goto error; } from_uri= euri_xmpp_sip(uri); + xmlFree(uri); if(from_uri== NULL) { LM_ERR("while encoding xmpp-sip uri\n"); - goto error; - } - xmlFree(uri); + goto error; + } from_uri_str.s= from_uri; from_uri_str.len= strlen(from_uri); - + memset(&subs, 0, sizeof(subs_info_t)); subs.pres_uri= &to_uri_str;

1 0

git:master: utils: free xcap_tree for xcap_auth_status()
by Daniel-Constantin Mierla 31 Jan '12

31 Jan '12

Module: sip-router Branch: master Commit: 7dd4828530a70df212cedd298418c1a9facd0326 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=7dd4828… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Jan 31 12:57:30 2012 +0100 utils: free xcap_tree for xcap_auth_status() - patch by Laura Testi --- modules/utils/xcap_auth.c | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/modules/utils/xcap_auth.c b/modules/utils/xcap_auth.c index a262684..eb87e7e 100644 --- a/modules/utils/xcap_auth.c +++ b/modules/utils/xcap_auth.c @@ -280,13 +280,16 @@ int pres_watcher_allowed(subs_t* subs) } node= get_rule_node(subs, xcap_tree); - if (node== NULL) + if (node== NULL) { + xmlFreeDoc(xcap_tree); return 0; + } /* process actions */ actions_node = xmlNodeGetChildByName(node, "actions"); if (actions_node == NULL) { LM_DBG("actions_node NULL\n"); + xmlFreeDoc(xcap_tree); return 0; } LM_DBG("actions_node->name= %s\n", actions_node->name); @@ -294,6 +297,7 @@ int pres_watcher_allowed(subs_t* subs) sub_handling_node = xmlNodeGetChildByName(actions_node, "sub-handling"); if (sub_handling_node== NULL) { LM_DBG("sub_handling_node NULL\n"); + xmlFreeDoc(xcap_tree); return 0; } sub_handling = (char*)xmlNodeGetContent(sub_handling_node); @@ -302,6 +306,7 @@ int pres_watcher_allowed(subs_t* subs) if (sub_handling == NULL) { LM_ERR("Couldn't get sub-handling content\n"); + xmlFreeDoc(xcap_tree); return -1; } if (strncmp((char*)sub_handling, "block", 5) == 0) { @@ -324,10 +329,12 @@ int pres_watcher_allowed(subs_t* subs) } else { LM_ERR("unknown subscription handling action\n"); + xmlFreeDoc(xcap_tree); xmlFree(sub_handling); return -1; } + xmlFreeDoc(xcap_tree); xmlFree(sub_handling); return 0;

1 0

IETF-RFCs and Diversion header
by Daniel-Constantin Mierla 31 Jan '12

31 Jan '12

Hello, Inaki, I pinged you since you may be more familiar with procedures about all this IETF output, but maybe others here can provide useful information as well. At this moment the parser in Kamailio for Diversion header considers it as with similar format as To header. Diversion (defined in an *Historic* rfc5806) is obsoleted by History-Info(*Standard track* rfc4244). Its definition in rfc5806 is: Diversion = "Diversion" ":" 1# (name-addr *( ";" diversion_params )) and it is like To header. However, a newer *Informational* rfc6044, redefines Diversion as: Diversion = "Diversion" HCOLON diversion-params *(COMMA diversion-params) diversion-params = name-addr *(SEMI (diversion-reason / diversion-counter / diversion-limit / diversion-privacy / diversion-screen / diversion-extension)) So in rfc6044, the sintax allow comma separated bodies for one Diversion header, which was not in rfc5806. Obviously, the current diversion parser throws error, so I guess it has to be updated to accept a format similar to Route headers. My question is, apart of rfc number and the policy more recent (bigger values) obsoletes older (lower value) specification, does the Informational or Historic category to set some extra rules? Btw, are people here still using (or ever used -- I didn't so far) Diversion? Cheers, Daniel -- Daniel-Constantin Mierla -- http://www.asipto.com http://linkedin.com/in/miconda -- http://twitter.com/miconda

3 2

new git commiter: Claudio Furrer
by Henning Westerholt 31 Jan '12

31 Jan '12

Hello, Claudio Furrer <elcaio(a)gmail.com> has now git commit access for his work on gentoo packaging. He create several patches and extensions for our gentoo package files in the last months and also maintain an own unofficial portage overlay tree at http://gpo.zugaina.org/net-misc/sip-router since some time. He just updated the gentoo packages in git master today, so if you're an gentoo user why not just give it a try - he is looking forward to your feedback. Welcome Claudio! Best regards, Henning

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2012