sr-dev June 2013

sr-dev@lists.kamailio.org

33 participants
235 discussions

git:master: modules/app_lua: fixed runtime warning: exports dlflags interface is deprecated

by Victor Seva

Module: sip-router Branch: master Commit: 03188db86347013ef43d3615c12be4b696349098 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=03188db… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: Mon Jun 17 10:48:45 2013 +0200 modules/app_lua: fixed runtime warning: exports dlflags interface is deprecated --- modules/app_lua/app_lua_mod.c | 8 +++++++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/modules/app_lua/app_lua_mod.c b/modules/app_lua/app_lua_mod.c index 3d18f44..da39bce 100644 --- a/modules/app_lua/app_lua_mod.c +++ b/modules/app_lua/app_lua_mod.c @@ -92,7 +92,7 @@ static cmd_export_t cmds[]={ struct module_exports exports = { "app_lua", - RTLD_NOW | RTLD_GLOBAL, /* dlopen flags */ + DEFAULT_DLFLAGS, /* dlopen flags */ cmds, params, 0, @@ -105,6 +105,12 @@ struct module_exports exports = { child_init /* per child init function */ }; +int mod_register(char *path, int *dlflags, void *p1, void *p2) +{ + *dlflags = RTLD_NOW | RTLD_GLOBAL; + return 0; +} + /** * init module function */

10 years, 10 months

[tracker] Comment added: Patch to Add s.urlencode.param (Attachment added)

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#311 - Patch to Add s.urlencode.param User who did this - Josh (JoshE) ---------- Updated patch to reflect what I actually implemented. This is working in my environment now. ---------- 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=311#comment965 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.

10 years, 10 months

[tracker] Task closed: Crash in topoh module when To header missing

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#303 - Crash in topoh module when To header missing User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Fixed More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=303 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.

10 years, 10 months

[tracker] Comment added: Crash in topoh module when To header missing

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#303 - Crash in topoh module when To header missing User who did this - Daniel-Constantin Mierla (miconda) ---------- Added a safety check for the case when sanity module checks are not enabled. It is committed in master branch for now - 4f3d04d547c66a1b59398cf80e93974175141514 Reopen if the issue is still there. ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=303#comment964 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.

10 years, 10 months

git:master: topoh: safety check for To header

by Daniel-Constantin Mierla

Module: sip-router Branch: master Commit: 4f3d04d547c66a1b59398cf80e93974175141514 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=4f3d04d… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Sun Jun 16 17:17:36 2013 +0200 topoh: safety check for To header - protection for the case when sanity module checks are not enabled - reported in FS#303 by Michel de Weerd --- modules/topoh/topoh_mod.c | 8 +++++++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/modules/topoh/topoh_mod.c b/modules/topoh/topoh_mod.c index 73ebf21..e5e7058 100644 --- a/modules/topoh/topoh_mod.c +++ b/modules/topoh/topoh_mod.c @@ -227,7 +227,13 @@ int th_prepare_msg(sip_msg_t *msg) LM_ERR("cannot parse FROM header\n"); return 3; } - + + if(get_to(msg)==NULL) + { + LM_ERR("cannot parse TO header\n"); + return 3; + } + return 0; }

10 years, 10 months

[tracker] Comment added: Crash in topoh module when To header missing

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#303 - Crash in topoh module when To header missing User who did this - Daniel-Constantin Mierla (miconda) ---------- You have to set another parameter for topoh: modparam("topoh", "sanity_checks", 1) ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=303#comment963 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.

10 years, 10 months

[tracker] Comment added: modules/app_lua: fixed runtime warning: exports dlflags interface is deprecated

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#312 - modules/app_lua: fixed runtime warning: exports dlflags interface is deprecated User who did this - Daniel-Constantin Mierla (miconda) ---------- Patch can be committed. ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=312#comment962 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.

10 years, 10 months

[tracker] Task closed: K. crush with radiusclient.conf error

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#310 - K. crush with radiusclient.conf error User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Won't fix More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=310 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.

10 years, 10 months

[tracker] Comment added: K. crush with radiusclient.conf error

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#310 - K. crush with radiusclient.conf error User who did this - Daniel-Constantin Mierla (miconda) ---------- The crash is inside the radiusclient-ng library: #0 0x000000080881f71c in rc_read_config () from /usr/local/lib/libradiusclient-ng.so.2 It is nothing we can do in kamailio acc_radius module. It has to be fixed in the library. ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=310#comment961 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.

10 years, 10 months

[tracker] Comment added: modules/debugger: add dbg_pv_dump config function. It dumps on json format the content of pv_cache

by sip-router

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#307 - modules/debugger: add dbg_pv_dump config function. It dumps on json format the content of pv_cache User who did this - Daniel-Constantin Mierla (miconda) ---------- In first patch, you do pkg_malloc but don't check if you get a non-NULL result. Then you memset and fill the structure. In second patch, there is a return 0 in the function executed by config, meaning that config file execution will be interrupted (like being an exit) -- probably you wanted to be FALSE in config check, so should be -1 (or other negative value). Otherwise (once the above are fixed), the patches could be committed -- the part in core in modules are related only to this functionality, not interfering with other code. But I think cloning all the items in the cache each time you want to use this function is a bit of overhead, although it is just for debugging purposes, thus not expected to be heavily used in production. Therefore I exposed the pv cache via pv_cache_get_table() function, its size is also defined in pvar.h. Now you can walk through the cache from the module. If you decide to go for this option, your first patch is no longer needed, second patch needs to be adapted. One improvement that I would suggest - add a second parameter (optional) to the new config function to specify the log level -- now it prints to DEBUG, implying that debug=3 must be used. Also, in the readme, you only pasted the defines for the flags that can be used in the first parameter. You should explain a bit what each is supposed to do. ---------- More information can be found at the following URL: https://sip-router.org/tracker/index.php?do=details&task_id=307#comment960 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.

10 years, 10 months

← Newer
1
...
5
6
7
8
9
10
11
...
24
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev June 2013