sr-dev January 2022

sr-dev@lists.kamailio.org

13 participants
260 discussions

[kamailio/kamailio] Enable CentOS 7 to build with OpenSSL 1.1.1 package from EPEL (#2779)

by space88man

#### Pre-Submission Checklist - [X] Commit message has the format required by CONTRIBUTING guide - [X] Commits are split per component (core, individual modules, libs, utils, ...) - [X] Each component has a single commit (if not, squash them into one commit) - [X] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [X] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [X] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description EPEL now includes a openssl11 package for CentOS 7. This feature is for users who want to build kamailio-tls with OpenSSL 1.1.1 instead of OpenSSL 1.0.2. It is implemented as a `--with=openssl11` conditional (the default os off), so should not affect/change the regular CentOS 7 builds as distributed by kamailio.org. This is for adventurous users on CentOS 7 willing to maintain their own OpenSSL 1.1.1-enabled TLS package. EPEL dependencies( openssl11-libs, openssl11-devel) are required at runtime and for build. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2779 -- Commit Summary -- * pkg: enable CentOS 7 to build with OpenSSL 1.1.1 -- File Changes -- M pkg/kamailio/obs/kamailio.spec (32) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2779.patch https://github.com/kamailio/kamailio/pull/2779.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/2779

2 years, 3 months

[kamailio/kamailio] fedora 36: build warnings (Issue #3014)

by sergey-safarov

**1** ``` CC (gcc) [kamailio] core/crypto/rijndael.o CC (gcc) [kamailio] core/crypto/sha256.o core/crypto/sha256.c:573:32: warning: argument 1 of type 'sha2_byte[]' {aka 'unsigned char[]'} with mismatched bound [-Warray-parameter=] 573 | void sr_SHA256_Final(sha2_byte digest[], SHA256_CTX* context) { | ~~~~~~~~~~^~~~~~~~ In file included from core/crypto/sha256.c:36: core/crypto/sha256.h:157:22: note: previously declared as 'u_int8_t[32]' {aka 'unsigned char[32]'} 157 | void sr_SHA256_Final(u_int8_t[SHA256_DIGEST_LENGTH], SHA256_CTX*); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ core/crypto/sha256.c:636:47: warning: argument 2 of type 'char[]' with mismatched bound [-Warray-parameter=] 636 | char *sr_SHA256_End(SHA256_CTX* context, char buffer[]) { | ~~~~~^~~~~~~~ In file included from core/crypto/sha256.c:36: core/crypto/sha256.h:158:34: note: previously declared as 'char[65]' 158 | char* sr_SHA256_End(SHA256_CTX*, char[SHA256_DIGEST_STRING_LENGTH]); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ core/crypto/sha256.c:936:32: warning: argument 1 of type 'sha2_byte[]' {aka 'unsigned char[]'} with mismatched bound [-Warray-parameter=] 936 | void sr_SHA512_Final(sha2_byte digest[], SHA512_CTX* context) { | ~~~~~~~~~~^~~~~~~~ In file included from core/crypto/sha256.c:36: core/crypto/sha256.h:169:22: note: previously declared as 'u_int8_t[64]' {aka 'unsigned char[64]'} 169 | void sr_SHA512_Final(u_int8_t[SHA512_DIGEST_LENGTH], SHA512_CTX*); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ core/crypto/sha256.c:965:47: warning: argument 2 of type 'char[]' with mismatched bound [-Warray-parameter=] 965 | char *sr_SHA512_End(SHA512_CTX* context, char buffer[]) { | ~~~~~^~~~~~~~ In file included from core/crypto/sha256.c:36: core/crypto/sha256.h:170:34: note: previously declared as 'char[129]' 170 | char* sr_SHA512_End(SHA512_CTX*, char[SHA512_DIGEST_STRING_LENGTH]); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ core/crypto/sha256.c:1011:32: warning: argument 1 of type 'sha2_byte[]' {aka 'unsigned char[]'} with mismatched bound [-Warray-parameter=] 1011 | void sr_SHA384_Final(sha2_byte digest[], SHA384_CTX* context) { | ~~~~~~~~~~^~~~~~~~ In file included from core/crypto/sha256.c:36: core/crypto/sha256.h:163:22: note: previously declared as 'u_int8_t[48]' {aka 'unsigned char[48]'} 163 | void sr_SHA384_Final(u_int8_t[SHA384_DIGEST_LENGTH], SHA384_CTX*); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ core/crypto/sha256.c:1040:47: warning: argument 2 of type 'char[]' with mismatched bound [-Warray-parameter=] 1040 | char *sr_SHA384_End(SHA384_CTX* context, char buffer[]) { | ~~~~~^~~~~~~~ In file included from core/crypto/sha256.c:36: core/crypto/sha256.h:164:34: note: previously declared as 'char[97]' 164 | char* sr_SHA384_End(SHA384_CTX*, char[SHA384_DIGEST_STRING_LENGTH]); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ CC (gcc) [kamailio] core/crypto/shautils.o CC (gcc) [kamailio] core/cfg/cfg.o CC (gcc) [kamailio] core/cfg/cfg_ctx.o core/cfg/cfg_ctx.c: In function 'cfg_update_defaults': core/cfg/cfg_ctx.c:287:28: warning: the comparison will always evaluate as 'true' for the pointer operand in '(unsigned char *)&ginst->vars + (sizetype)var->offset' must not be NULL [-Waddress] 287 | if(ginst->vars + var->offset) { | ^~~~~ CC (gcc) [kamailio] core/cfg/cfg_script.o CC (gcc) [kamailio] core/cfg/cfg_select.o CC (gcc) [kamailio] core/cfg/cfg_struct.o CC (gcc) [kamailio] core/utils/srjson.o CC (gcc) [kamailio] core/utils/sruid.o CC (gcc) [kamailio] core/utils/tmrec.o ``` **2** ``` LD (gcc) [M nat_traversal.so] nat_traversal.so CC (gcc) [M nathelper.so] nathelper.o nathelper.c: In function 'nh_alias_to_uri': nathelper.c:2840:12: warning: the comparison will always evaluate as 'true' for the pointer operand in 'memchr_pointer + 1' must not be NULL [-Waddress] 2840 | if(&memchr_pointer[1]){ | ^ CC (gcc) [M nathelper.so] nhelpr_funcs.o LD (gcc) [M nathelper.so] nathelper.so CC (gcc) [M path.so] path.o ``` **3** ``` CC (gcc) [M ims_dialog.so] dlg_handlers.o CC (gcc) [M ims_dialog.so] dlg_hash.o dlg_hash.c: In function 'destroy_dlg': dlg_hash.c:256:9: warning: the comparison will always evaluate as 'true' for the address of 'dlg_entry_out' will never be NULL [-Waddress] 256 | if (&(dlg->dlg_entry_out)) { | ^ In file included from dlg_var.h:26, from dlg_hash.c:15: dlg_hash.h:124:26: note: 'dlg_entry_out' declared here 124 | struct dlg_entry_out dlg_entry_out; /*!< list of dialog_out entries */ | ^~~~~~~~~~~~~ CC (gcc) [M ims_dialog.so] dlg_ng_stats.o CC (gcc) [M ims_dialog.so] dlg_profile.o ``` **4** ``` CC (gcc) [M app_lua_sr.so] app_lua_sr_api.o CC (gcc) [M app_lua_sr.so] app_lua_sr_exp.o app_lua_sr_exp.c: In function 'lua_sr_exp_openlibs': app_lua_sr_exp.c:3686:17: warning: implicit declaration of function 'luaL_openlib'; did you mean 'luaL_openlibs'? [-Wimplicit-function-declaration] 3686 | luaL_openlib(L, "sr.sl", _sr_sl_Map, 0); | ^~~~~~~~~~~~ | luaL_openlibs CC (gcc) [M app_lua_sr.so] app_lua_sr_mod.o LD (gcc) [M app_lua_sr.so] app_lua_sr.so CC (gcc) [M lwsc.so] lwsc_mod.o LD (gcc) [M lwsc.so] lwsc.so ``` **5** ``` CC (gcc) [M nats.so] nats_pub.o In file included from nats_pub.c:25: defs.h:49:24: warning: '_nats_rts' defined but not used [-Wunused-variable] 49 | static nats_evroutes_t _nats_rts; | ^~~~~~~~~ LD (gcc) [M nats.so] nats.so CC (gcc) [M app_perl.so] app_perl_mod.o CC (gcc) [M app_perl.so] perlfunc.o xsubpp -typemap `perl -MConfig -e 'print $Config{installprivlib}'`/ExtUtils/typemap -typemap typemap kamailioxs.xs > kamailioxs.c CC (gcc) [M app_perl.so] kamailioxs.o LD (gcc) [M app_perl.so] app_perl.so CC (gcc) [M db_perlvdb.so] db_perlvdb.o CC (gcc) [M db_perlvdb.so] perlvdb_conv.o perlvdb_conv.c: In function 'conds2perlarray': perlvdb_conv.c:60:29: warning: the comparison will always evaluate as 'true' for the pointer operand in 'ops + (sizetype)((long unsigned int)i * 8)' must not be NULL [-Waddress] 60 | if (ops + i) | ^~~ CC (gcc) [M db_perlvdb.so] perlvdb_oohelpers.o CC (gcc) [M db_perlvdb.so] perlvdbfunc.o make[3]: 'libsrdb1.so.1.0' is up to date. LD (gcc) [M db_perlvdb.so] db_perlvdb.so CC (gcc) [M phonenum.so] phonenum_mod.o CC (gcc) [M phonenum.so] phonenum_pv.o Compiling cphonenumber.cpp ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3014 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3014(a)github.com>

2 years, 3 months

[kamailio/kamailio] misleading-indentation warning (Issue #3012)

by sergey-safarov

**1** ``` CC (gcc) [M sms.so] libsms_putsms.o CC (gcc) [M sms.so] sms.o CC (gcc) [M sms.so] sms_funcs.o sms_funcs.c: In function 'modem_process': sms_funcs.c:884:17: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 884 | if (used_mem) | ^~ sms_funcs.c:886:25: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 886 | for(i=1,k=1;k<=used_mem && i<=max_mem;i++) { | ^~~ CC (gcc) [M sms.so] sms_report.o LD (gcc) [M sms.so] sms.so CC (gcc) [M sst.so] parse_sst.o ``` **2** ``` CC (gcc) [M ims_charging.so] ims_charging_stats.o CC (gcc) [M ims_charging.so] ims_ro.o ims_ro.c: In function 'create_cca_fui_avps': ims_ro.c:1622:5: warning: this 'else' clause does not guard... [-Wmisleading-indentation] 1622 | else | ^~~~ ims_ro.c:1625:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'else' 1625 | if (redirecturi && redirecturi->len >0 && redirecturi->s) { | ^~ CC (gcc) [M ims_charging.so] ro_avp.o CC (gcc) [M ims_charging.so] ro_db_handler.o CC (gcc) [M ims_charging.so] ro_fixup.o ``` **3** ``` CC (gcc) [M kazoo.so] kazoo.o CC (gcc) [M kazoo.so] kz_amqp.o kz_amqp.c: In function 'kz_amqp_channel_close': kz_amqp.c:802:5: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 802 | if (!rmq) | ^~ In file included from ../../core/mem/pkg.h:32, from ../../core/mem/mem.h:41, from kz_amqp.c:38: ../../core/mem/../dprint.h:298:25: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 298 | do { \ | ^~ ../../core/mem/../dprint.h:345:25: note: in expansion of macro 'LOG_FX' 345 | LOG_FX(facility, level, lname, prefix, _FUNC_NAME_, fmt, ## args) | ^~~~~~ ../../core/mem/../dprint.h:351:25: note: in expansion of macro 'LOG_FL' 351 | LOG_FL(facility, level, NULL, prefix, fmt, ## args) | ^~~~~~ ../../core/mem/../dprint.h:354:25: note: in expansion of macro 'LOG_FP' 354 | LOG_FP(DEFAULT_FACILITY, (level), LOC_INFO, fmt, ## args) | ^~~~~~ ../../core/mem/../dprint.h:401:45: note: in expansion of macro 'LOG' 401 | # define DBG(fmt, args...) LOG(L_DBG, fmt , ## args) | ^~~ ../../core/mem/../dprint.h:405:36: note: in expansion of macro 'DBG' 405 | # define DEBUG(fmt, args...) DBG(fmt , ## args) | ^~~ ../../core/mem/../dprint.h:422:16: note: in expansion of macro 'DEBUG' 422 | #define LM_DBG DEBUG | ^~~~~ kz_amqp.c:805:9: note: in expansion of macro 'LM_DBG' 805 | LM_DBG("close channel: %d rmq(%p)->channel(%d)\n", getpid(), (void *)rmq, channel); | ^~~~~~ kz_amqp.c: In function 'kz_amqp_pipe_send': kz_amqp.c:1132:5: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 1132 | if(s_local_routing_key) | ^~ kz_amqp.c:1135:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 1135 | return ret; | ^~~~~~ kz_amqp.c: In function 'kz_amqp_subscribe_simple': kz_amqp.c:2051:5: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 2051 | if(binding != NULL) | ^~ kz_amqp.c:2054:9: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 2054 | return -1; | ^~~~~~ CC (gcc) [M kazoo.so] kz_fixup.o CC (gcc) [M kazoo.so] kz_hash.o CC (gcc) [M kazoo.so] kz_json.o CC (gcc) [M kazoo.so] kz_pua.o CC (gcc) [M kazoo.so] kz_trans.o ``` **4** ``` CC (gcc) [M ndb_redis.so] ndb_redis_mod.o CC (gcc) [M ndb_redis.so] redis_client.o redis_client.c: In function 'redisc_init': redis_client.c:43:17: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~ redis_client.c:235:21: note: in expansion of macro 'redisCommandNR' 235 | if (redisCommandNR(rsrv->ctxRedis, "PING")) { | ^~~~~~~~~~~~~~ redis_client.c:43:52: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~~~~ redis_client.c:235:21: note: in expansion of macro 'redisCommandNR' 235 | if (redisCommandNR(rsrv->ctxRedis, "PING")) { | ^~~~~~~~~~~~~~ redis_client.c:43:17: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~ redis_client.c:240:51: note: in expansion of macro 'redisCommandNR' 240 | if ((redis_cluster_param == 0) && redisCommandNR(rsrv->ctxRedis, | ^~~~~~~~~~~~~~ redis_client.c:43:52: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~~~~ redis_client.c:240:51: note: in expansion of macro 'redisCommandNR' 240 | if ((redis_cluster_param == 0) && redisCommandNR(rsrv->ctxRedis, | ^~~~~~~~~~~~~~ redis_client.c: In function 'redisc_reconnect_server': redis_client.c:43:17: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~ redis_client.c:549:13: note: in expansion of macro 'redisCommandNR' 549 | if (redisCommandNR(rsrv->ctxRedis, "PING")) | ^~~~~~~~~~~~~~ redis_client.c:43:52: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~~~~ redis_client.c:549:13: note: in expansion of macro 'redisCommandNR' 549 | if (redisCommandNR(rsrv->ctxRedis, "PING")) | ^~~~~~~~~~~~~~ redis_client.c:43:52: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~~~~ redis_client.c:549:13: note: in expansion of macro 'redisCommandNR' 549 | if (redisCommandNR(rsrv->ctxRedis, "PING")) | ^~~~~~~~~~~~~~ redis_client.c:43:17: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~ redis_client.c:551:43: note: in expansion of macro 'redisCommandNR' 551 | if ((redis_cluster_param == 0) && redisCommandNR(rsrv->ctxRedis, | ^~~~~~~~~~~~~~ redis_client.c:43:52: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~~~~ redis_client.c:551:43: note: in expansion of macro 'redisCommandNR' 551 | if ((redis_cluster_param == 0) && redisCommandNR(rsrv->ctxRedis, | ^~~~~~~~~~~~~~ redis_client.c:43:17: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~ redis_client.c:555:21: note: in expansion of macro 'redisCommandNR' 555 | if (redisCommandNR(rsrv->ctxRedis, "FLUSHALL")) | ^~~~~~~~~~~~~~ redis_client.c:43:52: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 43 | if (__tmp) freeReplyObject(__tmp); __tmp ? 0 : -1;}) | ^~~~~ redis_client.c:555:21: note: in expansion of macro 'redisCommandNR' 555 | if (redisCommandNR(rsrv->ctxRedis, "FLUSHALL")) | ^~~~~~~~~~~~~~ LD (gcc) [M ndb_redis.so] ndb_redis.so CC (gcc) [M topos_redis.so] topos_redis_mod.o CC (gcc) [M topos_redis.so] topos_redis_storage.o ``` **5** ``` CC (gcc) [M xhttp_pi.so] xhttp_pi.o CC (gcc) [M xhttp_pi.so] xhttp_pi_fnc.o xhttp_pi_fnc.c: In function 'ph_run_pi_cmd': xhttp_pi_fnc.c:2904:25: warning: this 'if' clause does not guard... [-Wmisleading-indentation] 2904 | if(c_vals) pkg_free(c_vals); c_vals = NULL; | ^~ xhttp_pi_fnc.c:2904:54: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' 2904 | if(c_vals) pkg_free(c_vals); c_vals = NULL; | ^~~~~~ make[3]: 'libsrdb1.so.1.0' is up to date. LD (gcc) [M xhttp_pi.so] xhttp_pi.so CC (gcc) [M xmlrpc.so] http.o CC (gcc) [M xmlrpc.so] xmlrpc.o LD (gcc) [M xmlrpc.so] xmlrpc.so ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3012 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3012(a)github.com>

2 years, 3 months

[kamailio/kamailio] format-truncation warning (Issue #3013)

by sergey-safarov

**1** ``` CC (gcc) [M presence.so] notify.o CC (gcc) [M presence.so] presence.o presence.c: In function 'child_init': presence.c:570:62: warning: '%d' directive output may be truncated writing between 1 and 11 bytes into a region of size 3 [-Wformat-truncation=] 570 | snprintf(tmp, 21, "PRESENCE NOTIFIER %d", i); | ^~ presence.c:570:43: note: directive argument in the range [-2147483647, 2147483646] 570 | snprintf(tmp, 21, "PRESENCE NOTIFIER %d", i); | ^~~~~~~~~~~~~~~~~~~~~~ presence.c:570:25: note: 'snprintf' output between 20 and 30 bytes into a destination of size 21 570 | snprintf(tmp, 21, "PRESENCE NOTIFIER %d", i); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ CC (gcc) [M presence.so] presence_dmq.o CC (gcc) [M presence.so] presentity.o CC (gcc) [M presence.so] publish.o CC (gcc) [M presence.so] subscribe.o CC (gcc) [M presence.so] utils_func.o ``` **2** ``` CC (gcc) [M rls.so] resource_notify.o CC (gcc) [M rls.so] rls.o rls.c: In function 'child_init': rls.c:688:57: warning: '%d' directive output may be truncated writing between 1 and 11 bytes into a region of size 3 [-Wformat-truncation=] 688 | snprintf(tmp, 16, "RLS NOTIFIER %d", i); | ^~ rls.c:688:43: note: directive argument in the range [-2147483647, 2147483646] 688 | snprintf(tmp, 16, "RLS NOTIFIER %d", i); | ^~~~~~~~~~~~~~~~~ rls.c:688:25: note: 'snprintf' output between 15 and 25 bytes into a destination of size 16 688 | snprintf(tmp, 16, "RLS NOTIFIER %d", i); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ CC (gcc) [M rls.so] rls_db.o CC (gcc) [M rls.so] subscribe.o CC (gcc) [M rls.so] utils.o ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3013 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3013(a)github.com>

2 years, 3 months

[kamailio/kamailio] smsops: extend rpdata params manipulation (PR #3007)

by alexyosifov

- Add capability to set RPData originator and destination addresses flags. Default values 0x91.   #### Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3007 -- Commit Summary -- * smsops: extend rpdata params manipulation -- File Changes -- M src/modules/smsops/smsops_impl.c (52) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3007.patch https://github.com/kamailio/kamailio/pull/3007.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3007 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3007(a)github.com>

2 years, 3 months

[kamailio/kamailio] db_postgres: added support for async queries (PR #3010)

by Federico Cabiddu

#### Pre-Submission Checklist    - [X] Commit message has the format required by CONTRIBUTING guide - [X] Commits are split per component (core, individual modules, libs, utils, ...) - [X] Each component has a single commit (if not, squash them into one commit) - [X] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [X] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [X] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  This PR add support for async queries to db_postgres module. Implementation is the same as the mysql and unixodbx modules. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3010 -- Commit Summary -- * db_postgres: added support for async queries -- File Changes -- M src/modules/db_postgres/db_postgres.c (2) M src/modules/db_postgres/km_dbase.c (96) M src/modules/db_postgres/km_dbase.h (9) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3010.patch https://github.com/kamailio/kamailio/pull/3010.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3010 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3010(a)github.com>

2 years, 3 months

Kamailio Glob style include / wildcard.

by Brandon Armstead

Would it be possible to get glob style includes in place, i.e. include_file "/some/path/*.cfg". Reviewing the source, it looks like sr_push_yy_state in src/core/cfg.lex needs an update. Daniel - can you confirm if this is the correct place to modify such behavior and if there are any gotchas in implementation of this? It would be nice if it had full support for glob inclusion / imports, similar to nginx i.e. include_file "/home/*/foo/bar/*.cfg" would include files in /home/brandon/foo/bar/*.cfg and /home/dev/foo/bar/*.cfg although I realize this adds a bit more complexity, any reason you can see why not to add support for this? Thanks!

2 years, 3 months

[kamailio/kamailio] Localy generated transaction with dns failover and socket forcing (Issue #2987)

by Dennis

### Description DNS failover do not wok for localy generated transaction when script forcing socket or sets Via parameters with corex via_add_srvid/via_add_xavp_params, in log appear the errors: ``` Dec 24 13:30:17 sbc kamailio[9772]: 10(9800) DEBUG: <core> [core/parser/parse_uri.c:1306]: parse_uri(): bad uri, state 0 parsed: <<F3>gbf> (4) / <<F3>gbf#017^?> (38) Dec 24 13:30:17 sbc kamailio[9772]: 10(9800) ERROR: tm [ut.h:245]: uri2dst2(): bad_uri: [<F3>gbf#017^?] ``` #### Reproduction Config stripped for localize problem: ``` #!define IPADDR_THISNODE "192.168.122.130" #!subst "/IPADDR_THISNODE/192.168.122.130/" #!define MULTIDOMAIN 0 ####### Global Parameters ######### #!define WITH_DEBUG ### LOG Levels: 3=DBG, 2=INFO, 1=NOTICE, 0=WARN, -1=ERR #!ifdef WITH_DEBUG debug=4 log_stderror=yes #!else debug=2 log_stderror=no #!endif memdbg=5 memlog=5 log_facility=LOG_LOCAL0 log_prefix="{$mt $hdr(CSeq) $ci} " /* number of SIP routing processes */ children=8 server_id = 10 xavp_via_params = "via" /* uncomment the next line to disable TCP (default on) */ disable_tcp=yes /* uncomment the next line to disable the auto discovery of local aliases * based on reverse DNS on IPs (default on) */ # auto_aliases=no /* listen on local and anycast addresses */ listen=udp:IPADDR_THISNODE:5060 #### DNS settings use_dns_cache = on # Use KAMAILIO internal DNS cache use_dns_failover = on # Depends on KAMAILIO internal DNS cache #dns_srv_loadbalancing = on # #dns_try_naptr = on # dns_try_naptr = off # dns_retr_time=1 # Time in seconds before retrying a DNS request dns_retr_no=1 # Number of DNS retransmissions before giving up dns_try_ipv6=yes dns_use_search_list=no ####### Modules Section ######## /* set paths to location of modules */ loadmodule "jsonrpcs.so" loadmodule "kex.so" loadmodule "corex.so" loadmodule "tm.so" loadmodule "tmx.so" loadmodule "sl.so" loadmodule "rr.so" loadmodule "pv.so" loadmodule "maxfwd.so" loadmodule "siputils.so" loadmodule "xlog.so" loadmodule "ctl.so" loadmodule "cfg_rpc.so" loadmodule "counters.so" loadmodule "dispatcher.so" # ----------------- setting module-specific parameters --------------- # ----- jsonrpcs params ----- modparam("jsonrpcs", "pretty_format", 1) /* set the path to RPC fifo control file */ # modparam("jsonrpcs", "fifo_name", "/run/kamailio/kamailio_rpc.fifo") /* set the path to RPC unix socket control file */ # modparam("jsonrpcs", "dgram_socket", "/run/kamailio/kamailio_rpc.sock") # ----- tm params ----- # auto-discard branches from previous serial forking leg modparam("tm", "failure_reply_mode", 3) # ----- tm params ----- modparam("tm", "fr_timer", 2000) #modparam("tm", "reparse_on_dns_failover", 0) # ----- dispatcher params ----- modparam("dispatcher", "list_file", "/etc/kamailio/dispatcher.list") modparam("dispatcher", "ds_ping_interval", 10) modparam("dispatcher", "ds_probing_mode", 1) modparam("dispatcher", "ds_probing_threshold", 2) modparam("dispatcher", "ds_ping_method", "OPTIONS") modparam("dispatcher", "outbound_proxy", "sip:proxy") # ----- corex params ----- modparam("corex", "dns_cache", "type=A;name=proxy;addr=192.168.122.129;ttl=3600000;flags=2") modparam("corex", "dns_cache", "type=A;name=proxy;addr=192.168.122.131;ttl=3600000;flags=2") # ----- rr params ----- # set next param to 1 to add value to ;lr param (helps with some UAs) modparam("rr", "enable_full_lr", 0) # do not append from tag to the RR (no need for this script) modparam("rr", "append_fromtag", 0) onsend_route { xinfo(">> onsend_route snd_to=$sndto(ip)\n"); } event_route [tm:local-request] { xlog("L_INFO", ">> Routing locally generated $rm to $ru\n"); xinfo(">> mbu: $mbu"); $fs = "udp:IPADDR_THISNODE:5060"; // This have the same effect as forcing socket // via_add_srvid(1); t_set_fr(1000,250); } ``` disparcher list have only one target: `110 sip:192.168.122.131:5060;transport=udp 0 1` The scenario is simple - dispatcher try to ping target via proxy, config have two "A" records in cache, all work as expected until tm:local-request route modify Via header by setting $fs or one of the corex module functions which set flags FL_ADD_SRVID|FL_ADD_XAVP_VIA_PARAMS|FL_USE_XAVP_VIA_FIELDS, tm in this case (after error as above) decline failover. #### Debugging Data transaction fail log: ``` Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: dispatcher [dispatch.c:3546]: ds_ping_result_helper(): probe all, mode DS_PROBE_ALL Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: dispatcher [dispatch.c:3589]: ds_ping_set(): probing set #110, URI sip:192.168.122.131:5060;transport=udp Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: dispatcher [dispatch.c:3620]: ds_ping_set(): Default ping_from: sip:dispatcher@localhost Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: dispatcher [dispatch.c:3630]: ds_ping_set(): Default outbound proxy: sip:proxy Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: tm [uac.c:457]: t_uac_prepare(): next_hop=<sip:proxy> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (_sip._udp.proxy(15), 33), h=170 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/dns_cache.c:3050]: dns_srv_resolve_ip(): ("_sip._udp.proxy", 0, 0), ret=-5, ip= Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/dns_cache.c:527]: _dns_hash_find(): (proxy(5), 1), h=878 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/dns_cache.c:2807]: dns_a_resolve(): (proxy, 0) returning 0 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/dns_cache.c:3176]: dns_srv_sip_resolve(): (proxy, 0, 0), ip, ret=0 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: tm [uac.c:158]: dlg2hash(): hashid 55725 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: tm [uac.c:246]: t_run_local_req(): executing event_route[tm:local-request] Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_fline.c:249]: parse_first_line(): first line type 1 (request) flags 1 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:677]: parse_msg(): SIP Request: Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:679]: parse_msg(): method: <OPTIONS> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:681]: parse_msg(): uri: <sip:192.168.122.131:5060;transport=udp> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:683]: parse_msg(): version: <SIP/2.0> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [Via] type 1 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_via.c:1303]: parse_via_param(): Found param type 232, <branch> = <z9hG4bKda9d.ac749307000000000000000000000000.0>; state=16 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_via.c:2639]: parse_via(): end of header reached, state=5 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:555]: parse_headers(): Via found, flags=2 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:557]: parse_headers(): this is the first via Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) INFO: <script>: >> Routing locally generated OPTIONS to sip:192.168.122.131:5060;transport=udp Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/msg_translator.c:1799]: check_boundaries(): no multi-part body Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) INFO: <script>: >> mbu: OPTIONS sip:192.168.122.131:5060;transport=udp SIP/2.0 Dec 24 13:15:37 sbc kamailio[9772]: Via: SIP/2.0/UDP 192.168.122.130;branch=z9hG4bKda9d.ac749307000000000000000000000000.0 Dec 24 13:15:37 sbc kamailio[9772]: To: <sip:192.168.122.131:5060;transport=udp> Dec 24 13:15:37 sbc kamailio[9772]: From: <sip:dispatcher@localhost>;tag=e92e326f0a937cb326014225217c2c12-5929046b Dec 24 13:15:37 sbc kamailio[9772]: CSeq: 10 OPTIONS Dec 24 13:15:37 sbc kamailio[9772]: Call-ID: 62100fd077a3a243-9799(a)192.168.122.130 Dec 24 13:15:37 sbc kamailio[9772]: Max-Forwards: 70 Dec 24 13:15:37 sbc kamailio[9772]: Content-Length: 0 Dec 24 13:15:37 sbc kamailio[9772]: User-Agent: kamailio (5.5.3 (x86_64/linux)) Dec 24 13:15:37 sbc kamailio[9772]: #015 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: pv [pv_core.c:2703]: pv_set_force_sock(): trying to set send-socket to [udp:192.168.122.130:5060] Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/socket_info.c:641]: grep_sock_info(): checking if host==us: 15==15 && [192.168.122.130] == [192.168.122.130] Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/socket_info.c:648]: grep_sock_info(): checking if port 5060 (advertise 0) matches port 5060 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: tm [uac.c:314]: t_run_local_req(): local Via update - new socket: [192.168.122.130] - msg-flags: 0 9(9799) DEBUG: tm [uac.c:340]: t_run_local_req(): apply new updates with Via to sip msg Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/msg_translator.c:1799]: check_boundaries(): no multi-part body Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_fline.c:249]: parse_first_line(): first line type 1 (request) flags 1 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:677]: parse_msg(): SIP Request: Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:679]: parse_msg(): method: <OPTIONS> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:681]: parse_msg(): uri: <sip:192.168.122.131:5060;transport=udp> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:683]: parse_msg(): version: <SIP/2.0> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [Via] type 1 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_via.c:1303]: parse_via_param(): Found param type 232, <branch> = <z9hG4bKda9d.ac749307000000000000000000000000.0>; state=16 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_via.c:2639]: parse_via(): end of header reached, state=5 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:555]: parse_headers(): Via found, flags=2 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:557]: parse_headers(): this is the first via Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [To] type 3 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_addr_spec.c:884]: parse_addr_spec(): end of header reached, state=10 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:173]: get_hdr_field(): <To> [42]; uri=[sip:192.168.122.131:5060;transport=udp] Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:176]: get_hdr_field(): to body (42)[<sip:192.168.122.131:5060;transport=udp> Dec 24 13:15:37 sbc kamailio[9772]: ], to tag (0)[] Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [From] type 4 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [CSeq] type 5 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:154]: get_hdr_field(): cseq <CSeq>: <10> <OPTIONS> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [Call-ID] type 6 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [Max-Forwards] type 8 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [Content-Length] type 12 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:187]: get_hdr_field(): content_length=0 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [User-Agent] type 28 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:91]: get_hdr_field(): found end of header Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_fline.c:249]: parse_first_line(): first line type 1 (request) flags 1 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:677]: parse_msg(): SIP Request: Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:679]: parse_msg(): method: <OPTIONS> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:681]: parse_msg(): uri: <sip:192.168.122.131:5060;transport=udp> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:683]: parse_msg(): version: <SIP/2.0> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [Via] type 1 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_via.c:1303]: parse_via_param(): Found param type 232, <branch> = <z9hG4bKda9d.ac749307000000000000000000000000.0>; state=16 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_via.c:2639]: parse_via(): end of header reached, state=5 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:555]: parse_headers(): Via found, flags=2 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:557]: parse_headers(): this is the first via Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [To] type 3 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_addr_spec.c:884]: parse_addr_spec(): end of header reached, state=10 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:173]: get_hdr_field(): <To> [42]; uri=[sip:192.168.122.131:5060;transport=udp] Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:176]: get_hdr_field(): to body (42)[<sip:192.168.122.131:5060;transport=udp> Dec 24 13:15:37 sbc kamailio[9772]: ], to tag (0)[] Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [From] type 4 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [CSeq] type 5 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/msg_parser.c:154]: get_hdr_field(): cseq <CSeq>: <10> <OPTIONS> Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: <core> [core/parser/parse_hname2.c:293]: parse_sip_header_name(): parsed header name [Call-ID] type 6 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: tm [uac.c:189]: uac_refresh_hdr_shortcuts(): cseq: [CSeq: 10] Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) DEBUG: tm [uac.c:685]: send_prepared_request_impl(): uac: 0x7f0f6262b9c8 branch: 0 to 192.168.122.131:5060 Dec 24 13:15:37 sbc kamailio[9772]: 9(9799) INFO: <script>: >> onsend_route snd_to=192.168.122.131 Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: <core> [core/parser/parse_uri.c:1306]: parse_uri(): bad uri, state 0 parsed: <<F3>gbf> (4) / <<F3>gbf#017^?> (38) Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) ERROR: tm [ut.h:245]: uri2dst2(): bad_uri: [<F3>gbf#017^?] Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: tm [t_reply.c:1363]: t_should_relay_response(): ->>>>>>>>> T_code=0, new_code=408 Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: tm [t_reply.c:1592]: t_should_relay_response(): rps completed - uas status: 0 branch: 0 Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: tm [t_reply.c:2236]: local_reply(): branch=0, save=0, winner=0 Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: tm [t_reply.c:2275]: local_reply(): local transaction completed 408/0 (totag retr: 0/1024) Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: tm [t_hooks.c:256]: run_trans_callbacks_internal(): DBG: trans=0x7f0f6262b738, callback type 1024, id 0 entered Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: dispatcher [dispatch.c:3491]: ds_options_callback(): OPTIONS-Request was finished with code 408 (to sip:192.168.122.131:5060;transport=udp, group 110) Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: dispatcher [dispatch.c:2973]: ds_update_state(): update state for sip:192.168.122.131:5060;transport=udp in group 110 to 10 Dec 24 13:15:37 sbc kamailio[9772]: 10(9800) DEBUG: dispatcher [dispatch.c:3049]: ds_update_state(): old state was 9, set new state to 9 ``` I think that error somewhere in memory management when allocating/free structures for Via header modification which is hard to debug, i have insert several debug output in uac.c in my case the problem fire after a call of function insert_tmcb: ``` /* Register the callbacks after everything is successful and nothing can fail. Otherwise the callback parameter would be freed twise, once from TMCB_DESTROY, and again because of the negative return code. */ {// extra debug struct ua_client *uac=&request->my_T->uac[0]; LM_DBG(">>>>> Before insert_tmcb\n"); LM_DBG(">>>>> request uac: %p urip: %p uri: %.*s\n", uac, &uac->uri, STR_FMT(&uac->uri)); uac=&new_cell->uac[0]; LM_DBG(">>>>> new_cell uac: %p urip: %p uri: %.*s\n", uac, &uac->uri, STR_FMT(&uac->uri)); } if(uac_r->cb && insert_tmcb(&(new_cell->tmcb_hl), uac_r->cb_flags, *(uac_r->cb), uac_r->cbp, NULL)!=1){ ret=E_OUT_OF_MEM; LM_ERR("short of tmcb shmem\n"); goto error1; } {// extra debug struct ua_client *uac=&request->my_T->uac[0]; LM_DBG(">>>>> After insert_tmcb\n"); LM_DBG(">>>>> request uac: %p urip: %p uri: %.*s\n", uac, &uac->uri, STR_FMT(&uac->uri)); uac=&new_cell->uac[0]; LM_DBG(">>>>> new_cell uac: %p urip: %p uri: %.*s\n", uac, &uac->uri, STR_FMT(&uac->uri)); } ``` In log i see: ``` Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) DEBUG: tm [uac.c:189]: uac_refresh_hdr_shortcuts(): cseq: [CSeq: 10] Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) DEBUG: tm [uac.c:591]: t_uac_prepare(): >>>>> Before insert_tmcb Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) DEBUG: tm [uac.c:592]: t_uac_prepare(): >>>>> request uac: 0x7f72407959c8 urip: 0x7f7240795bd8 uri: sip:192.168.122.131:5060;transport=udp Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) DEBUG: tm [uac.c:594]: t_uac_prepare(): >>>>> new_cell uac: 0x7f72407959c8 urip: 0x7f7240795bd8 uri: sip:192.168.122.131:5060;transport=udp Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) DEBUG: tm [uac.c:604]: t_uac_prepare(): >>>>> After insert_tmcb Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) DEBUG: tm [uac.c:605]: t_uac_prepare(): >>>>> request uac: 0x7f72407959c8 urip: 0x7f7240795bd8 uri: <F3>#007yDr^? Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) DEBUG: tm [uac.c:607]: t_uac_prepare(): >>>>> new_cell uac: 0x7f72407959c8 urip: 0x7f7240795bd8 uri: <F3>#007yDr^? Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) DEBUG: tm [uac.c:699]: send_prepared_request_impl(): uac: 0x7f72407959c8 branch: 0 to 192.168.122.131:5060 Dec 24 14:36:39 sbc kamailio[12709]: 9(12732) INFO: <script>: >> onsend_route snd_to=192.168.122.131 Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: <core> [core/parser/parse_uri.c:1306]: parse_uri(): bad uri, state 0 parsed: <<F3>#007yD> (4) / <<F3>#007yDr^?> (38) Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) ERROR: tm [ut.h:245]: uri2dst2(): bad_uri: [<F3>#007yDr^?] Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: tm [t_reply.c:1363]: t_should_relay_response(): ->>>>>>>>> T_code=0, new_code=408 Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: tm [t_reply.c:1592]: t_should_relay_response(): rps completed - uas status: 0 branch: 0 Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: tm [t_reply.c:2236]: local_reply(): branch=0, save=0, winner=0 Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: tm [t_reply.c:2275]: local_reply(): local transaction completed 408/0 (totag retr: 0/1024) Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: tm [t_hooks.c:256]: run_trans_callbacks_internal(): DBG: trans=0x7f7240795738, callback type 1024, id 0 entered Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: dispatcher [dispatch.c:3491]: ds_options_callback(): OPTIONS-Request was finished with code 408 (to sip:192.168.122.131:5060;transport=udp, group 110) Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: dispatcher [dispatch.c:2973]: ds_update_state(): update state for sip:192.168.122.131:5060;transport=udp in group 110 to 10 Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: dispatcher [dispatch.c:3004]: ds_update_state(): destination did not replied 1 times, threshold 2 Dec 24 14:36:40 sbc kamailio[12709]: 10(12733) DEBUG: dispatcher [dispatch.c:3049]: ds_update_state(): old state was 0, set new state to 10 ``` insert_tmcb have shm_malloc fincion call in the beninig, after this call the content of uri structure is broken. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.5.3 (x86_64/linux) 473cef flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. ``` * **Operating System**: ``` Ubuntu 18.04.6 LTS ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/2987 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/2987(a)github.com>

2 years, 3 months

git:master:4031260f: modules: readme files regenerated - nathelper ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: 4031260fbb21a0330651977fdcaf4708dc270497 URL: https://github.com/kamailio/kamailio/commit/4031260fbb21a0330651977fdcaf470… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2022-01-31T19:46:12+01:00 modules: readme files regenerated - nathelper ... [skip ci] --- Modified: src/modules/nathelper/README --- Diff: https://github.com/kamailio/kamailio/commit/4031260fbb21a0330651977fdcaf470… Patch: https://github.com/kamailio/kamailio/commit/4031260fbb21a0330651977fdcaf470…

2 years, 3 months

git:master:36320b7a: nathelper: docs for alias_name parameter

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 36320b7a22982cc6d2aff610b325bf9620d99239 URL: https://github.com/kamailio/kamailio/commit/36320b7a22982cc6d2aff610b325bf9… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-01-31T19:32:35+01:00 nathelper: docs for alias_name parameter --- Modified: src/modules/nathelper/doc/nathelper_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/36320b7a22982cc6d2aff610b325bf9… Patch: https://github.com/kamailio/kamailio/commit/36320b7a22982cc6d2aff610b325bf9… --- diff --git a/src/modules/nathelper/doc/nathelper_admin.xml b/src/modules/nathelper/doc/nathelper_admin.xml index 832e324819..d0e8a18ea4 100644 --- a/src/modules/nathelper/doc/nathelper_admin.xml +++ b/src/modules/nathelper/doc/nathelper_admin.xml @@ -508,6 +508,27 @@ modparam("nathelper", "filter_server_id", 1) ... modparam("nathelper", "nat_addr_mode", 0) ... +</programlisting> + </example> + </section> + <section id="nathelper.p.alias_name"> + <title><varname>alias_name</varname> (string)</title> + <para> + The parameter sets the name of the Contact URI parameter set by + set_contact_alias() or add_contact_alias() and processed by + handle_ruri_alias() in R-URI. + </para> + <para> + <emphasis> + Default value is <quote>alias</quote>. + </emphasis> + </para> + <example> + <title>Set <varname>alias_name</varname> parameter</title> + <programlisting format="linespecific"> +... +modparam("nathelper", "alias_name", "saddr") +... </programlisting> </example> </section>

2 years, 3 months

← Newer
1
...
7
8
9
10
11
12
13
...
26
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2022