sr-dev May 2009

sr-dev@lists.kamailio.org

26 participants
263 discussions

git:master: * Fixed Makefiles of Radius related modules, which did not include

by Juha Heinanen

Module: sip-router Branch: master Commit: ecdcd5fabcfb5c6dada2ca3ee2acb1c92ba21c91 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ecdcd5f… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Fri May 29 09:23:09 2009 +0300 * Fixed Makefiles of Radius related modules, which did not include Makefile.radius. --- modules/auth_radius/Makefile | 5 +++-- modules/misc_radius/Makefile | 9 ++------- modules/peering/Makefile | 5 +++-- 3 files changed, 8 insertions(+), 11 deletions(-) diff --git a/modules/auth_radius/Makefile b/modules/auth_radius/Makefile index b6f0758..4dde7ed 100644 --- a/modules/auth_radius/Makefile +++ b/modules/auth_radius/Makefile @@ -6,13 +6,14 @@ # WARNING: do not run this directly, it should be run by the master Makefile include ../../Makefile.defs +include ../../Makefile.radius + auto_gen= NAME=auth_radius.so -DEFS+=-I$(LOCALBASE)/include -LIBS=-L$(LOCALBASE)/lib $(RADIUS_LIB) DEFS+=-DOPENSER_MOD_INTERFACE SERLIBPATH=../../lib SER_LIBS+=$(SERLIBPATH)/kcore/kcore + include ../../Makefile.modules diff --git a/modules/misc_radius/Makefile b/modules/misc_radius/Makefile index dc10c03..8d10177 100644 --- a/modules/misc_radius/Makefile +++ b/modules/misc_radius/Makefile @@ -3,16 +3,11 @@ # WARNING: do not run this directly, it should be run by the master Makefile include ../../Makefile.defs +include ../../Makefile.radius + auto_gen= NAME=misc_radius.so -# Comment the next two line and uncomment the following two if you want -# to enable Radius support -DEFS+= -DWITH_RADIUS_SUPPORT -I$(LOCALBASE)/include -LIBS=-L$(LOCALBASE)/lib $(RADIUS_LIB) - DEFS+=-DOPENSER_MOD_INTERFACE -SERLIBPATH=../../lib -SER_LIBS+=$(SERLIBPATH)/kcore/kcore include ../../Makefile.modules diff --git a/modules/peering/Makefile b/modules/peering/Makefile index 7b8425a..a6291e3 100644 --- a/modules/peering/Makefile +++ b/modules/peering/Makefile @@ -5,13 +5,14 @@ # WARNING: do not run this directly, it should be run by the master Makefile include ../../Makefile.defs +include ../../Makefile.radius + auto_gen= NAME=peering.so -DEFS+= -DWITH_RADIUS_SUPPORT -I$(LOCALBASE)/include -LIBS=-L$(LOCALBASE)/lib $(RADIUS_LIB) DEFS+=-DOPENSER_MOD_INTERFACE SERLIBPATH=../../lib SER_LIBS+=$(SERLIBPATH)/kcore/kcore + include ../../Makefile.modules

15 years, 6 months

[ openser-Bugs-2798154 ] [dialog] Crash when receiving empry "Record-Route"

by SourceForge.net

Bugs item #2798154, was opened at 2009-05-29 01:30 Message generated for change (Comment added) made by miconda You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2798154&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: None Status: Open Resolution: None Priority: 4 Private: No Submitted By: Iñaki Baz Castillo (ibc_sf) Assigned to: Nobody/Anonymous (nobody) Summary: [dialog] Crash when receiving empry "Record-Route" Initial Comment: Kamailio 1.5 rev 5834. I use 'dialog" module and "uac" module (this one to change From username). Kamailio receives an INVITE, changes From (so addes a long "vsf" parameter to Record-Route) and forwards it to a gateway. The gateway replies 183 and 200 containing a "Record-Route" with no value (empty). Then Kamailio shows this error and crashes: ERROR:core:print_rr_body: failed to parse RR ERROR:dialog:populate_leg_info: failed to print route records ERROR:dialog:dlg_onreply: could not add further info to the dialog CRITICAL:core:receive_fd: EOF on 12 INFO:core:handle_sigs: child process 4505 exited by a signal 11 INFO:core:handle_sigs: core was not generated INFO:core:handle_sigs: terminating due to SIGCHLD INFO:core:sig_usr: signal 15 received I attach the output of the coredump and also the SIP trace showing it. ---------------------------------------------------------------------- >Comment By: Daniel-Constantin Mierla (miconda) Date: 2009-05-29 09:20 Message: Seems similar to what Alex Hermann reported in #2797928. I will check. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2798154&group_…

