sr-dev March 2014

sr-dev@lists.kamailio.org

29 participants
298 discussions

New official Debian and Ubuntu repository
by Victor Seva 04 Mar '14

04 Mar '14

The new build system for Debian and Ubuntu packages is now in place. This service is kindly sponsored by SipWise [0] thanks to Andreas Granig [1]. Sipwise is providing the hosting and man power to create and manage this new system. deb.kamailio.org is based on jenkins-debian-glue[2] project running on AWS EC2 environment thanks to Michael Prokop [3] and myself. All the needed files, scripts and info to reproduce this system is kept public at [4]. In [5] is described all the repositories available. * nightly builds are been built if a change is detected in the branch, once by night. - kamailiodev-nightly branch: - 'master' distributions: jessie, wheezy, squeeze, precise - kamailio41-nightly branch: - '4.1' distributions: jessie, wheezy, squeeze, precise - kamailio40-nightly branch: - '4.0' distributions: lenny, squeeze, wheezy, lucid, precise - kamailio33-nightly branch: - '3.3' distributions: lenny, squeeze, wheezy, lucid, precise * tags are been built if a new tag is detected once by night. - kamailio41 branch: - '*/tags/4.1*' distributions: jessie, wheezy, squeeze, precise - kamailio40 branch: - '*/tags/4.0*' distributions: lenny, squeeze, wheezy, lucid, precise - kamailio33 branch: - '*/tags/3.3*' distributions: lenny, squeeze, wheezy, lucid, precise Now there is no 4.1.1 package built so the default kamailio repository is pointing to a empty repository. As soon as we release 4.1.2 this will be fixed. Regards, Victor Seva ( kamailio admin hat on ) [0] http://www.sipwise.com/ [1] http://www.kamailio.org/w/andreas-granig/ [2] https://github.com/mika/jenkins-debian-glue [3] http://michael-prokop.at/ [4] https://github.com/sipwise/kamailio-deb-jenkins [5] http://deb.kamailio.org/index.html

3 2

[tracker] Comment added: NATHELPER module to support RFC6598
by sip-router 04 Mar '14

04 Mar '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#402 - NATHELPER module to support RFC6598 User who did this - Klaus Darilion (klaus3000) ---------- Indeed: from the RFC: The Shared Address Space address range is 100.64.0.0/10. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=402#comment1335 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:3.3: sqlops: safety check for result row field
by Daniel-Constantin Mierla 04 Mar '14

04 Mar '14

