sr-dev

sr-dev@lists.kamailio.org

12 participants
44199 discussions

by Maxim Sobolev

sobomax 2009/05/30 02:15:28 CEST SER CVS Repository Modified files: . rtpp_log.c Log: Make it possible to specify syslog facility without the LOG_ prefix, i.e. -d warn:local5. Revision Changes Path 1.3 +3 -2 rtpproxy/rtpp_log.c http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtpp_log.c.diff?r1=1…

15 years, 7 months

CVS:commitlog: rtpproxy rtpproxy.8

by Maxim Sobolev

sobomax 2009/05/30 01:48:19 CEST SER CVS Repository Modified files: . rtpproxy.8 Log: Regen. Revision Changes Path 1.9 +8 -4 rtpproxy/rtpproxy.8 http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtpproxy.8.diff?r1=1…

15 years, 7 months

CVS:commitlog: rtpproxy main.c manpage.xml rtpp_defines.h rtpp_log.c rtpp_log.h

by Maxim Sobolev

sobomax 2009/05/30 01:48:04 CEST SER CVS Repository Modified files: . main.c manpage.xml rtpp_defines.h rtpp_log.c rtpp_log.h Log: Add ability to set syslog facility (by adding it after : to the argument of the -d option), i.e. -d WARN:LOG_LOCAL5. Suggested by: Inaki Baz Castillo Revision Changes Path 1.89 +11 -3 rtpproxy/main.c http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/main.c.diff?r1=1.88&… 1.10 +8 -4 rtpproxy/manpage.xml http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/manpage.xml.diff?r1=… 1.25 +2 -1 rtpproxy/rtpp_defines.h http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtpp_defines.h.diff?… 1.2 +47 -1 rtpproxy/rtpp_log.c http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtpp_log.c.diff?r1=1… 1.7 +2 -1 rtpproxy/rtpp_log.h http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/rtpproxy/rtpp_log.h.diff?r1=1…

15 years, 7 months

script errors

by jh＠tutpro.com