15 years, 6 months

[ openser-Bugs-2798154 ] [dialog] Crash when receiving empry "Record-Route"

by SourceForge.net

Bugs item #2798154, was opened at 2009-05-29 00:30 Message generated for change (Tracker Item Submitted) made by ibc_sf You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2798154&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: modules Group: None Status: Open Resolution: None Priority: 4 Private: No Submitted By: Iñaki Baz Castillo (ibc_sf) Assigned to: Nobody/Anonymous (nobody) Summary: [dialog] Crash when receiving empry "Record-Route" Initial Comment: Kamailio 1.5 rev 5834. I use 'dialog" module and "uac" module (this one to change From username). Kamailio receives an INVITE, changes From (so addes a long "vsf" parameter to Record-Route) and forwards it to a gateway. The gateway replies 183 and 200 containing a "Record-Route" with no value (empty). Then Kamailio shows this error and crashes: ERROR:core:print_rr_body: failed to parse RR ERROR:dialog:populate_leg_info: failed to print route records ERROR:dialog:dlg_onreply: could not add further info to the dialog CRITICAL:core:receive_fd: EOF on 12 INFO:core:handle_sigs: child process 4505 exited by a signal 11 INFO:core:handle_sigs: core was not generated INFO:core:handle_sigs: terminating due to SIGCHLD INFO:core:sig_usr: signal 15 received I attach the output of the coredump and also the SIP trace showing it. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2798154&group_…

15 years, 6 months

how to change MAIN_NAME?

by jh＠tutpro.com

Makefile.defs has MAIN_NAME=ser how can i change MAIN_NAME to sip-router? could the above be changed to MAIN_NAME=sip-router or MAIN_NAME ?= ser -- juha

15 years, 6 months

git:master: tls: warning fix

by Andrei Pelinescu-Onciul

Module: sip-router Branch: master Commit: d4fc2324bde2fbbdac7ffa3d484a175dbc107bcd URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d4fc232… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu May 28 20:56:18 2009 +0200 tls: warning fix - tls_h_nonblocking_write() commented out for now. --- modules/tls/tls_server.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/modules/tls/tls_server.c b/modules/tls/tls_server.c index 1054438..5516fed 100644 --- a/modules/tls/tls_server.c +++ b/modules/tls/tls_server.c @@ -800,6 +800,7 @@ again: +#if 0 /* not used for now */ /* nonblocking version */ int tls_h_nonblocking_write(struct tcp_connection *c, int fd, const char *buf, unsigned int len) @@ -852,7 +853,7 @@ error: end: return n; } - +#endif /*

15 years, 6 months

git:master: tls: minor fix: obey USE_IPV6

by Andrei Pelinescu-Onciul

Module: sip-router Branch: master Commit: 443c6e55197c12e868e64fd87c29b03ecce2783d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=443c6e5… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu May 28 20:53:56 2009 +0200 tls: minor fix: obey USE_IPV6 - if USE_IPV6 is not defined, don't try to parse/use ipv6 addresses. Reported-by: Juha Heinanen <jh(a)tutpro.com> --- modules/tls/tls_config.c | 11 ++++++++++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/modules/tls/tls_config.c b/modules/tls/tls_config.c index ad40d85..5dbd4c5 100644 --- a/modules/tls/tls_config.c +++ b/modules/tls/tls_config.c @@ -46,7 +46,7 @@ static tls_cfg_t* cfg = NULL; static tls_domain_t* domain = NULL; - +#ifdef USE_IPV6 static int parse_ipv6(struct ip_addr* ip, cfg_token_t* token, cfg_parser_t* st) { @@ -74,6 +74,7 @@ static int parse_ipv6(struct ip_addr* ip, cfg_token_t* token, st->file, token->start.line, token->start.col); return -1; } +#endif /* USE_IPV6 */ static int parse_ipv4(struct ip_addr* ip, cfg_token_t* token, @@ -196,7 +197,15 @@ static int parse_hostport(int* type, struct ip_addr* ip, unsigned int* port, } if (t.type == '[') { +#ifdef USE_IPV6 if (parse_ipv6(ip, &t, st) < 0) return -1; +#else + ERR("%s:%d:%d: IPv6 address not supported (compiled without IPv6" + " support)\n", + st->file, t.start.line, t.start.col); + return -1; + } +#endif /* USE_IPV6 */ } else if (t.type == CFG_TOKEN_ALPHA) { opt = cfg_lookup_token(token_default, &t.val); if (opt) {

15 years, 6 months

git:andrei/to_parser_fix: core: to parser fix

by Andrei Pelinescu-Onciul

Module: sip-router Branch: andrei/to_parser_fix Commit: ac7dd1cc97757e60d3d8e0b52767e89e2caa61be URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ac7dd1c… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu May 28 20:38:37 2009 +0200 core: to parser fix - parse parameters without value in To/From headers (e.g.: To: <sip:nils@invalid.com>;foo or To: <sip:nils@invalid.com>;foo= ). Parameters with an empty value ('=' followed by nothing) will have their value field pointing to en empty string (len==0, but s non-null), while parameters without a value (no '=') will point to a null string (len==0, s==0). - better handling of parameters at the end of the string Reported-and-tested-by: Nils Ohlmeier <nils(a)iptel.org> --- parser/parse_to.c | 68 +++++++++++++++++++++++++++++++++++++--------------- 1 files changed, 48 insertions(+), 20 deletions(-) diff --git a/parser/parse_to.c b/parser/parse_to.c index a493820..a8f3883 100644 --- a/parser/parse_to.c +++ b/parser/parse_to.c @@ -188,16 +188,17 @@ static /*inline*/ char* parse_to_param(char *buffer, char *end, case 0: switch (status) { -#ifndef NO_PINGTEL_TAG_HACK case TAG3: param->type = TAG_PARAM; - param->name.len = 3; + case PARA_NAME: + case TAG1: + case TAG2: + param->name.len = tmp-param->name.s; status = S_EQUAL; case S_EQUAL: case S_PARA_VALUE: saved_status=status; goto endofheader; -#endif case PARA_VALUE_TOKEN: status = E_PARA_VALUE; param->value.len = tmp-param->value.s; @@ -261,23 +262,21 @@ static /*inline*/ char* parse_to_param(char *buffer, char *end, { case PARA_VALUE_QUOTED: break; -#ifndef NO_PINGTEL_TAG_HACK case TAG3: param->type = TAG_PARAM; - param->name.len = 3; + case PARA_NAME: + case TAG1: + case TAG2: + param->name.len = tmp-param->name.s; case S_EQUAL: + param->value.s = 0; + param->value.len = 0; + goto semicolon_add_param; case S_PARA_VALUE: - if (param->type==TAG_PARAM) - param->value.s = tmp; - else { - LOG( L_ERR , "ERROR: parse_to_param : unexpected " - "char [%c] in status %d: <<%.*s>> .\n", - *tmp,status, (int)(tmp-buffer), ZSW(buffer)); - goto error; - } -#endif + param->value.s = tmp; case PARA_VALUE_TOKEN: param->value.len=tmp-param->value.s; +semicolon_add_param: add_param(param,to_b); case E_PARA_VALUE: param = (struct to_param*) @@ -466,18 +465,47 @@ static /*inline*/ char* parse_to_param(char *buffer, char *end, } }/*switch*/ }/*for*/ + if (!(status==F_CR || status==F_LF || status==F_CRLF)) + saved_status=status; endofheader: -#ifndef NO_PINGTEL_TAG_HACK - if (param->type==TAG_PARAM - && (saved_status==S_EQUAL||saved_status==S_PARA_VALUE) ) { - saved_status = E_PARA_VALUE; - param->value.s= 0; + switch(saved_status){ + case TAG3: + param->type = TAG_PARAM; /* tag at the end */ + /* no break */ + case PARA_NAME: + case TAG1: + case TAG2: + param->name.len = tmp-param->name.s; + /* no break */ + case S_EQUAL: + /* parameter without '=', e.g. foo */ + param->value.s=0; param->value.len=0; add_param(param, to_b); + saved_status=E_PARA_VALUE; + break; + case S_PARA_VALUE: + /* parameter with null value, e.g. foo= */ + param->value.s=tmp; + param->value.len=0; + add_param(param, to_b); + saved_status=E_PARA_VALUE; + break; + case PARA_VALUE_TOKEN: + param->value.len=tmp-param->value.s; + add_param(param, to_b); + saved_status=E_PARA_VALUE; + break; + case E_PARA_VALUE: + break; + default: + LOG( L_ERR , "ERROR: parse_to_param : unexpected end of header," + " status %d: <<%.*s>> .\n", + saved_status, (int)(tmp-buffer), ZSW(buffer)); + goto error; } -#endif *returned_status=saved_status; return tmp;

15 years, 6 months

SF.net SVN: openser:[5861] branches/1.3/modules/carrierroute

by Henning Westerholt

Revision: 5861 http://openser.svn.sourceforge.net/openser/?rev=5861&view=rev Author: henningw Date: 2009-05-28 16:50:14 +0000 (Thu, 28 May 2009) Log Message: ----------- - fix one bigger SHM memory leak and a few small PKG leaks in DB error paths - patch from Timo Reimann, timo dot reimann at 1und1 dot de Modified Paths: -------------- branches/1.3/modules/carrierroute/carrier_tree.c branches/1.3/modules/carrierroute/route_db.c This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

15 years, 6 months

[ openser-Bugs-2796454 ] Problems on 'kamctl' script on CR command

by SourceForge.net

Bugs item #2796454, was opened at 2009-05-25 11:35 Message generated for change (Comment added) made by egschubert You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2796454&group_… Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: tools Group: ver devel Status: Open Resolution: None Priority: 5 Private: No Submitted By: Edson - 4lists (egschubert) Assigned to: Nobody/Anonymous (nobody) Summary: Problems on 'kamctl' script on CR command Initial Comment: Just updated to the last SVN: ... Kamailio-151:~/sip-server # kamailio -V version: kamailio 1.5.1-notls (i386/linux) flags: STATISTICS, USE_IPV6, USE_TCP, USE_SCTP, DISABLE_NAGLE, USE_MCAST, SHM_MMAP, PKG_MALLOC, F_MALLOC, FAST_LOCK-ADAPTIVE_WAIT ADAPTIVE_WAIT_LOOPS=1024, MAX_RECV_BUFFER_SIZE 262144, MAX_LISTEN 16, MAX_URI_SIZE 1024, BUF_SIZE 65535, PKG_SIZE 4194304 poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. svnrevision: 2:5858M @(#) $Id: main.c 5608 2009-02-13 16:48:17Z henningw $ main.c compiled on 11:09:12 May 25 2009 with gcc 4.3 And 'kamctl cr' says: ... cr addcarrier <carrier> <scan_prefix> <domain> <rewrite_host> ................ <prob> <strip> <rewrite_prefix> <rewrite_suffix> ............... <flags> <mask> <comment> .........................add a carrier ... But command: ... Kamailio-151:~/sip-server # kamctl cr addcarrier 1 0 2 X.X.X.X 1 1 123456 '' '' '' 'Output_Route_X123' Output_Route_X123 ERROR 1064 (42000) at line 1: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' Rota_de_saida_pelo_IPIPGW)' at line 1 ERROR: cr - SQL Error Kamailio-151:~/sip-server # On script 'kamctl', line 1357: if [ $# -gt 8 ] ; then COMMENT=$9 Edson. ---------------------------------------------------------------------- >Comment By: Edson - 4lists (egschubert) Date: 2009-05-28 12:15 Message: re-sending... ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=743020&aid=2796454&group_…

15 years, 6 months

devel mailing lists to be unified in sr-dev@lists.sip-router.org

by Daniel-Constantin Mierla

Hello, as both SIP Express Router (SER) and Kamailio (OpenSER) projects have now the sources in the same GIT repository hosted by http://sip-router.org, next week the devel lists specific for each project (serdev(a)lists.iptel.org and devel(a)lists.kamailio.org) will be unified with sr-dev(a)lists.sip-router.org. In this way, new commits to the code, discussions about present and future development versions will be in one place. Current ML addresses will be aliased to sr-dev(a)lists.sip-router.org, so they will still be reachable. The archives to the date will stay independent, new registration will be directed to sr-dev. The whole process should be transparent to all existing subscribers, however, if you experience downtime, please be patient a bit, but if the issues last for long time, please report to me. The planned time for migration is Monday/Tuesday next week. . Cheers, Daniel -- Daniel-Constantin Mierla http://www.asipto.com/

15 years, 6 months

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

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev May 2009