sr-dev

sr-dev@lists.kamailio.org

15 participants
44239 discussions

by jh＠tutpro.com

i have twinkle registered from behind nat over tcp and another sip phone registered over udp using the same aor and q value. this aor is then called by another sip phone and sr forks the invite: Oct 28 17:58:37 localhost /usr/sbin/sip-proxy[28989]: INFO: Routing first INVITE to <sip:foo@192.168.0.169:5074;transport=tcp> and <<sip:JzlUogbN0_q5BaWAN1Zv@xx.xx.xx.xx>;q=0> ... twinkle answers and sends 200 ok. 200 ok is received by the calling phone, which then sends ack to twinkle via sr, the problem is that twinkle never receives the ack. instead sr reports a tcp error: Oct 28 17:58:46 localhost /usr/sbin/sip-proxy[28994]: INFO: Routing in-dialog ACK from <sip:foo.bar@foo.bar> to <sip:foo@192.168.0.169:5074;transport=TCP> Oct 28 17:58:46 lohi /usr/sbin/sip-proxy[28994]: WARNING: <core> [tcp_main.c:1200]: WARNING: tcp_do_connect 192.168.0.169:5074: could not find corresponding listening socket for 192.X.Y.2, using default... ... Oct 28 17:58:51 localhost /usr/sbin/sip-proxy[29140]: ERROR: <core> [tcp_main.c:3747]: connect 192.168.0.169:5074 failed (timeout) weird thing about the warning is that it mentions ip address 192.X.Y.2, which is not any of the ip addresses sr has configured to listen at. 192.X.Y.2 is the ip address of eth0 of sr host, but sr is listening at 192.X.Y.10, which is address of interface eth0:1. am i missing some magic from sr tcp config or what could be the reason that sr is trying to find non-existing listening socket? i do have wireshark capture of this if it helps. -- juha

15 years, 2 months

git:sr_3.0: core: added id to every field of gw table schema

by Juha Heinanen

Module: sip-router Branch: sr_3.0 Commit: a3a3ce31042a025fe93520350e53b049d5ef850d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a3a3ce3… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Fri Nov 6 05:58:59 2009 +0200 core: added id to every field of gw table schema --- lib/srdb1/schema/gw.xml | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/lib/srdb1/schema/gw.xml b/lib/srdb1/schema/gw.xml index bb2c354..a457719 100644 --- a/lib/srdb1/schema/gw.xml +++ b/lib/srdb1/schema/gw.xml @@ -47,14 +47,14 @@ <description>ID of gateway's group</description> </column> - <column> + <column id="ip_addr"> <name>ip_addr</name> <type>string</type> <size>15</size> <description>IP Address of the gateway</description> </column> - <column> + <column id="hostname"> <name>hostname</name> <type>string</type> <size>64</size> @@ -62,35 +62,35 @@ <description>Hostname of the gateway</description> </column> - <column> + <column id="port"> <name>port</name> <type>unsigned short</type> <null/> <description>Port of the gateway</description> </column> - <column> + <column id="uri_scheme"> <name>uri_scheme</name> <type>unsigned char</type> <null/> <description>URI scheme of gateway</description> </column> - <column> + <column id="transport"> <name>transport</name> <type>unsigned char</type> <null/> <description>Transport type to be used for the gateway</description> </column> - <column> + <column id="strip"> <name>strip</name> <type>unsigned char</type> <null/> <description>The number of digits to strip from Request URI user part before inserting tag</description> </column> - <column> + <column id="tag"> <name>tag</name> <type>string</type> <size>16</size> @@ -99,21 +99,21 @@ <description>Request URI user part tag</description> </column> - <column> + <column id="weight"> <name>weight</name> <type>unsigned int</type> <null/> <description>Weight of gateway within gw_grp. Valid values are 1-254.</description> </column> - <column> + <column id="flags"> <name>flags</name> <type>unsigned int</type> <default>&DEFAULT_FLAGS;</default> <description>Gateway specific flags</description> </column> - <column> + <column id="defunct"> <name>defunct</name> <type>unsigned int</type> <null/>

15 years, 2 months

default memdbg and memlog

by Klaus Darilion