this is my current list of unresolved script errors: - tm module lacks onreply_avp_mode param. how does sr tm behave related to avp visibility? - this kind of if expression results in parse error: if ([$avp(i:712) & 1]) { how to re-write it so that it is accepted? -- juha

15 years, 7 months

git:master: pua: use load_tm_api() instead of find_export()/load_tm()

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: d961145661f0ea20ae799b5cb667d25fe9791a42 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d961145… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri May 29 22:05:03 2009 +0300 pua: use load_tm_api() instead of find_export()/load_tm() --- modules_k/pua/pua.c | 13 ++----------- 1 files changed, 2 insertions(+), 11 deletions(-) diff --git a/modules_k/pua/pua.c b/modules_k/pua/pua.c index 8a9f5f7..46dadd3 100644 --- a/modules_k/pua/pua.c +++ b/modules_k/pua/pua.c @@ -147,8 +147,6 @@ struct module_exports exports= { */ static int mod_init(void) { - load_tm_f load_tm; - LM_DBG("...\n"); if(min_expires< 0) @@ -157,15 +155,8 @@ static int mod_init(void) if(default_expires< 600) default_expires= 3600; - /* import the TM auto-loading function */ - if((load_tm=(load_tm_f)find_export("load_tm", 0, 0))==NULL) - { - LM_ERR("can't import load_tm\n"); - return -1; - } - /* let the auto-loading function load all TM stuff */ - - if(load_tm(&tmb)==-1) + /* load TM API */ + if(load_tm_api(&tmb)==-1) { LM_ERR("can't load tm functions\n"); return -1;

15 years, 7 months

git:master: kex: added K pv_printf() function

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: f5cff79c5d6fa7270c56b1a7ca6e2d6902999c8c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f5cff79… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri May 29 21:21:08 2009 +0300 kex: added K pv_printf() function --- modules_k/kex/kex_mod.c | 2 + modules_k/kex/km_core.c | 74 +++++++++++++++++++++++++++++++++++++++++++++++ modules_k/kex/km_core.h | 2 + 3 files changed, 78 insertions(+), 0 deletions(-) diff --git a/modules_k/kex/kex_mod.c b/modules_k/kex/kex_mod.c index 64f26a8..7fa5830 100644 --- a/modules_k/kex/kex_mod.c +++ b/modules_k/kex/kex_mod.c @@ -74,6 +74,8 @@ static cmd_export_t cmds[]={ 0, ANY_ROUTE }, {"isdsturiset", (cmd_function)w_isdsturiset, 0, 0, 0, ANY_ROUTE }, + {"pv_printf", (cmd_function)w_pv_printf, 2, pv_printf_fixup, + 0, ANY_ROUTE }, {0,0,0,0,0,0} }; diff --git a/modules_k/kex/km_core.c b/modules_k/kex/km_core.c index cea12ea..72e2dde 100644 --- a/modules_k/kex/km_core.c +++ b/modules_k/kex/km_core.c @@ -27,6 +27,8 @@ #include "../../dprint.h" #include "../../dset.h" #include "../../flags.h" +#include "../../pvar.h" +#include "../../lvalue.h" #include "../../mod_fix.h" #include "km_core.h" @@ -95,3 +97,75 @@ int w_isdsturiset(struct sip_msg *msg, char *uri, str *s2) return 1; } +int pv_printf_fixup(void** param, int param_no) +{ + pv_spec_t *spec=NULL; + pv_elem_t *pvmodel=NULL; + str tstr; + + if(param_no==1) + { + spec = (pv_spec_t*)pkg_malloc(sizeof(pv_spec_t)); + if(spec==NULL) + { + LM_ERR("out of pkg\n"); + return -1; + } + memset(spec, 0, sizeof(pv_spec_t)); + tstr.s = (char*)(*param); + tstr.len = strlen(tstr.s); + if(pv_parse_spec(&tstr, spec)==NULL) + { + LM_ERR("unknown script variable in first parameter"); + pkg_free(spec); + return -1; + } + if(!pv_is_w(spec)) + { + LM_ERR("read-only script variable in first parameter"); + pkg_free(spec); + return -1; + } + *param = spec; + } else if(param_no==2) { + pvmodel = 0; + tstr.s = (char*)(*param); + tstr.len = strlen(tstr.s); + if(pv_parse_format(&tstr, &pvmodel)<0) + { + LM_ERR("error in second parameter"); + return -1; + } + *param = pvmodel; + } + return 0; +} + +int w_pv_printf(struct sip_msg *msg, char *s1, str *s2) +{ + pv_spec_t *spec=NULL; + pv_elem_t *model=NULL; + pv_value_t val; + + spec = (pv_spec_t*)s1; + + model = (pv_elem_t*)s2; + + memset(&val, 0, sizeof(pv_value_t)); + if(pv_printf_s(msg, model, &val.rs)!=0) + { + LM_ERR("cannot eval second parameter\n"); + goto error; + } + val.flags = PV_VAL_STR; + if(spec->setf(msg, &spec->pvp, EQ_T, &val)<0) + { + LM_ERR("setting PV failed\n"); + goto error; + } + + return 1; +error: + return -1; +} + diff --git a/modules_k/kex/km_core.h b/modules_k/kex/km_core.h index 84fe5dd..5fa08f8 100644 --- a/modules_k/kex/km_core.h +++ b/modules_k/kex/km_core.h @@ -29,5 +29,7 @@ int w_km_append_branch(struct sip_msg *msg, char *uri, str *s2); int w_setdsturi(struct sip_msg *msg, char *uri, str *s2); int w_resetdsturi(struct sip_msg *msg, char *uri, str *s2); int w_isdsturiset(struct sip_msg *msg, char *uri, str *s2); +int w_pv_printf(struct sip_msg *msg, char *s1, str *s2); +int pv_printf_fixup(void** param, int param_no); #endif

15 years, 7 months

64b - lib or lib64?

by Daniel-Constantin Mierla

Hello, kamailio installs the modules under lib64 on 64b architecture, SR does it in lib. I do not recall exactly the discussion when we did the update in openser, but probably we got somehow to the conclusion that it should be so. Looking now in my deb distro, i see in the system (/ and /usr): lib lib32 lib64 where lib64 is symlink to lib, so I guess SR way is still right. How is in other distros? Opinions? Cheers, Daniel -- Daniel-Constantin Mierla http://www.asipto.com/

15 years, 7 months

git:master: kex: added K dst uri related functions

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 6b9d28ee105960c790dae46119430764c1241b3d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6b9d28e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri May 29 20:03:57 2009 +0300 kex: added K dst uri related functions - setdsturi(uri)/resetdsturi()/isdsturiset() --- modules_k/kex/kex_mod.c | 6 ++++++ modules_k/kex/km_core.c | 30 ++++++++++++++++++++++++++++++ modules_k/kex/km_core.h | 3 +++ 3 files changed, 39 insertions(+), 0 deletions(-) diff --git a/modules_k/kex/kex_mod.c b/modules_k/kex/kex_mod.c index 9baa48d..64f26a8 100644 --- a/modules_k/kex/kex_mod.c +++ b/modules_k/kex/kex_mod.c @@ -68,6 +68,12 @@ static cmd_export_t cmds[]={ 0, REQUEST_ROUTE | FAILURE_ROUTE }, {"km_append_branch", (cmd_function)w_km_append_branch, 1, fixup_spve_null, 0, REQUEST_ROUTE | FAILURE_ROUTE }, + {"setdsturi", (cmd_function)w_setdsturi, 1, 0, + 0, ANY_ROUTE }, + {"resetdsturi", (cmd_function)w_resetdsturi, 0, 0, + 0, ANY_ROUTE }, + {"isdsturiset", (cmd_function)w_isdsturiset, 0, 0, + 0, ANY_ROUTE }, {0,0,0,0,0,0} }; diff --git a/modules_k/kex/km_core.c b/modules_k/kex/km_core.c index beebb35..cea12ea 100644 --- a/modules_k/kex/km_core.c +++ b/modules_k/kex/km_core.c @@ -65,3 +65,33 @@ int w_km_append_branch(struct sip_msg *msg, char *uri, str *sq) return ret; } +int w_setdsturi(struct sip_msg *msg, char *uri, str *s2) +{ + str s; + + /* todo: fixup */ + s.s = uri; + s.len = strlen(uri); + + if(set_dst_uri(msg, &s)!=0) + return -1; + return 1; + +} + +int w_resetdsturi(struct sip_msg *msg, char *uri, str *s2) +{ + if(msg->dst_uri.s!=0) + pkg_free(msg->dst_uri.s); + msg->dst_uri.s = 0; + msg->dst_uri.len = 0; + return 1; +} + +int w_isdsturiset(struct sip_msg *msg, char *uri, str *s2) +{ + if(msg->dst_uri.s==0 || msg->dst_uri.len<=0) + return -1; + return 1; +} + diff --git a/modules_k/kex/km_core.h b/modules_k/kex/km_core.h index a668c71..84fe5dd 100644 --- a/modules_k/kex/km_core.h +++ b/modules_k/kex/km_core.h @@ -26,5 +26,8 @@ #include "../../sr_module.h" int w_km_append_branch(struct sip_msg *msg, char *uri, str *s2); +int w_setdsturi(struct sip_msg *msg, char *uri, str *s2); +int w_resetdsturi(struct sip_msg *msg, char *uri, str *s2); +int w_isdsturiset(struct sip_msg *msg, char *uri, str *s2); #endif

15 years, 7 months

To parser bug

by Nils Ohlmeier

Hello, during testing SER at SIPit 24 I discovered something what I believe is a major bug in the To and From header parser of SER. When you send a request with a To or From header like this: To: <sip:nils@invalid.com>;foo the To header parameter parser fails, and the whole To header is not parsed. According to the BNF of 3261 generic parameters without parameters are allowed in To and From header. So I created the attached patch, which seems to work for me. Please review carefully and let me know if I overlooked something in the problem statement or the bug fix. Cheers Nils Index: parse_to.c =================================================================== RCS file: /cvsroot/ser/sip_router/parser/parse_to.c,v retrieving revision 1.22 diff -a -u -r1.22 parse_to.c --- parse_to.c 23 Jun 2008 17:09:57 -0000 1.22 +++ parse_to.c 20 May 2009 05:04:40 -0000 @@ -122,6 +122,11 @@ case TAG3: param->type=TAG_PARAM; case PARA_NAME: + param->value.len = tmp-param->value.s; + saved_status = E_PARA_VALUE; + status = F_LF; + add_param( param , to_b ); + break; case TAG1: case TAG2: param->name.len = tmp-param->name.s; @@ -161,6 +166,11 @@ case TAG3: param->type=TAG_PARAM; case PARA_NAME: + param->name.len = tmp-param->name.s; + saved_status = E_PARA_VALUE; + status = F_CR; + add_param( param , to_b ); + break; case TAG1: case TAG2: param->name.len = tmp-param->name.s; @@ -277,7 +287,13 @@ } #endif case PARA_VALUE_TOKEN: - param->value.len=tmp-param->value.s; + case PARA_NAME: + if (status == PARA_VALUE_TOKEN) { + param->value.len=tmp-param->value.s; + } + else { + param->name.len = tmp-param->name.s; + } add_param(param,to_b); case E_PARA_VALUE: param = (struct to_param*)

