sr-dev January 2014

sr-dev@lists.kamailio.org

27 participants
210 discussions

[tracker] Comment added: Possible memory leak when processing sht_reload
by sip-router 02 Jan '14

02 Jan '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#385 - Possible memory leak when processing sht_reload User who did this - Daniel-Constantin Mierla (miconda) ---------- On a quick look, I found the potential issue. Can you try the latest master branch? Or cherry pick the commit from next link in a stable branch: - http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=53689b5… If all goes fine, I will backport to stable branches. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=385#comment1255 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: htable: use pkg for temporary hash table slots used for reload
by Daniel-Constantin Mierla 02 Jan '14

02 Jan '14

Module: sip-router Branch: master Commit: 53689b55632c6208e6326f1a5f4b3c45da216f43 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=53689b5… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Jan 2 13:22:28 2014 +0100 htable: use pkg for temporary hash table slots used for reload - free the temporary slots after reload, reported in FS#385 --- modules/htable/htable.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/modules/htable/htable.c b/modules/htable/htable.c index 1900ec5..6345b9e 100644 --- a/modules/htable/htable.c +++ b/modules/htable/htable.c @@ -490,7 +490,7 @@ static struct mi_root* ht_mi_reload(struct mi_root* cmd_tree, void* param) return init_mi_tree( 500, "no such hash table", 18); } memcpy(&nht, ht, sizeof(ht_t)); - nht.entries = (ht_entry_t*)shm_malloc(nht.htsize*sizeof(ht_entry_t)); + nht.entries = (ht_entry_t*)pkg_malloc(nht.htsize*sizeof(ht_entry_t)); if(nht.entries == NULL) { ht_db_close_con(); @@ -525,6 +525,7 @@ static struct mi_root* ht_mi_reload(struct mi_root* cmd_tree, void* param) ht_cell_free(it); } } + pkg_free(nht.entries); ht_db_close_con(); return init_mi_tree( 200, MI_OK_S, MI_OK_LEN); }

1 0

[tracker] Comment added: Possible memory leak when processing sht_reload
by sip-router 02 Jan '14

02 Jan '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#385 - Possible memory leak when processing sht_reload User who did this - Daniel-Constantin Mierla (miconda) ---------- The link doesn't point to the kamailio.log.gz, maybe was deleted meanwhile. Can you make it available again? ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=385#comment1254 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: acc: allow pvar as first parameter for functions
by sip-router 02 Jan '14

02 Jan '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#386 - acc: allow pvar as first parameter for functions User who did this - Daniel-Constantin Mierla (miconda) ---------- Few comments: - the comparison to detect if it is a variable should be done against PV_MARKER instead of '$', to be easy to spot/maintain - i think the code can be kept optimized when a static string is provided, so in this case it is parsed only at startup. If it is a variable, then acc_get_param_value() will do the parsing with most of the code inside the IF. If not variable, the parsing is still done in the fixup. As an enhancement, I would allow a dynamic string (e.g., "$var(code) message $var(msg)") as parameter, not only a single variable. That means the fixup should search for PV_MARKER inside the parameter and if found then use pv_parse_format() in fixup and pv_printf_s() in acc_get_param_value(). ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=386#comment1253 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 opened: acc: allow pvar as first parameter for functions (Attachment added)
by sip-router 02 Jan '14

02 Jan '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Víctor Seva (linuxmaniac) Attached to Project - sip-router Summary - acc: allow pvar as first parameter for functions Task Type - Feature Request Category - Module Status - New Assigned To - Víctor Seva Operating System - All Severity - Low Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - Enable using pseudo variables in acc funtions as first parameter. $var(x)="403 Hola mamona"; xlog("L_INFO", "$$var(x):$var(x)\n"); acc_log_request("$var(x)"); $avp(y)="503 Internal Error"; acc_log_request("$avp(y)"); 1(13534) INFO: <script>: $var(x):403 Hola mamona 1(13534) NOTICE: acc [acc.c:315]: acc_log_request(): ACC: request accounted: timestamp=1388661266;method=INVITE;from_tag=76ff7a07-c091-4192-84a0-d56e91fe104f;to_tag=;call_id=r10815e0-bf17-4afa-8412-d9130a793d96(a)213.20.128.35;code=403;reason=Hola mamona;src_user=jiri;src_domain=iptel.org;src_ip=127.0.0.1;dst_ouser=jiri;d… 1(13534) NOTICE: acc [acc.c:315]: acc_log_request(): ACC: request accounted: timestamp=1388661266;method=INVITE;from_tag=76ff7a07-c091-4192-84a0-d56e91fe104f;to_tag=;call_id=r10815e0-bf17-4afa-8412-d9130a793d96(a)213.20.128.35;code=503;reason=Internal Error;src_user=jiri;src_domain=iptel.org;src_ip=127.0.0.1;dst_ouser=jiri;ds… One or more files have been attached. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=386 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: pkg: Add gzcompress module to CentOS spec file
by Hugh Waite 02 Jan '14