Hi! IMO it would be better to increase the default value of memdbg and memlog to 4 (L_DBG+1) to avoid the memory log messages in debug=3 mode. I often use debug=3 to hunt down script or sip-router errors and I am not interested in memory debugging. I think this confuses many new sip-router users too. regards klaus

15 years, 2 months

git:sr_3.0: tcp: use the forced socket ip as source

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: 6fcba958595c5189c1b9d97c8fc4c9f434a3c7ad URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6fcba95… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Nov 5 16:21:35 2009 +0100 tcp: use the forced socket ip as source - if a socket is forced always use the forced socket ip as source (if a connection exists with different ip as source, open a new one with the forced ip). --- forward.h | 23 +++++++++++++++++++++-- 1 files changed, 21 insertions(+), 2 deletions(-) diff --git a/forward.h b/forward.h index 2a6e91e..b365753 100644 --- a/forward.h +++ b/forward.h @@ -123,6 +123,11 @@ static inline int msg_send(struct dest_info* dst, char* buf, int len) { struct dest_info new_dst; str outb; +#ifdef USE_TCP + union sockaddr_union* from; + union sockaddr_union local_addr; +#endif + outb.s = buf; outb.len = len; sr_event_exec(SREV_NET_DATA_OUT, (void*)&outb); @@ -152,7 +157,14 @@ static inline int msg_send(struct dest_info* dst, char* buf, int len) " support is disabled\n"); goto error; }else{ - if (unlikely(tcp_send(dst, 0, outb.s, outb.len)<0)){ + from=0; + if (unlikely((dst->send_flags & SND_F_FORCE_SOCKET) && + dst->send_sock)) { + local_addr=dst->send_sock->su; + su_setport(&local_addr, 0); /* any local port will do */ + from=&local_addr; + } + if (unlikely(tcp_send(dst, from, outb.s, outb.len)<0)){ STATS_TX_DROPS; LOG(L_ERR, "msg_send: ERROR: tcp_send failed\n"); goto error; @@ -167,7 +179,14 @@ static inline int msg_send(struct dest_info* dst, char* buf, int len) " support is disabled\n"); goto error; }else{ - if (unlikely(tcp_send(dst, 0, outb.s, outb.len)<0)){ + from=0; + if (unlikely((dst->send_flags & SND_F_FORCE_SOCKET) && + dst->send_sock)) { + local_addr=dst->send_sock->su; + su_setport(&local_addr, 0); /* any local port will do */ + from=&local_addr; + } + if (unlikely(tcp_send(dst, from, outb.s, outb.len)<0)){ STATS_TX_DROPS; LOG(L_ERR, "msg_send: ERROR: tcp_send failed\n"); goto error;

15 years, 2 months

db_flatstore debian packages overlap

by Klaus Darilion

Hi! Bug: db_flatstore is in db_modules package and ser standard package I think it should be removed from db_modules packages as it does not have any dependencies on db-libraries. regards klaus

15 years, 2 months

git:sr_3.0: kex: use set_force_socket()

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: 097bdc69d58956e1678b1d6776d60187b079b731 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=097bdc6… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Nov 5 15:14:31 2009 +0100 kex: use set_force_socket() - use set_force_socket() instead of msg->force_send_socket --- modules_k/kex/km_core.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules_k/kex/km_core.c b/modules_k/kex/km_core.c index 72e2dde..0abb673 100644 --- a/modules_k/kex/km_core.c +++ b/modules_k/kex/km_core.c @@ -44,7 +44,7 @@ int w_km_append_branch(struct sip_msg *msg, char *uri, str *sq) ret = km_append_branch(msg, 0, &msg->dst_uri, &msg->path_vec, q, branch_flags, msg->force_send_socket); /* reset all branch info */ - msg->force_send_socket = 0; + reset_force_socket(msg); setbflagsval(0, 0); if(msg->dst_uri.s!=0) pkg_free(msg->dst_uri.s);

15 years, 2 months

git:sr_3.0: rr(k): use set_force_socket()

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: bae241c28017b9a764876aee942065b941f28922 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=bae241c… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Nov 5 15:16:44 2009 +0100 rr(k): use set_force_socket() - use set_force_socket() instead of msg->force_send_socket --- modules_k/rr/loose.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules_k/rr/loose.c b/modules_k/rr/loose.c index ad85bf0..ab2f006 100644 --- a/modules_k/rr/loose.c +++ b/modules_k/rr/loose.c @@ -597,7 +597,7 @@ static inline int after_strict(struct sip_msg* _m) * send interface the one advertise in second Route */ si = grep_sock_info( &puri.host, puri.port_no, puri.proto); if (si) { - _m->force_send_socket = si; + set_force_socket(_m, si); } else { LM_WARN("no socket found for match second RR\n"); } @@ -813,7 +813,7 @@ static inline int after_loose(struct sip_msg* _m, int preloaded) } si = grep_sock_info( &puri.host, puri.port_no, puri.proto); if (si) { - _m->force_send_socket = si; + set_force_socket(_m, si); } else { LM_WARN("no socket found for match second RR\n"); }

