sr-dev October 2010

sr-dev@lists.kamailio.org

33 participants
336 discussions

git:kamailio_3.0: Makefile: version set 3.0.4

by Daniel-Constantin Mierla

Module: sip-router Branch: kamailio_3.0 Commit: bee1ef7e0b2a8a49e70f553843feb4fb271f3aea URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=bee1ef7… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Oct 20 17:13:26 2010 +0200 Makefile: version set 3.0.4 --- Makefile.defs | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile.defs b/Makefile.defs index a6c3a3c..5e75842 100644 --- a/Makefile.defs +++ b/Makefile.defs @@ -154,7 +154,7 @@ INSTALL_FLAVOUR=$(FLAVOUR) #version number VERSION = 3 PATCHLEVEL = 0 -SUBLEVEL = 3 +SUBLEVEL = 4 EXTRAVERSION = # memory debugger switcher

13 years, 6 months

kamailio 1.5.5 and 3.0.4

by Daniel-Constantin Mierla

Hello, Based on our project policy to support officially the last two stable branches, I was thinking of packaging kamailio 1.5.5 to mark its end. New fixes will go to svn if it is the case, but no new release in 1.5.x series will be done. Also, since testing for 3.1.0 brought some fixes to 3.0.x, we should do as well 3.0.4. My proposal is next week, Wednesday, Oct 20. Don't forget to report issues you are aware for these two branches. Cheers, Daniel -- Daniel-Constantin Mierla http://www.asipto.com

13 years, 6 months

cfg_parse_selects crash

by Juha Heinanen

this call caused sr from master branch to crash at startup: t_set_fr("@cfg_get.local.invite_timeout", "@cfg_get.local.gw_timeout"); i had misspelled the name of the configuration variable when i set it at the top of the script. -- juha Oct 19 18:16:37 squeeze /usr/sbin/trunk-proxy[1738]: ERROR: <core> [cfg/cfg_select.c:117]: ERROR: cfg_parse_selects(): unknown variable: local.gw_timeout Oct 19 18:16:37 squeeze /usr/sbin/trunk-proxy[1738]: : <core> [main.c:1519]: could not initialize shared configuration Oct 19 18:16:37 squeeze kernel: [ 338.729036] trunk-proxy[1738]: segfault at 10 ip 081638ad sp bf984d70 error 4 in trunk-proxy[8048000+184000]

13 years, 6 months

git:3.1: cfg framework: segfault during exit is fixed

by Miklos Tirpak

Module: sip-router Branch: 3.1 Commit: a10d3c62d4eb5e18e54564ed6ae354dc0ce9dfd0 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a10d3c6… Author: Miklos Tirpak <miklos(a)iptel.org> Committer: Miklos Tirpak <miklos(a)iptel.org> Date: Wed Oct 20 15:32:18 2010 +0200 cfg framework: segfault during exit is fixed Verify that *cfg_global is not NULL when destroying the configuration. t_set_fr("@cfg_get.local.invite_timeout", "@cfg_get.local.gw_timeout"); caused a segfault during startup when the variable was not defined. Reported by Juha Heinanen (cherry picked from commit d85ff11cf0db4aad859f6dcfd18dfb2011e3d78a) --- cfg/cfg_struct.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/cfg/cfg_struct.c b/cfg/cfg_struct.c index f2f5bfe..11a825d 100644 --- a/cfg/cfg_struct.c +++ b/cfg/cfg_struct.c @@ -333,7 +333,7 @@ void cfg_destroy(void) cfg_ctx_destroy(); /* free the list of groups */ - cfg_destory_groups(cfg_global ? (*cfg_global)->vars : NULL); + cfg_destory_groups((cfg_global && (*cfg_global)) ? (*cfg_global)->vars : NULL); /* free the select list */ cfg_free_selects();

13 years, 6 months

git:master: cfg framework: segfault during exit is fixed

by Miklos Tirpak

