Ambiguous was possible in params for multiple modules in one config line. For example ``` loadmodule "presence_dialoginfo.so" modparam("presence|dialplan", "db_url", "mysql://DBUSER:DBPASSWORD@localhost/DBNAME") ``` produced error `parameter <db_url> of type <1> not found in module <presence_dialoginfo>` because of ["presence_dialoginfo" =~ /^presence|dialplan$/] => True You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/703
-- Commit Summary --
* core: fix regex error for modules parameters
-- File Changes --
M modparam.c (8)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/703.patch https://github.com/kamailio/kamailio/pull/703.diff
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/703
The size in pkg_malloc() needs to be increased, too -- from:
``` reg = pkg_malloc(len + 2 + 1); ```
to:
``` reg = pkg_malloc(len + 4 + 1); ```
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/703#issuecomment-231580428
right you are! changed
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/703#issuecomment-231592386
Merged #703.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/703#event-718324185
-
Daniel-Constantin Mierla -
Dmitri Savolainen