sr-dev December 2014

sr-dev@lists.kamailio.org

33 participants
255 discussions

[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 … [View More]

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 … [View More]

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 … [View More]

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 … [View More]

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.… [View More]

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 … [View More]

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 … [View More]

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 … [View More]

1 0

[tracker] Task closed: tm: possible crash if trying to aapend branch on cancelled transaction
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#498 - tm: possible crash if trying to aapend branch on cancelled transaction User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Fixed Additional comments about closing: Patch pushed to master. More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=498 You are receiving this message because you have requested it from the … [View More]

1 0

git:master:7a6b8657: tm: don't clean local vars when trying to append a new branch to an active canceled transaction
by Daniel-Constantin Mierla 22 Dec '14

22 Dec '14

Module: kamailio Branch: master Commit: 7a6b86576220b423c4dc5ccab7e7f22a50be27bd URL: https://github.com/kamailio/kamailio/commit/7a6b86576220b423c4dc5ccab7e7f22… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2014-12-22T18:37:25+01:00 tm: don't clean local vars when trying to append a new branch to an active canceled transaction - they are not set, reported by Ovidiu Sas, FS#498 --- Modified: modules/tm/… [View More]t_append_branches.c --- Diff: https://github.com/kamailio/kamailio/commit/7a6b86576220b423c4dc5ccab7e7f22… Patch: https://github.com/kamailio/kamailio/commit/7a6b86576220b423c4dc5ccab7e7f22… --- diff --git a/modules/tm/t_append_branches.c b/modules/tm/t_append_branches.c index be9731b..9600a21 100644 --- a/modules/tm/t_append_branches.c +++ b/modules/tm/t_append_branches.c @@ -75,12 +75,16 @@ int t_append_branches(void) { LM_DBG("transaction %u:%u in status %d\n", t->hash_index, t->label, t->uas.status); /* test if transaction has already been canceled */ - if (t->flags & T_CANCELED) goto canceled; + if (t->flags & T_CANCELED) { + ser_error=E_CANCELED; + return -1; + } if ((t->uas.status >= 200 && t->uas.status<=399) || ((t->uas.status >= 600 && t->uas.status) && !(t->flags & (T_6xx | T_DISABLE_6xx))) ) { - LM_DBG("transaction %u:%u in status %d: cannot append new branch\n", t->hash_index, t->label, t->uas.status); + LM_DBG("transaction %u:%u in status %d: cannot append new branch\n", + t->hash_index, t->label, t->uas.status); return -1; } @@ -90,7 +94,8 @@ int t_append_branches(void) { outgoings = t->nr_of_outgoings; orig_msg = t->uas.request; - LM_DBG("Call %.*s: %d (%d) outgoing branches\n",orig_msg->callid->body.len, orig_msg->callid->body.s,outgoings, nr_branches); + LM_DBG("Call %.*s: %d (%d) outgoing branches\n",orig_msg->callid->body.len, + orig_msg->callid->body.s,outgoings, nr_branches); lowest_ret=E_UNSPEC; added_branches=0; @@ -142,7 +147,8 @@ int t_append_branches(void) { clear_branches(); - LM_DBG("Call %.*s: %d (%d) outgoing branches after clear_branches()\n",orig_msg->callid->body.len, orig_msg->callid->body.s,outgoings, nr_branches); + LM_DBG("Call %.*s: %d (%d) outgoing branches after clear_branches()\n", + orig_msg->callid->body.len, orig_msg->callid->body.s,outgoings, nr_branches); setbflagsval(0, backup_bflags); /* update message flags, if changed in branch route */ @@ -169,7 +175,8 @@ int t_append_branches(void) { if (branch_ret==i) { /* success */ success_branch++; if (unlikely(has_tran_tmcbs(t, TMCB_REQUEST_OUT))) - run_trans_callbacks_with_buf( TMCB_REQUEST_OUT, &t->uac[nr_branches].request, + run_trans_callbacks_with_buf( TMCB_REQUEST_OUT, + &t->uac[nr_branches].request, orig_msg, 0, -orig_msg->REQ_METHOD); } else /* new branch added */ [View Less]

1 0

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

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev December 2014