Module: sip-router Branch: master Commit: d85ff11cf0db4aad859f6dcfd18dfb2011e3d78a URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d85ff11… Author: Miklos Tirpak <miklos(a)iptel.org> Committer: Miklos Tirpak <miklos(a)iptel.org> Date: Wed Oct 20 15:32:18 2010 +0200 cfg framework: segfault during exit is fixed Verify that *cfg_global is not NULL when destroying the configuration. t_set_fr("@cfg_get.local.invite_timeout", "@cfg_get.local.gw_timeout"); caused a segfault during startup when the variable was not defined. Reported by Juha Heinanen --- cfg/cfg_struct.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/cfg/cfg_struct.c b/cfg/cfg_struct.c index f2f5bfe..11a825d 100644 --- a/cfg/cfg_struct.c +++ b/cfg/cfg_struct.c @@ -333,7 +333,7 @@ void cfg_destroy(void) cfg_ctx_destroy(); /* free the list of groups */ - cfg_destory_groups(cfg_global ? (*cfg_global)->vars : NULL); + cfg_destory_groups((cfg_global && (*cfg_global)) ? (*cfg_global)->vars : NULL); /* free the select list */ cfg_free_selects();

13 years, 6 months

git:master: tm: fixed typo in docs

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 97d4e101219dbb035f2a734a2a1ddea119782f55 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=97d4e10… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Wed Oct 20 12:54:40 2010 +0200 tm: fixed typo in docs - some ingeter parameters related to control of reason header for local cancel were provided as string values instead of integer - reported by Mikko Lehto --- modules/tm/README | 10 +++++----- modules/tm/doc/params.xml | 4 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/modules/tm/README b/modules/tm/README index 318c3bf..4e535bc 100644 --- a/modules/tm/README +++ b/modules/tm/README @@ -8,9 +8,9 @@ Juha Heinanen <jh(a)tutpro.com> - Copyright � 2003 FhG FOKUS + Copyright © 2003 FhG FOKUS - Copyright � 2008 Juha Heinanen + Copyright © 2008 Juha Heinanen Revision History Revision $Revision$ $Date$ __________________________________________________________________ @@ -177,7 +177,7 @@ Juha Heinanen Note Several Kamailio (OpenSER) TM module functionalities are now - implemented in the TMX module: "modules_k/tmx". Check it to see if what + implemented in the TMX module: “modules_k/tmx”. Check it to see if what you are looking for is there. 1.2. Serial Forking Based on Q Value @@ -1149,7 +1149,7 @@ modparam("tm", "failure_reply_mode", 3) Example 38. Set local_cancel_reason parameter ... -modparam("tm", "local_cancel_reason", "0") +modparam("tm", "local_cancel_reason", 0) ... 1.4.39. e2e_cancel_reason (boolean) @@ -1167,7 +1167,7 @@ modparam("tm", "local_cancel_reason", "0") Example 39. Set e2e_cancel_reason parameter ... -modparam("tm", "e2e_cancel_reason", "0") +modparam("tm", "e2e_cancel_reason", 0) ... 1.5. Functions diff --git a/modules/tm/doc/params.xml b/modules/tm/doc/params.xml index 2bd0724..f881af5 100644 --- a/modules/tm/doc/params.xml +++ b/modules/tm/doc/params.xml @@ -1198,7 +1198,7 @@ modparam("tm", "failure_reply_mode", 3) <title>Set <varname>local_cancel_reason</varname> parameter</title> <programlisting> ... -modparam("tm", "local_cancel_reason", "0") +modparam("tm", "local_cancel_reason", 0) ... </programlisting> </example> @@ -1230,7 +1230,7 @@ modparam("tm", "local_cancel_reason", "0") <title>Set <varname>e2e_cancel_reason</varname> parameter</title> <programlisting> ... -modparam("tm", "e2e_cancel_reason", "0") +modparam("tm", "e2e_cancel_reason", 0) ... </programlisting> </example>

13 years, 6 months

git:master: pdbt: add another possible data source for german carrier directory

by Henning Westerholt