15 years, 7 months

can't anymore build my debian package

by jh＠tutpro.com

could it be possible that the compiler detection fix broke something, because i can't anymore build my debian package: Makefile:365: "make TLS option is obsoleted, try TLS_HOOKS or CORE_TLS" /bin/sh: -c: line 0: syntax error near unexpected token `)' /bin/sh: -c: line 0: `echo CFG_DEFS:= -DNAME=\'\"sip-router\"\' -DVERSION=\'\"2.1.0-dev23-make\"\' -DARCH=\'\"i386\"\' -DOS=\'linux_\' -DOS_QUOTED=\'\"linux\"\' -DCOMPILER=\'\"gcc //g\' -e \'s/^.*$[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*$.*/\1/\' -e \'s/^[^0-9].*$[0-9][0-9]*\.[0-9][0-9]*$.*/\1/\')\"\' -D__CPU_i386 -D__OS_linux -DSER_VER=2001000 -DCFG_DIR=\'\"/etc/sip-router\"\' -DPKG_MALLOC -DSHM_MEM -DSHM_MMAP -DDNS_IP_HACK -DUSE_MCAST -DUSE_TCP -DDISABLE_NAGLE -DHAVE_RESOLV_RES -DUSE_DNS_CACHE -DUSE_DNS_FAILOVER -DUSE_DST_BLACKLIST -DUSE_NAPTR -DF_MALLOC -DUSE_TLS -DTLS_HOOKS -DFAST_LOCK -DADAPTIVE_WAIT -DADAPTIVE_WAIT_LOOPS=1024 -DCC_GCC_LIKE_ASM -DHAVE_GETHOSTBYNAME2 -DHAVE_UNION_SEMUN -DHAVE_SCHED_YIELD -DHAVE_MSG_NOSIGNAL -DHAVE_MSGHDR_MSG_CONTROL -DHAVE_ALLOCA_H -DHAVE_TIMEGM -DHAVE_SCHED_SETSCHEDULER -DHAVE_EPOLL -DHAVE_SIGIO_RT -DSIGINFO64_WORKARROUND -DUSE_FUTEX -DHAVE_SELECT -DUSE_SCTP >>makecfg.lst' make[1]: *** No rule to make target `makecfg.lst', needed by `action.o'. Stop. -- juha

15 years, 7 months

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev