sr-dev August 2011

sr-dev@lists.kamailio.org

33 participants
292 discussions

git:jh/mtree: lib/srdb1/schema/mtrees.xml: increased version number due to index change
by Juha Heinanen 28 Aug '11

28 Aug '11

Module: sip-router Branch: jh/mtree Commit: 7bab7bfcbe51089d3db3d9b611c96f5d8ae78c05 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=7bab7bf… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Sun Aug 28 19:45:09 2011 +0300 lib/srdb1/schema/mtrees.xml: increased version number due to index change --- lib/srdb1/schema/mtrees.xml | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/srdb1/schema/mtrees.xml b/lib/srdb1/schema/mtrees.xml index 6913766..eb3f1ab 100644 --- a/lib/srdb1/schema/mtrees.xml +++ b/lib/srdb1/schema/mtrees.xml @@ -9,7 +9,7 @@ <table id="mtrees" xmlns:db="http://docbook.org/ns/docbook"> <name>mtrees</name> - <version>1</version> + <version>2</version> <type db="mysql">&MYSQL_TABLE_TYPE;</type> <description> <db:para>This table is used by the mtree module to load values in shared memory trees at start up. More information about the mtree module can be found at: &KAMAILIO_MOD_DOC;mtree.html

1 0

git:jh/mtree: modules/mtree: mtree enhancements
by Juha Heinanen 28 Aug '11

28 Aug '11

Module: sip-router Branch: jh/mtree Commit: 54335d6730328f7ff3d5dea8549a52f47f04416d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=54335d6… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Sun Aug 28 15:46:44 2011 +0300 modules/mtree: mtree enhancements - Added integer tree payload type with type value 2. - Same prefix can have multiple payload values if mt_allow_duplicates module parameter has value 1. --- lib/srdb1/schema/mtree.xml | 3 +- lib/srdb1/schema/mtrees.xml | 3 +- modules/mtree/README | 44 +++++--- modules/mtree/doc/mtree_admin.xml | 28 ++++- modules/mtree/mtree.c | 218 ++++++++++++++++++++++--------------- modules/mtree/mtree.h | 27 +++-- modules/mtree/mtree_mod.c | 27 ++++- 7 files changed, 227 insertions(+), 123 deletions(-) Diff: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commitdiff;h=543…

1 0

git:master: modules/mtree: removed unused variable.
by Juha Heinanen 28 Aug '11

28 Aug '11

Module: sip-router Branch: master Commit: 1b3cc04023100e3c27d130002dc03c4bf33e21ff URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1b3cc04… Author: Juha Heinanen <jh(a)tutpro.com> Committer: Juha Heinanen <jh(a)tutpro.com> Date: Sun Aug 28 15:36:56 2011 +0300 modules/mtree: removed unused variable. --- modules/mtree/mtree.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/modules/mtree/mtree.c b/modules/mtree/mtree.c index 6be1481..b2ad347 100644 --- a/modules/mtree/mtree.c +++ b/modules/mtree/mtree.c @@ -314,7 +314,7 @@ str* mt_get_tvalue(m_tree_t *pt, str *tomatch, int *plen) int mt_add_tvalues(struct sip_msg *msg, m_tree_t *pt, str *tomatch) { - int l, len; + int l; mt_node_t *itn; int_str val, values_avp_name; unsigned short values_name_type; @@ -330,7 +330,7 @@ int mt_add_tvalues(struct sip_msg *msg, m_tree_t *pt, str *tomatch) return -1; } - l = len = 0; + l = 0; itn = pt->head; while (itn != NULL && l < tomatch->len && l < MT_MAX_DEPTH) { @@ -347,7 +347,6 @@ int mt_add_tvalues(struct sip_msg *msg, m_tree_t *pt, str *tomatch) values_avp_name.s.len, values_avp_name.s.s, val.s.len, val.s.s); add_avp(values_name_type|AVP_VAL_STR, values_avp_name, val); - len = l+1; } itn = itn[_mt_char_table[(unsigned int)tomatch->s[l]]].child;

1 0

[tracker] Comment added: Kamailio 3.1.3 crash
by sip-router 27 Aug '11

27 Aug '11

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#122 - Kamailio 3.1.3 crash User who did this - Timo Reimann (tr) ---------- @Jasmin: Over the last couple of days, I added a few patches to the dialog module. One of them improves dlg_manage() to deal with spiral detection properly. The code is part of the master and latest 3.1 branches. @Marius: Is the "quick and dirty" fix to the initial problem part of upstream now so that we can close this bug? If not, you may want to consider renaming the report's topic to something more meaningful that mirrors the real cause of the problem (which you had analyzed quite thoroughly here). ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=122#comment253 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: DLGCB_CONFIRMED triggered on 200/OK, not ACK
by sip-router 27 Aug '11