15 years, 2 months

git:sr_3.0: rr(s): use set_force_socket()

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: fbd87475909465547dbba5cdccb5fceeca61178f URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=fbd8747… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Nov 5 15:12:52 2009 +0100 rr(s): use set_force_socket() - use set_force_socket() instead of msg->force_send_socket --- modules_s/rr/loose.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/modules_s/rr/loose.c b/modules_s/rr/loose.c index 33803fe..0ba9337 100644 --- a/modules_s/rr/loose.c +++ b/modules_s/rr/loose.c @@ -901,9 +901,9 @@ static inline int after_loose(struct sip_msg* _m, struct sip_uri* _pru, int _rou LOG(L_ERR, "after_loose: Error while parsing the second route header\n"); return RR_ERROR; } - _m->force_send_socket = grep_sock_info(&parsed_uri.host, - parsed_uri.port_no, - parsed_uri.proto); + set_force_socket(_m, grep_sock_info(&parsed_uri.host, + parsed_uri.port_no, + parsed_uri.proto) ); if (_m->force_send_socket == 0) LOG(L_WARN, "after_loose: send socket cannot be set" " based on the second route header\n");

15 years, 2 months

git:sr_3.0: tmx: pv_t_copy_msg: copy also the send flags

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: 2c3aaa7aad0907a23e7f5d5bb3e743d25f734faf URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=2c3aaa7… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Nov 5 15:19:29 2009 +0100 tmx: pv_t_copy_msg: copy also the send flags --- modules_k/tmx/t_var.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/modules_k/tmx/t_var.c b/modules_k/tmx/t_var.c index e54e99f..6441763 100644 --- a/modules_k/tmx/t_var.c +++ b/modules_k/tmx/t_var.c @@ -52,6 +52,8 @@ int pv_t_copy_msg(struct sip_msg *src, struct sip_msg *dst) dst->set_global_address=src->set_global_address; dst->set_global_port=src->set_global_port; dst->flags = src->flags; + dst->fwd_send_flags = src->fwd_send_flags; + dst->rpl_send_flags = src->rpl_send_flags; dst->force_send_socket = src->force_send_socket; if (parse_msg(dst->buf, dst->len, dst)!=0)

15 years, 2 months

git:sr_3.0: domainpolicy: use set_force_socket()

by Andrei Pelinescu-Onciul

Module: sip-router Branch: sr_3.0 Commit: 67b7508a1c8964abbf55f3f09363a45f8aa871f8 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=67b7508… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Nov 5 15:13:36 2009 +0100 domainpolicy: use set_force_socket() - use set_force_socket() instead of msg->force_send_socket --- modules_k/domainpolicy/domainpolicy.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules_k/domainpolicy/domainpolicy.c b/modules_k/domainpolicy/domainpolicy.c index ee8cc29..f77095e 100644 --- a/modules_k/domainpolicy/domainpolicy.c +++ b/modules_k/domainpolicy/domainpolicy.c @@ -850,7 +850,7 @@ int dp_apply_policy(struct sip_msg* _msg, char* _s1, char* _s2) { } si = grep_sock_info( &host, (unsigned short) port, (unsigned short) proto); if (si) { - _msg->force_send_socket = si; + set_force_socket(_msg, si); } else { LM_WARN("could not find socket for" "send_socket '%.*s'\n", val.s.len, ZSW(val.s.s));

15 years, 2 months

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev