December 2014 - sr-dev - kamailio.org

git:lazedo/4.2.1:57f58f90: kazoo : fork process with tcp option
by lazedo 23 Dec '14

23 Dec '14

Module: kamailio Branch: lazedo/4.2.1 Commit: 57f58f903fae47e30e6cb165db05710394cc9738 URL: https://github.com/kamailio/kamailio/commit/57f58f903fae47e30e6cb165db05710… Author: lazedo <luis.azedo(a)factorlusitano.com> Committer: lazedo <luis.azedo(a)factorlusitano.com> Date: 2014-12-23T00:56:56Z kazoo : fork process with tcp option --- Modified: modules/kazoo/kazoo.c --- Diff: https://github.com/kamailio/kamailio/commit/57f58f903fae47e30e6cb165db05710… Patch: https://github.com/kamailio/kamailio/commit/57f58f903fae47e30e6cb165db05710… --- diff --git a/modules/kazoo/kazoo.c b/modules/kazoo/kazoo.c index 42eafd8..e41a690 100644 --- a/modules/kazoo/kazoo.c +++ b/modules/kazoo/kazoo.c @@ -295,11 +295,11 @@ static int mod_child_init(int rank) if (pid<0) return -1; /* error */ if(pid==0){ - kz_amqp_manager_loop(0); + kz_amqp_manager_loop(1); } else { for(i=0; i < dbk_consumer_processes; i++) { - pid=fork_process(i+2, "AMQP Consumer", 0); + pid=fork_process(i+2, "AMQP Consumer", 1); if (pid<0) return -1; /* error */ if(pid==0){

1 0

[tracker] Task closed: usrloc: REGISTER with new callid gets updated on memory but not on DB
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#494 - usrloc: REGISTER with new callid gets updated on memory but not on DB User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Implemented More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=494 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 closed: our Record-Route is doubled on subsequent branches
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#466 - our Record-Route is doubled on subsequent branches User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Implemented Additional comments about closing: If still an issue, open with the afferent details on github tracker. More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=466 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 closed: add support to auth_db to validate source IP address
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#499 - add support to auth_db to validate source IP address User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Won't implement More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=499 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 closed: Implement "u" rpc formating char
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#64 - Implement "u" rpc formating char User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Implemented Additional comments about closing: u has been implemented for jsonrpc-s, should be propagated to other modules. More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=64 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 closed: crash in dns_cache.c when using natping
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#491 - crash in dns_cache.c when using natping User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Fixed Additional comments about closing: Pushed safety check to master. To be backported soon. More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=491 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:aff1dcd3: core: safety checks on proto variable for dsn resolving
by Daniel-Constantin Mierla 22 Dec '14

22 Dec '14