27 Aug '11

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#143 - DLGCB_CONFIRMED triggered on 200/OK, not ACK User who did this - Timo Reimann (tr) Reason for closing: Fixed Additional comments about closing: Fixed in commit 934adcfb9, to be shipped in SIP Router 3.2. More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=143 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.1: modules_k/dialog When hash_size was smaller then 1, consider this as a value(1 == 2^0)
by Timo Reimann 27 Aug '11

27 Aug '11

Module: sip-router Branch: 3.1 Commit: 18881421102eaac97c85d84cbbd1597c3f342d23 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1888142… Author: Marius Zbihlei <marius.zbihlei(a)1and1.ro> Committer: Timo Reimann <timo.reimann(a)1und1.de> Date: Wed Aug 17 15:00:00 2011 +0300 modules_k/dialog When hash_size was smaller then 1, consider this as a value(1 == 2^0) The 1<<(n-1) check for the power of two smaller than the given number doesn't work for n == 0 (undefined behavior), so values of dlg_hash_size smaller than 1 where not checked correctly. (cherry picked from commit 11033b05be673593a87afdc2a1d8a2c887fb9ab7) --- modules_k/dialog/dialog.c | 9 ++++++++- 1 files changed, 8 insertions(+), 1 deletions(-) diff --git a/modules_k/dialog/dialog.c b/modules_k/dialog/dialog.c index 2c63d87..38a30b5 100644 --- a/modules_k/dialog/dialog.c +++ b/modules_k/dialog/dialog.c @@ -559,11 +559,18 @@ static int mod_init(void) return -1; } + /* sanitize dlg_hash_zie */ + if (dlg_hash_size < 1){ + LM_WARN("hash_size is smaller " + "then 1 -> rounding from %d to 1\n", + dlg_hash_size); + dlg_hash_size = 1; + } /* initialized the hash table */ for( n=0 ; n<(8*sizeof(n)) ; n++) { if (dlg_hash_size==(1<<n)) break; - if (dlg_hash_size<(1<<n)) { + if (n && dlg_hash_size<(1<<n)) { LM_WARN("hash_size is not a power " "of 2 as it should be -> rounding from %d to %d\n", dlg_hash_size, 1<<(n-1));

1 0

git:master: modules_k/dialog: Remove old parameter "del" from interface
by Timo Reimann 27 Aug '11

27 Aug '11

Module: sip-router Branch: master Commit: 99db13c459888a2a7e8ac4af0fc7642541ba5ccb URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=99db13c… Author: Timo Reimann <timo.reimann(a)1und1.de> Committer: Timo Reimann <timo.reimann(a)1und1.de> Date: Sat Aug 27 02:41:51 2011 +0200 modules_k/dialog: Remove old parameter "del" from interface description. --- modules_k/dialog/dlg_hash.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/modules_k/dialog/dlg_hash.c b/modules_k/dialog/dlg_hash.c index 0aee562..35e5bc1 100644 --- a/modules_k/dialog/dlg_hash.c +++ b/modules_k/dialog/dlg_hash.c @@ -472,7 +472,6 @@ not_found: * \param ftag from tag * \param ttag to tag * \param dir direction - * \param del will set to 1 if dialog is deleted * \return dialog structure on success, NULL on failure */ static inline struct dlg_cell* internal_get_dlg(unsigned int h_entry,

1 0

git:3.1: modules_k/dialog: Fix commit backport mistakes in dlg_handlers.c
by Timo Reimann 27 Aug '11

27 Aug '11

Module: sip-router Branch: 3.1 Commit: cb21425e6fa9e824a51b44829b0ad9d9bf9beb99 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=cb21425… Author: Timo Reimann <timo.reimann(a)1und1.de> Committer: Timo Reimann <timo.reimann(a)1und1.de> Date: Sat Aug 27 02:20:00 2011 +0200 modules_k/dialog: Fix commit backport mistakes in dlg_handlers.c causing compiler run to fail. - Add and set missing "spiral_detected" flag variable. - Use correct SIP message variable name (req -> msg). --- modules_k/dialog/dlg_handlers.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/modules_k/dialog/dlg_handlers.c b/modules_k/dialog/dlg_handlers.c index 3500fad..89fb0b3 100644 --- a/modules_k/dialog/dlg_handlers.c +++ b/modules_k/dialog/dlg_handlers.c @@ -664,6 +664,7 @@ int dlg_new_dialog(struct sip_msg *msg, struct cell *t) str ttag; str req_uri; unsigned int dir; + int spiral_detected; if(current_dlg_pointer != NULL) return -1; @@ -694,6 +695,7 @@ int dlg_new_dialog(struct sip_msg *msg, struct cell *t) { LM_DBG("Callid '%.*s' found, must be a spiraled request\n", callid.len, callid.s); + spiral_detected = 1; run_dlg_callbacks( DLGCB_SPIRALED, dlg, msg, DLG_DIR_DOWNSTREAM, 0); @@ -702,6 +704,7 @@ int dlg_new_dialog(struct sip_msg *msg, struct cell *t) goto finish; } } + spiral_detected = 0; dlg = build_new_dlg (&callid /*callid*/, &(get_from(msg)->uri) /*from uri*/, @@ -775,7 +778,7 @@ finish: } } - set_current_dialog(req, dlg); + set_current_dialog(msg, dlg); _dlg_ctx.dlg = dlg; ref_dlg(dlg, 1);