02 Jan '14

Module: sip-router Branch: master Commit: 3ca144aed1e550eac919b6c36a5103acdedbe22b URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3ca144a… Author: Hugh Waite <hugh.waite(a)crocodile-rcs.com> Committer: Hugh Waite <hugh.waite(a)crocodile-rcs.com> Date: Sat Dec 21 19:05:37 2013 +0000 pkg: Add gzcompress module to CentOS spec file --- pkg/kamailio/centos/6/kamailio.spec | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/pkg/kamailio/centos/6/kamailio.spec b/pkg/kamailio/centos/6/kamailio.spec index 02345e8..8cdd254 100644 --- a/pkg/kamailio/centos/6/kamailio.spec +++ b/pkg/kamailio/centos/6/kamailio.spec @@ -422,10 +422,10 @@ make make every-module skip_modules="app_mono db_cassandra db_oracle iptrtpproxy \ jabber ndb_cassandra osp" \ group_include="kstandard kautheph kberkeley kcarrierroute kcpl \ - kdnssec kgeoip kims kjava kjson kldap klua kmemcached kmi_xmlrpc \ - kmysql koutbound kperl kpostgres kpresence kpurple kpython kradius \ - kredis ksctp ksnmpstats ksqlite ktls kunixodbc kutils kwebsocket \ - kxml kxmpp" + kdnssec kgeoip kgzcompress kims kjava kjson kldap klua kmemcached \ + kmi_xmlrpc kmysql koutbound kperl kpostgres kpresence kpurple kpython \ + kradius kredis ksctp ksnmpstats ksqlite ktls kunixodbc kutils \ + kwebsocket kxml kxmpp" cd modules/app_java/kamailio_java_folder/java ant cd ../../../.. @@ -440,10 +440,10 @@ make install make install-modules-all skip_modules="app_mono db_cassandra db_oracle \ iptrtpproxy jabber osp" \ group_include="kstandard kautheph kberkeley kcarrierroute kcpl \ - kdnssec kgeoip kims kjava kjson kldap klua kmemcached kmi_xmlrpc \ - kmysql koutbound kperl kpostgres kpresence kpurple kpython kradius \ - kredis ksctp ksnmpstats ksqlite ktls kunixodbc kutils kwebsocket \ - kxml kxmpp" + kdnssec kgeoip kgzcompress kims kjava kjson kldap klua kmemcached \ + kmi_xmlrpc kmysql koutbound kperl kpostgres kpresence kpurple kpython \ + kradius kredis ksctp ksnmpstats ksqlite ktls kunixodbc kutils \ + kwebsocket kxml kxmpp" mkdir -p $RPM_BUILD_ROOT/%{_libdir}/kamailio/java install -m644 modules/app_java/kamailio_java_folder/java/Kamailio.class \

1 0

[tracker] Comment added: Unmet dependency for kamailio-perl-modules
by sip-router 02 Jan '14

02 Jan '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#361 - Unmet dependency for kamailio-perl-modules User who did this - Víctor Seva (linuxmaniac) ---------- I'm working on the creation of the new infrastructure to build the official Debian packages for Kamailio. This issue will be fixed as soon as that is working. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=361#comment1252 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: refresed READMEs for some modules that had xml files updated
by Daniel-Constantin Mierla 02 Jan '14

02 Jan '14

Module: sip-router Branch: master Commit: e7dc20ba0b787a05ab111367f6f8b176f92826b9 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e7dc20b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Jan 2 11:27:02 2014 +0100 modules: refresed READMEs for some modules that had xml files updated --- modules/avp/README | 213 ++++++++++++++++++++++---------------------- modules/avpops/README | 46 +++++----- modules/dmq/README | 24 +++--- modules/htable/README | 40 ++++---- modules/rtpproxy-ng/README | 110 +++++++++++----------- modules/tm/README | 12 ++-- modules/usrloc/README | 3 +- 7 files changed, 222 insertions(+), 226 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=e7d…