Module: kamailio Branch: master Commit: aff1dcd3dc63c0fe124fc138d553da0723c615a8 URL: https://github.com/kamailio/kamailio/commit/aff1dcd3dc63c0fe124fc138d553da0… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2014-12-22T21:41:53+01:00 core: safety checks on proto variable for dsn resolving - avoid crashes if functions used inadequately with a null pointer --- Modified: dns_cache.c Modified: msg_translator.c --- Diff: https://github.com/kamailio/kamailio/commit/aff1dcd3dc63c0fe124fc138d553da0… Patch: https://github.com/kamailio/kamailio/commit/aff1dcd3dc63c0fe124fc138d553da0… --- diff --git a/dns_cache.c b/dns_cache.c index 285c6ad..2e963a8 100644 --- a/dns_cache.c +++ b/dns_cache.c @@ -2808,7 +2808,11 @@ struct hostent* dns_naptr_sip_resolvehost(str* name, unsigned short* port, char origproto; str srv_name; - origproto=*proto; + if(proto) { + origproto=*proto; + } else { + origproto=PROTO_NONE; + } he=0; if (dns_hash==0){ /* not init => use normal, non-cached version */ LM_WARN("called before dns cache initialization\n"); @@ -2848,7 +2852,7 @@ struct hostent* dns_naptr_sip_resolvehost(str* name, unsigned short* port, dns_hash_put(e); } naptr_not_found: - *proto = origproto; + if(proto) *proto = origproto; he = no_naptr_srv_sip_resolvehost(name,port,proto); /* fallback all the way down to A/AAAA */ if (he==0) { diff --git a/msg_translator.c b/msg_translator.c index df1a753..8bca35f 100644 --- a/msg_translator.c +++ b/msg_translator.c @@ -197,6 +197,7 @@ static int check_via_address(struct ip_addr* ip, str *name, int i; char* s; int len; + char lproto; /* maybe we are lucky and name it's an ip */ s=ip_addr2a(ip); @@ -229,7 +230,8 @@ static int check_via_address(struct ip_addr* ip, str *name, if (resolver&DO_DNS){ DBG("check_via_address: doing dns lookup\n"); /* try all names ips */ - he=sip_resolvehost(name, &port, 0); /* don't use naptr */ + lproto = PROTO_NONE; + he=sip_resolvehost(name, &port, &lproto); /* don't use naptr */ if (he && ip->af==he->h_addrtype){ for(i=0;he && he->h_addr_list[i];i++){ if ( memcmp(&he->h_addr_list[i], ip->u.addr, ip->len)==0)

1 0

git:master:0bbcbe83: nat_traversal: use local variable for port and proto on keepalive resolve
by Daniel-Constantin Mierla 22 Dec '14

22 Dec '14

Module: kamailio Branch: master Commit: 0bbcbe8328d558a9bf3cb151ab53be93b09c2669 URL: https://github.com/kamailio/kamailio/commit/0bbcbe8328d558a9bf3cb151ab53be9… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2014-12-22T21:40:52+01:00 nat_traversal: use local variable for port and proto on keepalive resolve - safe execution for those resolving functions expecting PROTO_NONE - reported by Jason Penton, FS#491 --- Modified: modules/nat_traversal/nat_traversal.c --- Diff: https://github.com/kamailio/kamailio/commit/0bbcbe8328d558a9bf3cb151ab53be9… Patch: https://github.com/kamailio/kamailio/commit/0bbcbe8328d558a9bf3cb151ab53be9… --- diff --git a/modules/nat_traversal/nat_traversal.c b/modules/nat_traversal/nat_traversal.c index 2f0ffbe..87aaf50 100644 --- a/modules/nat_traversal/nat_traversal.c +++ b/modules/nat_traversal/nat_traversal.c @@ -1510,6 +1510,8 @@ send_keepalive(NAT_Contact *contact) struct dest_info dst; int nat_port, len; str nat_ip; + unsigned short lport; + char lproto; if (keepalive_params.from == NULL) { if (contact->socket != last_socket) { @@ -1554,7 +1556,9 @@ send_keepalive(NAT_Contact *contact) ptr = strchr(nat_ip.s, ':'); nat_ip.len = ptr - nat_ip.s; nat_port = strtol(ptr+1, NULL, 10); - hostent = sip_resolvehost(&nat_ip, NULL, NULL); + lport = 0; + lproto = PROTO_NONE; + hostent = sip_resolvehost(&nat_ip, &lport, &lproto); hostent2su(&dst.to, hostent, 0, nat_port); dst.proto=PROTO_UDP; dst.send_sock=contact->socket;

1 0

[tracker] Comment added: CRASH: segfault in parser/parse_via.c
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#478 - CRASH: segfault in parser/parse_via.c User who did this - Daniel-Constantin Mierla (miconda) ---------- I had a wrong command, wanted to get output of: p *t->uas.request ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=478#comment1740 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: FreeBSD + mariadb make fail
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#503 - FreeBSD + mariadb make fail User who did this - Daniel-Constantin Mierla (miconda) ---------- What version are you using? The line is in the Makefile for db_mysql module: * https://github.com/kamailio/kamailio/blob/master/modules/db_mysql/Makefile#… ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=503#comment1739 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