sr-dev August 2013

sr-dev@lists.kamailio.org

29 participants
149 discussions

git:master: modules/debugger: refresh README
by Victor Seva 13 Aug '13

13 Aug '13

Module: sip-router Branch: master Commit: 9061cf2a85ac89cf403910b7b4fa93b29dccb350 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=9061cf2… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: Tue Aug 13 11:45:02 2013 +0200 modules/debugger: refresh README --- modules/debugger/README | 65 ++++++++++++++++++++++++++++++++++------------ 1 files changed, 48 insertions(+), 17 deletions(-) diff --git a/modules/debugger/README b/modules/debugger/README index 03e069d..3a51c02 100644 --- a/modules/debugger/README +++ b/modules/debugger/README @@ -10,7 +10,7 @@ Daniel-Constantin Mierla <miconda(a)gmail.com> - Copyright (c) 2010 Daniel-Constantin Mierla (asipto.com) + Copyright © 2010 Daniel-Constantin Mierla (asipto.com) __________________________________________________________________ Table of Contents @@ -41,6 +41,7 @@ Daniel-Constantin Mierla 4.1. dbg_breakpoint(mode) 4.2. dbg_pv_dump([mask] [, level]) + 4.3. reset_msgid (int) 5. Exported RPC Functions @@ -48,6 +49,7 @@ Daniel-Constantin Mierla 5.2. dbg.trace 5.3. dbg.bp 5.4. dbg.mod_level + 5.5. dbg.reset_msgid 6. Usage @@ -66,6 +68,7 @@ Daniel-Constantin Mierla 1.11. Set log_assign parameter 1.12. dbg_breakpoint usage 1.13. dbg_pv_dump usage + 1.14. Set reset_msgid parameter Chapter 1. Admin Guide @@ -95,6 +98,7 @@ Chapter 1. Admin Guide 4.1. dbg_breakpoint(mode) 4.2. dbg_pv_dump([mask] [, level]) + 4.3. reset_msgid (int) 5. Exported RPC Functions @@ -102,6 +106,7 @@ Chapter 1. Admin Guide 5.2. dbg.trace 5.3. dbg.bp 5.4. dbg.mod_level + 5.5. dbg.reset_msgid 6. Usage @@ -164,7 +169,7 @@ Chapter 1. Admin Guide startup. You can change the value at runtime without restart, globally or per process. - Default value is "0" (disabled). + Default value is “0” (disabled). Example 1.1. Set cfgtrace parameter ... @@ -176,7 +181,7 @@ modparam("debugger", "cfgtrace", 1) Control whether every line (global) breakpoint is enabled or disabled at startup. - Default value is "0" (disabled). + Default value is “0” (disabled). Example 1.2. Set breakpoint parameter ... @@ -187,7 +192,7 @@ modparam("debugger", "breakpoint", 1) What log level is to be used to print module-specific messages. - Default value is "-1" (L_ERR). + Default value is “-1” (L_ERR). Example 1.3. Set log_level parameter ... @@ -200,7 +205,7 @@ modparam("debugger", "log_level", 1) using this setting, you can configure syslog to send debug messages to a separate log channel, like a specific kamailio-debug log file. - Default value is "NULL" (default from core). + Default value is “NULL” (default from core). Example 1.4. Set log_facility parameter ... @@ -211,7 +216,7 @@ modparam("debugger", "log_facility", "LOG_DAEMON") String to print before any module-specific messages. - Default value is "*** cfgtrace:". + Default value is “*** cfgtrace:”. Example 1.5. Set log_prefix parameter ... @@ -223,7 +228,7 @@ modparam("debugger", "log_prefix", "from-debugger-with-love:") Microseconds to sleep before checking for new commands when waiting at a breakpoint. - Default value is "100000" (that is 0.1 sec). + Default value is “100000” (that is 0.1 sec). Example 1.6. Set step_usleep parameter ... @@ -237,7 +242,7 @@ modparam("debugger", "step_usleep", 500000) blocking RPC process forever in case the worker process 'forgets' to write back a reply. - Default value is "200". + Default value is “200”. Example 1.7. Set step_loops parameter ... @@ -251,7 +256,7 @@ modparam("debugger", "step_loops", 100) slots). This parameter is accesible readonly via the Kamailio config framework. - Default value is "0" - feature disabled. + Default value is “0” - feature disabled. Example 1.8. Set mod_hash_size parameter ... @@ -263,7 +268,7 @@ modparam("debugger", "mod_hash_size", 5) Enable or disable per module log level (0 - disabled, 1 - enabled). This parameter is tunable via the Kamailio config framework. - Default value is "0". + Default value is “0”. Example 1.9. Set mod_level_mode parameter ... @@ -287,7 +292,7 @@ modparam("debugger", "mod_level", "tm=3") Enable or disable log assign actions on config (0 - disabled, 1 - enabled). - Default value is "0". + Default value is “0”. Example 1.11. Set log_assign parameter ... @@ -298,8 +303,9 @@ modparam("debugger", "log_assign", 1) 4.1. dbg_breakpoint(mode) 4.2. dbg_pv_dump([mask] [, level]) + 4.3. reset_msgid (int) -4.1. dbg_breakpoint(mode) +4.1. dbg_breakpoint(mode) Anchor a breakpoint at the current line of the config (the one on which this function is called). The 'mode' specifies whether the breakpoint @@ -314,7 +320,7 @@ if($si=="10.0.0.10") dbg_breakpoint("1"); ... -4.2. dbg_pv_dump([mask] [, level]) +4.2. dbg_pv_dump([mask] [, level]) Prints the content of pv_cache on json format. Defaults are mask=31 and level = "L_DBG" @@ -359,14 +365,27 @@ vp(x)":[{"different":["foo"]},{"other":[2,1],"more":["hi","bye"]}],"$T_branch_id x":0,"$var(empty)":0} ... +4.3. reset_msgid (int) + + Used to enable or disable the ability to reset the msgid ($mi) through + the dbg.reset_msgid RPC command. (0 - disabled, 1 - enabled). + + Default value is “0” - feature disabled. + + Example 1.14. Set reset_msgid parameter +... +modparam("debugger", "reset_msgid", 1) +... + 5. Exported RPC Functions 5.1. dbg.ls 5.2. dbg.trace 5.3. dbg.bp 5.4. dbg.mod_level + 5.5. dbg.reset_msgid -5.1. dbg.ls +5.1. dbg.ls List Kamailio processes with info related to interactive debugging. @@ -380,7 +399,7 @@ x":0,"$var(empty)":0} dbg.ls dbg.ls 1234 -5.2. dbg.trace +5.2. dbg.trace Control config script running trace. @@ -397,7 +416,7 @@ x":0,"$var(empty)":0} dbg.trace off dbg.trace on 1234 -5.3. dbg.bp +5.3. dbg.bp Control breakpoints and config execution. @@ -435,7 +454,7 @@ x":0,"$var(empty)":0} dbg.bp eval 1234 $fu dbg.bp move 1234 -5.4. dbg.mod_level +5.4. dbg.mod_level Specify module log level. @@ -449,6 +468,18 @@ x":0,"$var(empty)":0} dbg.mod_level core 3 dbg.mod_level tm 3 +5.5. dbg.reset_msgid + + Resets the message sequence ($mi). Internally there is no real change. + This can be useful for unit test cases in order to be able to replicate + exactly the same kamailio output. You need to set the debugger + parameter reset_msgid to 1 to activate this functionallity. + + Name: dbg.reset_msgid + + Examples of use with kamcmd: + dbg.reset_msgid + 6. Usage A common usage is to investigate the execution path for a specific SIP

1 0

[tracker] Comment added: debugger: new RPC command dbg.reset_msgid
by sip-router 13 Aug '13

13 Aug '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#330 - debugger: new RPC command dbg.reset_msgid User who did this - Daniel-Constantin Mierla (miconda) ---------- It can be committed. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=330#comment1047 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:4.0: usrloc: safety checks to catch empty ruid
by Daniel-Constantin Mierla 13 Aug '13

13 Aug '13

Module: sip-router Branch: 4.0 Commit: 3898507f56eefe5c9d8e72b37280e0d133f6e357 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=3898507… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Aug 13 09:37:21 2013 +0200 usrloc: safety checks to catch empty ruid (cherry picked from commit b0e9132ccd1e8385b8b4faf72db806320d48e2f5) --- modules/usrloc/ucontact.c | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/modules/usrloc/ucontact.c b/modules/usrloc/ucontact.c index 35dfa0d..996e8d5 100644 --- a/modules/usrloc/ucontact.c +++ b/modules/usrloc/ucontact.c @@ -93,6 +93,11 @@ ucontact_t* new_ucontact(str* _dom, str* _aor, str* _contact, ucontact_info_t* _ { ucontact_t *c; + if(unlikely(_ci->ruid.len<=0)) { + LM_ERR("no ruid for aor: %.*s\n", _aor->len, ZSW(_aor->s)); + return 0; + } + c = (ucontact_t*)shm_malloc(sizeof(ucontact_t)); if (!c) { LM_ERR("no more shm memory\n"); @@ -465,6 +470,11 @@ int db_insert_ucontact(ucontact_t* _c) if (_c->flags & FL_MEM) { return 0; } + if(unlikely(_c->ruid.len<=0)) { + LM_ERR("invalid ruid for aor: %.*s\n", + _c->aor->len, ZSW(_c->aor->s)); + return -1; + } keys[0] = &user_col; keys[1] = &contact_col;

1 0

git:master: usrloc: safety checks to catch empty ruid
by Daniel-Constantin Mierla 13 Aug '13

13 Aug '13

Module: sip-router Branch: master Commit: b0e9132ccd1e8385b8b4faf72db806320d48e2f5 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b0e9132… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Aug 13 09:37:21 2013 +0200 usrloc: safety checks to catch empty ruid --- modules/usrloc/ucontact.c | 10 ++++++++++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/modules/usrloc/ucontact.c b/modules/usrloc/ucontact.c index c8da8ec..d1ae1f3 100644 --- a/modules/usrloc/ucontact.c +++ b/modules/usrloc/ucontact.c @@ -93,6 +93,11 @@ ucontact_t* new_ucontact(str* _dom, str* _aor, str* _contact, ucontact_info_t* _ { ucontact_t *c; + if(unlikely(_ci->ruid.len<=0)) { + LM_ERR("no ruid for aor: %.*s\n", _aor->len, ZSW(_aor->s)); + return 0; + } + c = (ucontact_t*)shm_malloc(sizeof(ucontact_t)); if (!c) { LM_ERR("no more shm memory\n"); @@ -465,6 +470,11 @@ int db_insert_ucontact(ucontact_t* _c) if (_c->flags & FL_MEM) { return 0; } + if(unlikely(_c->ruid.len<=0)) { + LM_ERR("invalid ruid for aor: %.*s\n", + _c->aor->len, ZSW(_c->aor->s)); + return -1; + } keys[0] = &user_col; keys[1] = &contact_col;

1 0

git:tmp/tm_async_reply_support: Merge branch 'master' into tmp/ tm_async_reply_support
by Richard Good 13 Aug '13

13 Aug '13

Module: sip-router Branch: tmp/tm_async_reply_support Commit: 7637a9ae26113813394f8a4def451e08b9660cd8 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=7637a9a… Author: Richard Good <richard.good(a)smilecoms.com> Committer: Richard Good <richard.good(a)smilecoms.com> Date: Tue Aug 13 09:28:22 2013 +0200 Merge branch 'master' into tmp/tm_async_reply_support ---

1 0

[tracker] Comment added: debugger: new RPC command dbg.reset_msgid (Attachment added)
by sip-router 13 Aug '13

13 Aug '13

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#330 - debugger: new RPC command dbg.reset_msgid User who did this - Víctor Seva (linuxmaniac) ---------- This version does not need any changes in core. As you said I use a pre-script callback to modify the msg->id. Internally there is no change. The module debugger changes the id when it's necessary. When the dbg.reset_msgid command is triggered the flag reset_msg is set to 1 ( per process ). If the flag is set, I save the value of the current msg->id ( per process ) as msgid_base and the msg->id is "reset". Thanks for the help! ---------- 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=330#comment1046 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:richard.good/diameter_rx_media: Merge branch 'master' into richard.good /diameter_rx_media
by Richard Good 13 Aug '13

13 Aug '13

Module: sip-router Branch: richard.good/diameter_rx_media Commit: cf23fa53124a13db8610bf84b82762b5096eaf5b URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=cf23fa5… Author: Richard Good <richard.good(a)smilecoms.com> Committer: Richard Good <richard.good(a)smilecoms.com> Date: Tue Aug 13 09:21:25 2013 +0200 Merge branch 'master' into richard.good/diameter_rx_media ---

1 0

git:4.0: usrloc: more verbose log messages in case of failure to update db records
by Daniel-Constantin Mierla 13 Aug '13

13 Aug '13

Module: sip-router Branch: 4.0 Commit: e59bb3303252c0b47f6027ae2b99a11096521154 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e59bb33… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Aug 13 09:19:02 2013 +0200 usrloc: more verbose log messages in case of failure to update db records (cherry picked from commit 66c8e730d59f9816f09573f5300e4463e5997876) --- modules/usrloc/urecord.c | 11 ++++++++--- 1 files changed, 8 insertions(+), 3 deletions(-) diff --git a/modules/usrloc/urecord.c b/modules/usrloc/urecord.c index dd98450..75ddd1a 100644 --- a/modules/usrloc/urecord.c +++ b/modules/usrloc/urecord.c @@ -333,7 +333,9 @@ static inline void wb_timer(urecord_t* _r) /* Should we remove the contact from the database ? */ if (st_expired_ucontact(t) == 1) { if (db_delete_ucontact(t) < 0) { - LM_ERR("failed to delete contact from the database\n"); + LM_ERR("failed to delete contact from the database" + " (aor: %.*s)\n", + ptr->aor->len, ZSW(ptr->aor->s)); } } @@ -349,7 +351,9 @@ static inline void wb_timer(urecord_t* _r) case 1: /* insert */ if (db_insert_ucontact(ptr) < 0) { - LM_ERR("inserting contact into database failed\n"); + LM_ERR("inserting contact into database failed" + " (aor: %.*s)\n", + ptr->aor->len, ZSW(ptr->aor->s)); ptr->state = old_state; } break; @@ -360,7 +364,8 @@ static inline void wb_timer(urecord_t* _r) else res = db_update_ucontact(ptr); if (res < 0) { - LM_ERR("updating contact in db failed\n"); + LM_ERR("updating contact in db failed (aor: %.*s)\n", + ptr->aor->len, ZSW(ptr->aor->s)); ptr->state = old_state; } break;

1 0

git:master: usrloc: more verbose log messages in case of failure to update db records
by Daniel-Constantin Mierla 13 Aug '13

13 Aug '13

Module: sip-router Branch: master Commit: 66c8e730d59f9816f09573f5300e4463e5997876 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=66c8e73… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Aug 13 09:19:02 2013 +0200 usrloc: more verbose log messages in case of failure to update db records --- modules/usrloc/urecord.c | 11 ++++++++--- 1 files changed, 8 insertions(+), 3 deletions(-) diff --git a/modules/usrloc/urecord.c b/modules/usrloc/urecord.c index 89defe0..93c8446 100644 --- a/modules/usrloc/urecord.c +++ b/modules/usrloc/urecord.c @@ -333,7 +333,9 @@ static inline void wb_timer(urecord_t* _r) /* Should we remove the contact from the database ? */ if (st_expired_ucontact(t) == 1) { if (db_delete_ucontact(t) < 0) { - LM_ERR("failed to delete contact from the database\n"); + LM_ERR("failed to delete contact from the database" + " (aor: %.*s)\n", + ptr->aor->len, ZSW(ptr->aor->s)); } } @@ -349,7 +351,9 @@ static inline void wb_timer(urecord_t* _r) case 1: /* insert */ if (db_insert_ucontact(ptr) < 0) { - LM_ERR("inserting contact into database failed\n"); + LM_ERR("inserting contact into database failed" + " (aor: %.*s)\n", + ptr->aor->len, ZSW(ptr->aor->s)); ptr->state = old_state; } break; @@ -360,7 +364,8 @@ static inline void wb_timer(urecord_t* _r) else res = db_update_ucontact(ptr); if (res < 0) { - LM_ERR("updating contact in db failed\n"); + LM_ERR("updating contact in db failed (aor: %.*s)\n", + ptr->aor->len, ZSW(ptr->aor->s)); ptr->state = old_state; } break;

1 0

git:master: presence: Iterate correctly around presence updates
by Hugh Waite 12 Aug '13

12 Aug '13

Module: sip-router Branch: master Commit: d1cda7ca9a64cf818b82f7b777fa2dd640717c53 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d1cda7c… Author: Hugh Waite <hugh.waite(a)crocodile-rcs.com> Committer: Hugh Waite <hugh.waite(a)crocodile-rcs.com> Date: Mon Aug 12 11:15:55 2013 +0100 presence: Iterate correctly around presence updates - timer_send_notify should increment the subset by one on each loop --- modules/presence/notify.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/modules/presence/notify.c b/modules/presence/notify.c index d06fa6c..0a7ffb2 100644 --- a/modules/presence/notify.c +++ b/modules/presence/notify.c @@ -2667,9 +2667,6 @@ int process_dialogs(int round, int presence_winfo) int updated = 0; db_query_f query_fn = pa_dbf.query_lock ? pa_dbf.query_lock : pa_dbf.query; - if (++subset > (pres_waitn_time * pres_notifier_poll_rate) -1) - subset = 0; - query_cols[n_query_cols] = &str_updated_col; query_vals[n_query_cols].type = DB1_INT; query_vals[n_query_cols].nul = 0; @@ -3000,6 +2997,9 @@ void pres_timer_send_notify(unsigned int ticks, void *param) int round = subset + (pres_waitn_time * pres_notifier_poll_rate * process_num); + if (++subset > (pres_waitn_time * pres_notifier_poll_rate) -1) + subset = 0; + if (process_dialogs(round, 0) < 0) { LM_ERR("Handling non presence.winfo dialogs\n");

1 0

← Newer
1
...
8
9
10
11
12
13
14
15
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2013