1 0

git:master: tmrec: updated the examples with tmrec duration
by Daniel-Constantin Mierla 02 Jan '14

02 Jan '14

Module: sip-router Branch: master Commit: 99d1170469c60206992c083f2ba0e9dc6b580d37 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=99d1170… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Thu Jan 2 11:25:56 2014 +0100 tmrec: updated the examples with tmrec duration - hint to read rfc2445 for duration format --- modules/tmrec/README | 8 ++++---- modules/tmrec/doc/tmrec_admin.xml | 7 ++++--- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/modules/tmrec/README b/modules/tmrec/README index 6cffee0..84b7a56 100644 --- a/modules/tmrec/README +++ b/modules/tmrec/README @@ -138,7 +138,7 @@ modparam("tmrec", "separator", ";") subsequent intervals do not overlap. For non-recurring intervals, durations of any positive length are permitted. Zero-length duration means "forever". Negative-length durations are not - allowed. + allowed. See RFC 2445 for the format of duration. * frequency - can be one of the following values: "daily" - specify repeating periods based on an interval of a day or more; "weekly" - specify repeating periods based on an interval of a week or more; @@ -222,9 +222,9 @@ modparam("tmrec", "separator", ";") Example 1.2. tmrec_match usage ... - if(tmrec_match("20120101T000000|24H|weekly|||SA,SU") + if(tmrec_match("20120101T000000|PT24H|weekly|||SA,SU") xdbg("it is weekend!\n"); - if(tmrec_match("20120101T083000|10H|weekly|||MO,TU,WE,TH,FR") + if(tmrec_match("20120101T083000|PT10H|weekly|||MO,TU,WE,TH,FR") xdbg("it is with working hours\n"); ... @@ -298,7 +298,7 @@ if(is_leap_year("2010")) Example 1.4. time_period_match usage ... -if(time_period_match("wd{1-5} hr{8-16}, wd{1-5} hr{17} min{0-29}")) +if(time_period_match("wd{2-6} hr{8-16}, wd{1-5} hr{17} min{0-29}")) xdbg("Monday to Friday, 8:00 to 17:30\n"); if(time_period_match("weekday { sat sun }, weekday {mo-fr} hr {17-8},wd{mo-wed}h diff --git a/modules/tmrec/doc/tmrec_admin.xml b/modules/tmrec/doc/tmrec_admin.xml index 782b1f1..7a429eb 100644 --- a/modules/tmrec/doc/tmrec_admin.xml +++ b/modules/tmrec/doc/tmrec_admin.xml @@ -127,7 +127,8 @@ modparam("tmrec", "separator", ";") be small enough such that subsequent intervals do not overlap. For non-recurring intervals, durations of any positive length are permitted. Zero-length duration means <quote>forever</quote>. - Negative-length durations are not allowed. + Negative-length durations are not allowed. See RFC 2445 for + the format of duration. </para> </listitem> <listitem> @@ -274,9 +275,9 @@ modparam("tmrec", "separator", ";") <title><function>tmrec_match</function> usage</title> <programlisting format="linespecific"> ... - if(tmrec_match("20120101T000000|24H|weekly|||SA,SU") + if(tmrec_match("20120101T000000|PT24H|weekly|||SA,SU") xdbg("it is weekend!\n"); - if(tmrec_match("20120101T083000|10H|weekly|||MO,TU,WE,TH,FR") + if(tmrec_match("20120101T083000|PT10H|weekly|||MO,TU,WE,TH,FR") xdbg("it is with working hours\n"); ... </programlisting>

1 0

Happy New Year!
by Daniel-Constantin Mierla 02 Jan '14

02 Jan '14

Another year packed in the archive, thanks everyone for filling it with excellent achievements and, along them, keeping Kamailio project moving forward! I expect another wonderful year ahead for the project and I am looking forward to collaborating further within this brilliant community as well as meeting many of you at Kamailio World and other events worldwide. I wish everyone a prosperous and successful 2014! Happy new year! Daniel -- Daniel-Constantin Mierla - http://www.asipto.com http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda

2 1

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2014