December 2014 - sr-dev - kamailio.org

[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 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: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/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 */

1 0

[tracker] Task opened: IMPORTANT: DO NOT ADD NEW ITEMS HERE - USE GITHUB ISSUE TRACKER
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Daniel-Constantin Mierla (miconda) Attached to Project - sip-router Summary - IMPORTANT: DO NOT ADD NEW ITEMS HERE - USE GITHUB ISSUE TRACKER Task Type - Bug Report Category - Core Status - New Assigned To - Operating System - All Severity - Critical Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - This tracker is going to be decommissioned. Add new issues to: * https://github.com/kamailio/kamailio/issues More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=504 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 changed: dialplan: Reload failure
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has been changed. The changes are listed below. For full information about what has changed, visit the URL and click the History tab. FS#497 - dialplan: Reload failure User who did this: Daniel-Constantin Mierla (miconda) Severity: Critical -> Medium More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=497 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 (Attachment added)
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 - Victor (coyote) ---------- # diff -u modules/db_mysql/Makefile{,-} --- modules/db_mysql/Makefile 2014-12-22 12:47:36.631590149 +0300 +++ modules/db_mysql/Makefile- 2014-12-22 12:47:50.818588504 +0300 @@ -21,7 +21,7 @@ ifneq ($(MYSQLCFG),) # use autodetection - DEFS += $(shell $(MYSQLCFG) --include ) + DEFS += $(shell $(MYSQLCFG) --include | sed 's/$-I.*$\/mysql/\1/g' ) LIBS = $(shell $(MYSQLCFG) --libs) else # mysql.h locations (freebsd,openbsd solaris) ---------- One or more files have been attached. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=503#comment1738 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) ---------- Ca you attach the output of? git diff or diff -u oldfile newfile ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=503#comment1737 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 opened: FreeBSD + mariadb make fail
by sip-router 22 Dec '14

22 Dec '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. A new Flyspray task has been opened. Details are below. User who did this - Victor (coyote) Attached to Project - sip-router Summary - FreeBSD + mariadb make fail Task Type - Bug Report Category - Core Status - Unconfirmed Assigned To - Operating System - FreeBSD Severity - Medium Priority - Normal Reported Version - Development Due in Version - Undecided Due Date - Undecided Details - With mariadb + FreeBSD we got error message adout <mysql/mysql.h> not found. #grep -A2 -B2 sed modules/db_mysql/Makefile ifneq ($(MYSQLCFG),) # use autodetection DEFS += $(shell $(MYSQLCFG) --include | sed 's/$-I.*$\/mysql/\1/g' ) LIBS = $(shell $(MYSQLCFG) --libs) # mysql_config --include -I/usr/local/include/mysql -I/usr/local/include/mysql/.. see last path - cause of error. # mysql_config --include | sed 's/$-I.*$\/mysql/\1/g' -I/usr/local/include/mysql -I/usr/local/include/.. Standart mysql headers in BSD's: /usr/local/include/mysql/msql.h, #include <mysql/mysql.h> /usr/local/include/mysql/<mysql/mysql.h> - not found /usr/local/include/../<mysql/mysql.h> - not found too For correct this situation need exclude sed at all. # diff modules/db_mysql/Makefile{,-} 24c24 < DEFS += $(shell $(MYSQLCFG) --include ) --- > DEFS += $(shell $(MYSQLCFG) --include | sed 's/$-I.*$\/mysql/\1/g' ) More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=503 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

Passed: kamailio/kamailio#5 (4.2 - a2aa22d)
by Travis CI 19 Dec '14

19 Dec '14

Build Update for kamailio/kamailio ------------------------------------- Build: #5 Status: Passed Duration: 4 minutes and 57 seconds Commit: a2aa22d (4.2) Author: Victor Seva Message: pkg/kamailio: Initial Travis CI configuration (cherry picked from commit 3cdfb23b6e9d13fea3ee0994b450279c3cc6a488) View the changeset: https://github.com/kamailio/kamailio/compare/9bb59f922da4...a2aa22d8f23a View the full build log and details: https://travis-ci.org/kamailio/kamailio/builds/44573038 -- You can configure recipients for build notifications in your .travis.yml file. See http://docs.travis-ci.com/user/notifications

1 0

git:4.2:a2aa22d8: pkg/kamailio: Initial Travis CI configuration
by Victor Seva 19 Dec '14

19 Dec '14

Module: kamailio Branch: 4.2 Commit: a2aa22d8f23a8bcad0ed83831d0f6f10e9059a7c URL: https://github.com/kamailio/kamailio/commit/a2aa22d8f23a8bcad0ed83831d0f6f1… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2014-12-19T14:12:26+01:00 pkg/kamailio: Initial Travis CI configuration (cherry picked from commit 3cdfb23b6e9d13fea3ee0994b450279c3cc6a488) --- Added: .travis.yml Added: pkg/kamailio/deb/build_deps.sh Added: pkg/kamailio/deb/build_travis.sh --- Diff: https://github.com/kamailio/kamailio/commit/a2aa22d8f23a8bcad0ed83831d0f6f1… Patch: https://github.com/kamailio/kamailio/commit/a2aa22d8f23a8bcad0ed83831d0f6f1… --- diff --git a/.travis.yml b/.travis.yml new file mode 100644 index 0000000..d5b76b0 --- /dev/null +++ b/.travis.yml @@ -0,0 +1,17 @@ +language: c +compiler: + - gcc + - clang +before_install: + - sudo apt-get update -qq + - sudo apt-get install -y gdebi-core + - sudo pkg/kamailio/deb/build_deps.sh precise $TRAVIS_BUILD_DIR +script: ./pkg/kamailio/deb/build_travis.sh +notifications: + irc: + channels: + - "chat.freenode.net#Kamailio" + on_success: change + on_failure: always + email: + - sr-dev(a)lists.sip-router.org diff --git a/pkg/kamailio/deb/build_deps.sh b/pkg/kamailio/deb/build_deps.sh new file mode 100755 index 0000000..10387b2 --- /dev/null +++ b/pkg/kamailio/deb/build_deps.sh @@ -0,0 +1,23 @@ +#!/bin/bash +# +# build_deps script for travis CI +# installs the build_deps packages needed to build Kamailio +# environment based on Ubuntu 12.04 LTS (precise) +# +DIST=${1:-precise} +BASE_DIR=${2:-$(pwd)} +CONTROL_FILE="${BASE_DIR}/pkg/kamailio/deb/${DIST}/control" +if ! [ -f "${CONTROL_FILE}" ]; then + echo "Error: No ${CONTROL_FILE} found" + exit 1 +fi + +BUILD_DEPS=$(/usr/bin/gdebi --quiet --non-interactive \ + --option=APT::Install-Recommends=false \ + --apt-line ${CONTROL_FILE}) +if [ -z "${BUILD_DEPS}" ]; then + echo "Error: no build deps packages resolved" + exit 2 +fi + +apt-get install -y $BUILD_DEPS diff --git a/pkg/kamailio/deb/build_travis.sh b/pkg/kamailio/deb/build_travis.sh new file mode 100755 index 0000000..a18b2e7 --- /dev/null +++ b/pkg/kamailio/deb/build_travis.sh @@ -0,0 +1,22 @@ +#!/bin/bash +# +# build script for travis CI +# environment based on Ubuntu 12.04 LTS (precise) +# +export JAVA_HOME="/usr/lib/jvm/java-gcj" +EXCLUDED_MODULES="" +EXTRA_EXCLUDED_MODULES="bdb dbtext oracle pa iptrtpproxy mi_xmlrpc dnssec kazoo cnxcc" +PACKAGE_GROUPS="mysql postgres berkeley unixodbc radius presence ldap xml perl utils lua memcached \ + snmpstats carrierroute xmpp cpl redis python geoip\ + sqlite json mono ims sctp java \ + purple tls outbound websocket autheph" +echo "make cfg" +make FLAVOUR=kamailio cfg \ + skip_modules="${EXCLUDED_MODULES} ${EXTRA_EXCLUDED_MODULES}" \ + group_include="kstandard" +echo "make all" +make all +echo "make groups" +for grp in ${PACKAGE_GROUPS}; do + make every-module group_include="k${grp}" +done

1 0

Passed: linuxmaniac/kamailio#13 (vseva/travis_4.2 - a2aa22d)
by Travis CI 19 Dec '14

19 Dec '14

Build Update for linuxmaniac/kamailio ------------------------------------- Build: #13 Status: Passed Duration: 5 minutes and 24 seconds Commit: a2aa22d (vseva/travis_4.2) Author: Victor Seva Message: pkg/kamailio: Initial Travis CI configuration (cherry picked from commit 3cdfb23b6e9d13fea3ee0994b450279c3cc6a488) View the changeset: https://github.com/linuxmaniac/kamailio/compare/1630ec25958b^...a2aa22d8f23a View the full build log and details: https://travis-ci.org/linuxmaniac/kamailio/builds/44571829 -- You can configure recipients for build notifications in your .travis.yml file. See http://docs.travis-ci.com/user/notifications

1 0