Module: sip-router Branch: master Commit: 6a88284a4b312baae97e431e27e4927d5023c16f URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6a88284… Author: Henning Westerholt <henning.westerholt(a)1und1.de> Committer: Henning Westerholt <henning.westerholt(a)1und1.de> Date: Wed Oct 20 09:30:46 2010 +0200 pdbt: add another possible data source for german carrier directory --- utils/pdbt/scripts/get_carrier_names_germany.sh | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/utils/pdbt/scripts/get_carrier_names_germany.sh b/utils/pdbt/scripts/get_carrier_names_germany.sh index bb3bc6f..4ffd7d4 100755 --- a/utils/pdbt/scripts/get_carrier_names_germany.sh +++ b/utils/pdbt/scripts/get_carrier_names_germany.sh @@ -30,3 +30,7 @@ export LANG="C" wget -O - "$url" | recode latin1..utf8 | sed 's/^*.Verzeichnis der Portierungskennungen//' | awk '/<tbody>/, /<\/tbody>/' | tr -d '\r' | tr '\n' '@' | sed 's/<\/table>.*$//' | sed 's/<\/tbody>.*$//' +# probably also possible to use this: +# http://www.bundesnetzagentur.de/cae/servlet/contentblob/156772/publicationF… +# main page (for reference): +# http://www.bundesnetzagentur.de/cln_1932/DE/Sachgebiete/Telekommunikation/R… \ No newline at end of file

13 years, 6 months

Re: [sr-dev] extend check_blacklist() in userblacklist module

by Henning Westerholt