1 0

git:3.1: modules_k/dialog: Improve error handling during dialog creation.
by Timo Reimann 27 Aug '11

27 Aug '11

Module: sip-router Branch: 3.1 Commit: d5e90ab587746a39c9be44d9f9f1c325b8d4902d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d5e90ab… Author: Timo Reimann <timo.reimann(a)1und1.de> Committer: Timo Reimann <timo.reimann(a)1und1.de> Date: Sat Aug 27 01:44:03 2011 +0200 modules_k/dialog: Improve error handling during dialog creation. - Set current dialog only after all other setup work succeeded. (Eliminates call to profile_cleanup() and guarantees that dialog profiling does not yield to segfault if dialog creation fails for certain reasons.) - During error handling, refrain from decrementing the reference counter for dialog linking if this is a spiraled request (because positive spiral detection skips the linkage code path). (cherry picked from commit 71dd754818d8c3ec304298f1002a45889c2da0aa) Conflicts: modules_k/dialog/dlg_handlers.c --- modules_k/dialog/dlg_handlers.c | 13 +++++++------ 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/modules_k/dialog/dlg_handlers.c b/modules_k/dialog/dlg_handlers.c index 16eda6c..3500fad 100644 --- a/modules_k/dialog/dlg_handlers.c +++ b/modules_k/dialog/dlg_handlers.c @@ -757,10 +757,6 @@ int dlg_new_dialog(struct sip_msg *msg, struct cell *t) if_update_stat( dlg_enable_stats, processed_dlgs, 1); finish: - set_current_dialog(msg, dlg); - _dlg_ctx.dlg = dlg; - ref_dlg(dlg, 1); - if (t) { // transaction exists ==> keep ref counter large enough to // avoid premature cleanup and ensure proper dialog referencing @@ -779,10 +775,15 @@ finish: } } + set_current_dialog(req, dlg); + _dlg_ctx.dlg = dlg; + ref_dlg(dlg, 1); + return 0; + error: - unref_dlg(dlg,1); // undo ref regarding linking - profile_cleanup(msg, 0, NULL); // undo ref regarding setting current dialog + if (!spiral_detected) + unref_dlg(dlg,1); // undo ref regarding linking return -1; }

1 0

git:master: modules_k/dialog: Improve error handling during dialog creation .
by Timo Reimann 26 Aug '11

26 Aug '11

Module: sip-router Branch: master Commit: 71dd754818d8c3ec304298f1002a45889c2da0aa URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=71dd754… Author: Timo Reimann <timo.reimann(a)1und1.de> Committer: Timo Reimann <timo.reimann(a)1und1.de> Date: Sat Aug 27 01:32:49 2011 +0200 modules_k/dialog: Improve error handling during dialog creation. - Set current dialog only after all other setup work succeeded. (Eliminates call to profile_cleanup() and guarantees that dialog profiling does not yield to segfault if dialog creation fails for certain reasons.) - During error handling, refrain from decrementing the reference counter for dialog linking if this is a spiraled request (because positive spiral detection skips the linkage code path). --- modules_k/dialog/dlg_handlers.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/modules_k/dialog/dlg_handlers.c b/modules_k/dialog/dlg_handlers.c index 4951515..a5eb007 100644 --- a/modules_k/dialog/dlg_handlers.c +++ b/modules_k/dialog/dlg_handlers.c @@ -846,11 +846,6 @@ int dlg_new_dialog(struct sip_msg *req, struct cell *t, const int run_initial_cb if_update_stat( dlg_enable_stats, processed_dlgs, 1); finish: - - set_current_dialog(req, dlg); - _dlg_ctx.dlg = dlg; - ref_dlg(dlg, 1); - if (t) { // transaction exists ==> keep ref counter large enough to // avoid premature cleanup and ensure proper dialog referencing @@ -870,10 +865,15 @@ finish: } } + set_current_dialog(req, dlg); + _dlg_ctx.dlg = dlg; + ref_dlg(dlg, 1); + return 0; + error: - unref_dlg(dlg,1); // undo ref regarding linking - profile_cleanup(req, 0, NULL); // undo ref regarding setting current dialog + if (!spiral_detected) + unref_dlg(dlg,1); // undo ref regarding linking return -1; }

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2011