Module: sip-router Branch: 3.3 Commit: d3d1e6947ffe4fbf02a1bddb64d2b9ca969c24a9 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d3d1e69… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Mar 4 12:04:13 2014 +0100 sqlops: safety check for result row field - it might not be set in case of a failure in the middle of result conversion from db to internal memory - freeing the partial result could result in a crash - reported by Juha Heinanen (cherry picked from commit f27df26ca01394250cbe12c5ccf694c6afd3707f) (cherry picked from commit 20276e667631ec281851c2ef58d71863ae2318c4) --- modules_k/sqlops/sql_api.c | 13 ++++++++----- 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/modules_k/sqlops/sql_api.c b/modules_k/sqlops/sql_api.c index 83b3690..c4c05dc 100644 --- a/modules_k/sqlops/sql_api.c +++ b/modules_k/sqlops/sql_api.c @@ -226,13 +226,16 @@ void sql_reset_result(sql_result_t *res) { for(i=0; i<res->nrows; i++) { - for(j=0; j<res->ncols; j++) + if(res->vals[i]) { - if(res->vals[i][j].flags&PV_VAL_STR - && res->vals[i][j].value.s.len>0) - pkg_free(res->vals[i][j].value.s.s); + for(j=0; j<res->ncols; j++) + { + if(res->vals[i][j].flags&PV_VAL_STR + && res->vals[i][j].value.s.len>0) + pkg_free(res->vals[i][j].value.s.s); + } + pkg_free(res->vals[i]); } - pkg_free(res->vals[i]); } pkg_free(res->vals); res->vals = NULL;

1 0

git:4.0: sqlops: safety check for result row field
by Daniel-Constantin Mierla 04 Mar '14

04 Mar '14

Module: sip-router Branch: 4.0 Commit: 00f26f831133d770b9fd9ae5ea0026a125abcf40 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=00f26f8… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Mar 4 09:59:07 2014 +0100 sqlops: safety check for result row field - it might not be set in case of a failure in the middle of result conversion from db to internal memory - freeing the partial result could result in a crash - reported by Juha Heinanen (cherry picked from commit f27df26ca01394250cbe12c5ccf694c6afd3707f) (cherry picked from commit 20276e667631ec281851c2ef58d71863ae2318c4) --- modules/sqlops/sql_api.c | 13 ++++++++----- 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/modules/sqlops/sql_api.c b/modules/sqlops/sql_api.c index 6528078..bfd4e96 100644 --- a/modules/sqlops/sql_api.c +++ b/modules/sqlops/sql_api.c @@ -228,13 +228,16 @@ void sql_reset_result(sql_result_t *res) { for(i=0; i<res->nrows; i++) { - for(j=0; j<res->ncols; j++) + if(res->vals[i]) { - if(res->vals[i][j].flags&PV_VAL_STR - && res->vals[i][j].value.s.len>0) - pkg_free(res->vals[i][j].value.s.s); + for(j=0; j<res->ncols; j++) + { + if(res->vals[i][j].flags&PV_VAL_STR + && res->vals[i][j].value.s.len>0) + pkg_free(res->vals[i][j].value.s.s); + } + pkg_free(res->vals[i]); } - pkg_free(res->vals[i]); } pkg_free(res->vals); res->vals = NULL;

1 0

git:4.1: sqlops: safety check for result row field
by Daniel-Constantin Mierla 04 Mar '14

04 Mar '14

Module: sip-router Branch: 4.1 Commit: 20276e667631ec281851c2ef58d71863ae2318c4 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=20276e6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Mar 4 09:59:07 2014 +0100 sqlops: safety check for result row field - it might not be set in case of a failure in the middle of result conversion from db to internal memory - freeing the partial result could result in a crash - reported by Juha Heinanen (cherry picked from commit f27df26ca01394250cbe12c5ccf694c6afd3707f) --- modules/sqlops/sql_api.c | 13 ++++++++----- 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/modules/sqlops/sql_api.c b/modules/sqlops/sql_api.c index 6528078..bfd4e96 100644 --- a/modules/sqlops/sql_api.c +++ b/modules/sqlops/sql_api.c @@ -228,13 +228,16 @@ void sql_reset_result(sql_result_t *res) { for(i=0; i<res->nrows; i++) { - for(j=0; j<res->ncols; j++) + if(res->vals[i]) { - if(res->vals[i][j].flags&PV_VAL_STR - && res->vals[i][j].value.s.len>0) - pkg_free(res->vals[i][j].value.s.s); + for(j=0; j<res->ncols; j++) + { + if(res->vals[i][j].flags&PV_VAL_STR + && res->vals[i][j].value.s.len>0) + pkg_free(res->vals[i][j].value.s.s); + } + pkg_free(res->vals[i]); } - pkg_free(res->vals[i]); } pkg_free(res->vals); res->vals = NULL;

1 0

git:master: sqlops: safety check for result row field
by Daniel-Constantin Mierla 04 Mar '14

04 Mar '14

Module: sip-router Branch: master Commit: f27df26ca01394250cbe12c5ccf694c6afd3707f URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f27df26… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Mar 4 09:59:07 2014 +0100 sqlops: safety check for result row field - it might not be set in case of a failure in the middle of result conversion from db to internal memory - freeing the partial result could result in a crash - reported by Juha Heinanen --- modules/sqlops/sql_api.c | 13 ++++++++----- 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/modules/sqlops/sql_api.c b/modules/sqlops/sql_api.c index 6528078..bfd4e96 100644 --- a/modules/sqlops/sql_api.c +++ b/modules/sqlops/sql_api.c @@ -228,13 +228,16 @@ void sql_reset_result(sql_result_t *res) { for(i=0; i<res->nrows; i++) { - for(j=0; j<res->ncols; j++) + if(res->vals[i]) { - if(res->vals[i][j].flags&PV_VAL_STR - && res->vals[i][j].value.s.len>0) - pkg_free(res->vals[i][j].value.s.s); + for(j=0; j<res->ncols; j++) + { + if(res->vals[i][j].flags&PV_VAL_STR + && res->vals[i][j].value.s.len>0) + pkg_free(res->vals[i][j].value.s.s); + } + pkg_free(res->vals[i]); } - pkg_free(res->vals[i]); } pkg_free(res->vals); res->vals = NULL;

1 0

git:master: ims_registrar_pcscf: Make sure, Contact-Info-Structure is properly initialized.
by Carsten Bock 04 Mar '14

04 Mar '14

Module: sip-router Branch: master Commit: 2cfe0862faafa736b342f15d640581acdf15808c URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=2cfe086… Author: Carsten Bock <carsten(a)ng-voice.com> Committer: Carsten Bock <carsten(a)ng-voice.com> Date: Tue Mar 4 11:50:19 2014 +0100 ims_registrar_pcscf: Make sure, Contact-Info-Structure is properly initialized. --- modules/ims_registrar_pcscf/save.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/modules/ims_registrar_pcscf/save.c b/modules/ims_registrar_pcscf/save.c index e9bcd7d..2843278 100644 --- a/modules/ims_registrar_pcscf/save.c +++ b/modules/ims_registrar_pcscf/save.c @@ -131,6 +131,10 @@ static inline int update_contacts(struct sip_msg *req,struct sip_msg *rpl, udoma * then, we will update on NOTIFY */ return 0; } + + // Set the structure to "0", to make sure it's properly initialized + memset(&ci, 0, sizeof(struct pcontact_info)); + for (h = rpl->contact; h; h = h->next) { if (h->type == HDR_CONTACT_T && h->parsed) for (c = ((contact_body_t*) h->parsed)->contacts; c; c = c->next) {

1 0

git:master: kamctl: Minor typo (missing ; ; ), resulted in Syntax Error: syntax error near unexpected token `)' (on Debian 7)
by Carsten Bock 04 Mar '14

04 Mar '14

Module: sip-router Branch: master Commit: 80536540caf19a594f91ed9999c7f812f4f40e82 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=8053654… Author: Carsten Bock <carsten(a)ng-voice.com> Committer: Carsten Bock <carsten(a)ng-voice.com> Date: Tue Mar 4 11:35:31 2014 +0100 kamctl: Minor typo (missing ;;), resulted in Syntax Error: syntax error near unexpected token `)' (on Debian 7) --- utils/kamctl/kamctl | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/utils/kamctl/kamctl b/utils/kamctl/kamctl index 30dddcf..aac4533 100755 --- a/utils/kamctl/kamctl +++ b/utils/kamctl/kamctl @@ -1017,6 +1017,7 @@ db_ops() { exit 1 fi $DBCMDCONNECT + ;; *) usage_db_ops exit 1

1 0

git:master: geoip: for safety checks for the case when geoip_match() is not executed
by Daniel-Constantin Mierla 04 Mar '14

04 Mar '14

Module: sip-router Branch: master Commit: 56ed48bf48c3d78ff5d214833e09a5759f0b5928 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=56ed48b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Mar 4 09:43:39 2014 +0100 geoip: for safety checks for the case when geoip_match() is not executed - but the geoip pv is accessed - reported by Juha Heinanen --- modules/geoip/geoip_pv.c | 20 ++++++++++++++++++++ 1 files changed, 20 insertions(+), 0 deletions(-) diff --git a/modules/geoip/geoip_pv.c b/modules/geoip/geoip_pv.c index 8aa5222..6a18461 100644 --- a/modules/geoip/geoip_pv.c +++ b/modules/geoip/geoip_pv.c @@ -257,6 +257,8 @@ int pv_get_geoip(struct sip_msg *msg, pv_param_t *param, { if(gpv->item->r.flags&1) return pv_get_null(msg, param, res); + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); gpv->item->r.time_zone = (char*)GeoIP_time_zone_by_country_and_region( gpv->item->r.record->country_code, @@ -266,11 +268,15 @@ int pv_get_geoip(struct sip_msg *msg, pv_param_t *param, return pv_geoip_get_strzval(msg, param, res, gpv->item->r.time_zone); case 2: /* zip */ + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); return pv_geoip_get_strzval(msg, param, res, gpv->item->r.record->postal_code); case 3: /* lat */ if((gpv->item->r.flags&2)==0) { + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); snprintf(gpv->item->r.latitude, 15, "%f", gpv->item->r.record->latitude); gpv->item->r.flags |= 2; @@ -280,6 +286,8 @@ int pv_get_geoip(struct sip_msg *msg, pv_param_t *param, case 4: /* lon */ if((gpv->item->r.flags&4)==0) { + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); snprintf(gpv->item->r.longitude, 15, "%f", gpv->item->r.record->longitude); gpv->item->r.flags |= 4; @@ -287,6 +295,8 @@ int pv_get_geoip(struct sip_msg *msg, pv_param_t *param, return pv_geoip_get_strzval(msg, param, res, gpv->item->r.longitude); case 5: /* dma */ + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); return pv_get_sintval(msg, param, res, gpv->item->r.record->dma_code); case 6: /* ips */ @@ -308,14 +318,20 @@ int pv_get_geoip(struct sip_msg *msg, pv_param_t *param, return pv_geoip_get_strzval(msg, param, res, gpv->item->r.record->city); case 9: /* area */ + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); return pv_get_sintval(msg, param, res, gpv->item->r.record->area_code); case 10: /* regc */ + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); return pv_geoip_get_strzval(msg, param, res, gpv->item->r.record->region); case 11: /* regn */ if((gpv->item->r.flags&16)==0) { + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); gpv->item->r.region_name = (char*)GeoIP_region_name_by_code( gpv->item->r.record->country_code, @@ -325,9 +341,13 @@ int pv_get_geoip(struct sip_msg *msg, pv_param_t *param, return pv_geoip_get_strzval(msg, param, res, gpv->item->r.region_name); case 12: /* metro */ + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); return pv_get_sintval(msg, param, res, gpv->item->r.record->metro_code); default: /* cc */ + if(gpv->item->r.record==NULL) + return pv_get_null(msg, param, res); return pv_geoip_get_strzval(msg, param, res, gpv->item->r.record->country_code); }

1 0

[tracker] Comment added: Kamailio doesn't to handle any SIP message at all after several dialogs... (CPU high usage)
by sip-router 04 Mar '14

04 Mar '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#397 - Kamailio doesn't to handle any SIP message at all after several dialogs... (CPU high usage) User who did this - Jack Wang (GoGoJack) ---------- OMG...I DO NOW understand why my config.mak may have some problems - For taking a customized settings, the right way should be done such like "make cfg ARCH=arm" before (and the -DUSE_PTHREAD_MUTEX was set right there), but I modify it after "make cfg" which may cause abnormal processing afterwards. Oh My poor concept about the Makefile system :P ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=397#comment1334 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

← Newer
1
...
23
24
25
26
27
28
29
30
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev March 2014