On Monday 18 October 2010, Nicolas Rüger wrote: > [..] > I'd like to add some functionality to the userblacklist module and > therefore I'd like to modify the function > > check_blacklist ([string table]) > > because I need a check for the CALLER's URI _instead_ of the CALLEE's URI. Hello Nicolas, i understand. At the moment this function does not support parameters, but patches are of course welcome. :-) > I want to change that for me, as I do deal with SPIT-prevention and I need > a global_list to be able to block or delay some caller's URIs globally and > whitelist others (like hospitals and stuff) for incoming calls. > [..] > I had a look at "userblacklist.c" and I guess that's the one to patch. My > idea is to implement the changes and name the method "check_blacklist1()". > > Here it seems the the URI that's checked is stored in "req_number". > > [...] > > if ((parse_sip_msg_uri(msg) < 0) || (!msg->parsed_uri.user.s) || > (msg->parsed_uri.user.len > MAXNUMBERLEN)) { LM_ERR("cannot parse msg > URI\n"); > return -1; > } > strncpy(req_number, msg->parsed_uri.user.s, msg->parsed_uri.user.len); > > [...] > > > I guess I have to change "msg->parsed_uri.user.s" to something that returns > the CALLER instead of the CALLEE!? You could indeed change the c-code like this. > I had a look at the struct "sip_uri" (in msg_parser.h) because that's the > type of variable "parsed_uri" and found the following... > [..] > So user is just a string (pointer to first char + length) and so it seems I > have even to replace "parsed_uri" by something else. > > Then I had a look to struct "sip_msg" (in msg_parser.h) as it's the type of > "msg" and next level of abstraction. There I found... > [..] > Another idea is to extend the parameter array of "check_blacklist()" so > that one can pass the URI which has to be checked as an extra parameter. > > What do you think is the better way??? I'd think that adding functionality to parse arbitrary parameter is the better and more flexible way instead of the req_number change you proposed above. Take a look to the check_userblacklist functions to get some ideas for the implementation. > Furthermore I need a function that checks the global list for whitelisted > entries. > > The flag "whitelist" is already present but there's no method to query it, > as "check_blacklist()" returns true, when the prefix is whitelisted and > the same when it's not on the list (so there's no way to distinguish > between both possibilities yet) Maybe you would like to add some functionality like the check_userwhitelist functionality, for the check_blacklist function? Regards, Henning

13 years, 6 months

avpops doc mentions obsolete parameter onreply_avp_mode

by Andrew Pogrebennyk

Hi, avpops module documentation still mentions onreply_avp_mode which is obsolete now: http://kamailio.org/docs/modules/3.1.x/modules/avpops.html <<AVPs are persistent per SIP transaction, being available in "route", "branch_route" and "failure_route". To make them available in "onreply_route" armed via TM module, set "onreply_avp_mode" parameter of TM module (note that in the default "onreply_route", the AVPs of the transaction are not available).>> Please remove the references. Thank you. -- Sincerely, Andrew Pogrebennyk

13 years, 6 months

git:3.1: avpops: doc fixes

by Henning Westerholt

Module: sip-router Branch: 3.1 Commit: 0ff81c3f6e768c84d30ec179e57432fd20e5589d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0ff81c3… Author: Henning Westerholt <henning.westerholt(a)1und1.de> Committer: Henning Westerholt <henning.westerholt(a)1und1.de> Date: Mon Oct 18 17:53:06 2010 +0200 avpops: doc fixes - remove obselete TM reference, rephrase text a bit - reported from Andrew Pogrebennyk, andrew dot pogrebennyk at portaone dot com (cherry picked from commit 741521e4dfc236934eb5f28454e49ab574f0a494) --- modules/avpops/README | 57 ++++++++++++++++------------------- modules/avpops/doc/avpops_admin.xml | 11 +----- 2 files changed, 28 insertions(+), 40 deletions(-) diff --git a/modules/avpops/README b/modules/avpops/README index 35a36cb..a12fd2b 100644 --- a/modules/avpops/README +++ b/modules/avpops/README @@ -10,7 +10,7 @@ Ramona-Elena Modroiu <ramona(a)rosdev.ro> - Copyright � 2004, 2005 voice-system.ro + Copyright © 2004, 2005 voice-system.ro Revision History Revision $Revision$ $Date$ __________________________________________________________________ @@ -136,14 +136,9 @@ Chapter 1. Admin Guide an AVP. AVPs are persistent per SIP transaction, being available in "route", - "branch_route" and "failure_route". To make them available in - "onreply_route" armed via TM module, set "onreply_avp_mode" parameter - of TM module (note that in the default "onreply_route", the AVPs of the - transaction are not available). - - An up-to-date tutorial providing more information (detailed - explanations and commented examples) can be found on the SIP-router web - site. + "branch_route" and "failure_route". A tutorial providing more + information (detailed explanations and commented examples) can be found + on the SIP-router web site. 2. Dependencies @@ -246,7 +241,7 @@ modparam("avpops","use_domain",1) Name of column containing the uuid (unique user id). - Default value is "uuid". + Default value is “uuid”. Example 1.5. Set uuid_column parameter ... @@ -257,7 +252,7 @@ modparam("avpops","uuid_column","uuid") Name of column containing the username. - Default value is "username". + Default value is “username”. Example 1.6. Set username_column parameter ... @@ -268,7 +263,7 @@ modparam("avpops","username_column","username") Name of column containing the domain name. - Default value is "domain". + Default value is “domain”. Example 1.7. Set domain_column parameter ... @@ -279,7 +274,7 @@ modparam("avpops","domain_column","domain") Name of column containing the attribute name (AVP name). - Default value is "attribute". + Default value is “attribute”. Example 1.8. Set attribute_column parameter ... @@ -290,7 +285,7 @@ modparam("avpops","attribute_column","attribute") Name of column containing the AVP value. - Default value is "value". + Default value is “value”. Example 1.9. Set value_column parameter ... @@ -301,7 +296,7 @@ modparam("avpops","value_column","value") Name of integer column containing the AVP type. - Default value is "type". + Default value is “type”. Possible column values are * 0 - AVP with string name and string value @@ -329,7 +324,7 @@ modparam("avpops","type_column","type") + 'value_type='('integer'|'string') + 'table='string - Default value is "NULL". + Default value is “NULL”. Example 1.11. Set db_scheme parameter ... @@ -353,7 +348,7 @@ modparam("avpops","db_scheme", 5.12. is_avp_set(name) 5.13. avp_print() -5.1. avp_db_load(source,name) +5.1. avp_db_load(source,name) Loads from DB into memory the AVPs corresponding to the given source. If given, it sets the script flags for loaded AVPs. It returns true if @@ -391,7 +386,7 @@ avp_db_load("$uuid","$avp(s:404fwd)/fwd_table"); avp_db_load("$ru","$avp(i1:123)/$some_scheme"); ... -5.2. avp_db_store(source,name) +5.2. avp_db_store(source,name) Stores to DB the AVPs corresponding to the given source. @@ -407,7 +402,7 @@ avp_db_store("$tu","$avp(i:678)"); avp_db_store("$ru/username","$avp(email)"); ... -5.3. avp_db_delete(source,name) +5.3. avp_db_delete(source,name) Deletes from DB the AVPs corresponding to the given source. @@ -424,7 +419,7 @@ avp_db_delete("$ru/username","$avp(email)"); avp_db_delete("$uuid","$avp(s:404fwd)/fwd_table"); ... -5.4. avp_db_query(query[,dest]) +5.4. avp_db_query(query[,dest]) Make a database query and store the result in AVPs. @@ -438,8 +433,8 @@ avp_db_delete("$uuid","$avp(s:404fwd)/fwd_table"); used in the query makes you vulnerable to SQL injection, e.g. make it possible for an outside attacker to alter your database content. * dest - a list with AVP names where to store the result. The format - is "$avp(name1);$avp(name2);...". If this parameter is ommited, the - result is stored in "$avp(i:1);$avp(i:2);...". If the result gives + is “$avp(name1);$avp(name2);...”. If this parameter is ommited, the + result is stored in “$avp(i:1);$avp(i:2);...”. If the result gives many rows, then multiple AVPs with corresponding name will be added. The value type of the AVP (string or integer) will be derived from the type of the columns. Please note that only this @@ -462,7 +457,7 @@ avp_db_query("select password, ha1 from subscriber where username='$tu'", avp_db_query("delete from subscriber"); ... -5.5. avp_delete(name) +5.5. avp_delete(name) Deletes from memory the AVPs with name or, if empty, all AVPs. @@ -483,7 +478,7 @@ avp_delete("i"); avp_delete("a3"); ... -5.6. avp_pushto(destination,name) +5.6. avp_pushto(destination,name) Pushes the value of AVP(s) into the SIP message. @@ -513,7 +508,7 @@ avp_pushto("$du","$avp(i:679)"); avp_pushto("$br","$avp(i:680)"); ... -5.7. avp_check(name,op_value) +5.7. avp_check(name,op_value) Checks the value of the AVP(s) against an operator and value. @@ -555,7 +550,7 @@ avp_check("$avp(s:foo)","re/sip:.*@bar.net/g"); avp_check("$avp(s:foo)","fm/$avp(fm_avp)/g"); ... -5.8. avp_copy(old_name,new_name) +5.8. avp_copy(old_name,new_name) Copy / move an avp under a new name. @@ -576,7 +571,7 @@ avp_copy("$avp(i:678)", "$avp(s:345)/g"); avp_copy("$avp(old)","$avp(new)/gd"); ... -5.9. avp_printf(dest, format) +5.9. avp_printf(dest, format) NOTE: since Kamailio 1.3.0 the function has been moved to core and it is an alias to pv_printf(). @@ -600,7 +595,7 @@ avp_copy("$avp(old)","$avp(new)/gd"); avp_printf("$avp(i:20)", "This is a $rm request with call-id $hdr(call-id)"); ... -5.10. avp_subst(avps, subst) +5.10. avp_subst(avps, subst) Perl/sed-like subst applied to AVPs having string value. @@ -642,7 +637,7 @@ avp_subst("$avp(i:678)/$avp(i:679)/g", "/(.*)@(.*)/\1@$rd/"); after the first src_avp is processed, it will be added in avp list and next processing will use it. -5.11. avp_op(name,op_value) +5.11. avp_op(name,op_value) Different integer operations with avps. @@ -674,7 +669,7 @@ avp_op("$avp(i:678)", "add/i:345/g"); avp_op("$avp(number)","sub/$avp(number2)/d"); ... -5.12. is_avp_set(name) +5.12. is_avp_set(name) Check if any AVP with name is set. @@ -693,7 +688,7 @@ if(is_avp_set("$avp(i:678)")) log("AVP with integer id 678 exists\n"); ... -5.13. avp_print() +5.13. avp_print() Prints the list with all the AVPs from memory. This is only a helper/debug function. diff --git a/modules/avpops/doc/avpops_admin.xml b/modules/avpops/doc/avpops_admin.xml index e4cee52..9b60cc9 100644 --- a/modules/avpops/doc/avpops_admin.xml +++ b/modules/avpops/doc/avpops_admin.xml @@ -30,15 +30,8 @@ </para> <para> AVPs are persistent per SIP transaction, being available in "route", - "branch_route" and "failure_route". To make them available in - "onreply_route" armed via TM module, set "onreply_avp_mode" parameter - of TM module (note that in the default "onreply_route", the AVPs of - the transaction are not available). - </para> - <para> - An up-to-date tutorial providing more information (detailed - explanations and commented examples) can be found on the SIP-router - web site. + "branch_route" and "failure_route". A tutorial providing more information (detailed + explanations and commented examples) can be found on the SIP-router web site. </para> </section> <section>

13 years, 6 months

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

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev October 2010