sr-dev January 2021

sr-dev@lists.kamailio.org

17 participants
185 discussions

[kamailio/kamailio] lost: new features and http_client fix (#2606)
by Wolfgang Kampichler 27 Jan '21

27 Jan '21

#### 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 - [x] 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:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [x] Related to issue #2574 #### Description  ##### http_client: http_client_request shall include default clientcert, clientkey,... The lost module uses http_client API functions and in the course of NG112 client certificates are used for authentication when querying LIS or ECRF, the fix allows these to be read out via http_client module parameters. ##### lost: new features LoST redirect and NAPTR lookup (LIS, LoST) The extension of the module allows dynamic querying of LIS/HELD and LOST services via NAPTR lookup. In the case of LOST, a redirect response is evaluated. In case a lost_held_request (used to connect to a LIS) is passed with an empty string ("") for the connection parameter, then P-A-I or From header value hostnames are used for NAPTR lookup to get a corresponding service. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2606 -- Commit Summary -- * initial commit -- File Changes -- M src/modules/http_client/functions.c (12) M src/modules/lost/functions.c (575) M src/modules/lost/lost.c (5) A src/modules/lost/naptr.c (254) A src/modules/lost/naptr.h (38) M src/modules/lost/pidf.c (5) A src/modules/lost/response.c (946) A src/modules/lost/response.h (131) M src/modules/lost/utilities.c (229) M src/modules/lost/utilities.h (5) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2606.patch https://github.com/kamailio/kamailio/pull/2606.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/2606

4 19

Errored: kamailio/kamailio#9832 (master - 5a6a679)
by Travis CI 27 Jan '21

27 Jan '21

Build Update for kamailio/kamailio ------------------------------------- Build: #9832 Status: Errored Duration: 5 mins and 37 secs Commit: 5a6a679 (master) Author: Daniel-Constantin Mierla Message: tm: prper timer cleanup of responses for rpc tm.t_uac_wait_block View the changeset: https://github.com/kamailio/kamailio/compare/34301ea16812...5a6a679985fe View the full build log and details: https://travis-ci.org/github/kamailio/kamailio/builds/756409697?utm_medium=… -- You can unsubscribe from build emails from the kamailio/kamailio repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=3596148&ut…. Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notificati…. Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

1 0

git:master:5a6a6799: tm: prper timer cleanup of responses for rpc tm.t_uac_wait_block
by Daniel-Constantin Mierla 27 Jan '21

27 Jan '21

Module: kamailio Branch: master Commit: 5a6a679985fecfc168b9b26f0be3c0e64e641e64 URL: https://github.com/kamailio/kamailio/commit/5a6a679985fecfc168b9b26f0be3c0e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-01-27T14:10:34+01:00 tm: prper timer cleanup of responses for rpc tm.t_uac_wait_block --- Modified: src/modules/tm/rpc_uac.c --- Diff: https://github.com/kamailio/kamailio/commit/5a6a679985fecfc168b9b26f0be3c0e… Patch: https://github.com/kamailio/kamailio/commit/5a6a679985fecfc168b9b26f0be3c0e… --- diff --git a/src/modules/tm/rpc_uac.c b/src/modules/tm/rpc_uac.c index 94782dde00..4ca2274c9d 100644 --- a/src/modules/tm/rpc_uac.c +++ b/src/modules/tm/rpc_uac.c @@ -204,14 +204,16 @@ void tm_rpc_response_list_clean(unsigned int ticks, void *param) ri1 = _tm_rpc_response_list->rlist; while(ri1!=NULL) { if(ri1->rtime < tnow - TM_RPC_RESPONSE_LIFETIME) { + LM_DBG("freeing item [%.*s]\n", ri1->ruid.len, ri1->ruid.s); if(ri0 == NULL) { _tm_rpc_response_list->rlist = ri1->next; + shm_free(ri1); + ri1 = _tm_rpc_response_list->rlist; } else { ri0->next = ri1->next; + shm_free(ri1); + ri1 = ri0->next; } - LM_DBG("freeing item [%.*s]\n", ri1->ruid.len, ri1->ruid.s); - shm_free(ri1); - ri1 = ri0->next; } else { ri0 = ri1; ri1 = ri1->next;

1 0

event_route[core:receive-parse-error] missing from docs
by Henning Westerholt 27 Jan '21

27 Jan '21

Hello, I noticed that the core "event_route[core:receive-parse-error]" was missing from the cookbook, so I have added it there. https://www.kamailio.org/wiki/cookbooks/devel/core#event_route Cheers, Henning -- Henning Westerholt - https://skalatan.de/blog/ Kamailio services - https://gilawa.com<https://gilawa.com/>

1 0

[kamailio/kamailio] build warings (#2611)
by sergey-safarov 26 Jan '21

26 Jan '21

**core/ut.h:751** ``` core/ut.h:751:13: warning: ‘shm_str_dup_block’ defined but not used [-Wunused-function] static str* shm_str_dup_block(const str* src) ^~~~~~~~~~~~~~~~~ CC (gcc) [kamailio] core/proxy.o In file included from core/ip_addr.h:39, from core/proxy.h:32, from core/proxy.c:33: ``` -- 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/issues/2611

2 2

git:master:34301ea1: core: ut.h - add inline to static function to avoid warning from including files
by Daniel-Constantin Mierla 26 Jan '21

26 Jan '21

Module: kamailio Branch: master Commit: 34301ea16812eccbb914ea14cafb0140ac9bba61 URL: https://github.com/kamailio/kamailio/commit/34301ea16812eccbb914ea14cafb014… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2021-01-26T11:54:34+01:00 core: ut.h - add inline to static function to avoid warning from including files - for shm_str_dup_block() --- Modified: src/core/ut.h --- Diff: https://github.com/kamailio/kamailio/commit/34301ea16812eccbb914ea14cafb014… Patch: https://github.com/kamailio/kamailio/commit/34301ea16812eccbb914ea14cafb014… --- diff --git a/src/core/ut.h b/src/core/ut.h index 99e12a4136..4ab08024a7 100644 --- a/src/core/ut.h +++ b/src/core/ut.h @@ -748,7 +748,7 @@ static inline int strz2sint(char* _s, int* _r) /** * duplicate str structure and content in a single shm block */ -static str* shm_str_dup_block(const str* src) +static inline str* shm_str_dup_block(const str* src) { str *dst;

1 0

[kamailio/kamailio] UAC send SIP requests but no socket found and no corresponding socket found (#2610)
by lbli 26 Jan '21

26 Jan '21

I had the same problem,but it did not fix issue commit a752411 kamailio.cfg uac block below: #!define DBURL "sqlite:///app-service/cloudland/edge-server/etc/kamailio/kamailio.db" modparam("rr", "append_fromtag", 1) #modparam("ctl", "binrpc", "unix:/var/run/kamailio/kamailio_ctl") modparam("uac", "reg_db_url", DBURL); modparam("uac", "reg_contact_addr", "172.26.0.83:5060"); #modparam("uac", "reg_keep_callid", 1); modparam("uac", "reg_timer_interval", 10); modparam("uac", "reg_retry_interval", 20); sqlite database has a user in the uacreg table with the socket column set to udp:172.26.0.83:5060 details as below sqlite> select * from uacreg; id|l_uuid|l_username|l_domain|r_username|r_domain|realm|auth_username|auth_password|auth_ha1|auth_proxy|expires|flags|reg_delay|socket 1|1002|1002|172.26.0.83|1002|172.26.0.83|172.26.0.83|1002|1002|23598b35f7c629a6790d79ae72934856|sip:172.26.0.83|360|0|0|udp:172.26.0.83:5060 Log Messages 13(21432) DEBUG: db_sqlite [dbase.c:223]: db_sqlite_submit_query(): submit_query: select l_uuid,l_username,l_domain,r_username,r_domain,realm,auth_username,auth_password,auth_ha1,auth_proxy,expires,flags,reg_delay,socket from uacreg 13(21432) DEBUG: [db_query.c:72]: db_do_submit_query(): submitted query: select l_uuid,l_username,l_domain,r_username,r_domain,realm,auth_username,auth_password,auth_ha1,auth_proxy,expires,flags,reg_delay,socket from uacreg , result 0 13(21432) DEBUG: [db_res.c:120]: db_new_result(): allocate 56 bytes for result set at 0x7efd8c53c768 13(21432) DEBUG: [db_res.c:158]: db_allocate_columns(): allocate 112 bytes for result names at 0x7efd8c53c808 13(21432) DEBUG: [db_res.c:169]: db_allocate_columns(): allocate 56 bytes for result types at 0x7efd8c53c8e0 13(21432) DEBUG: [db_row.c:117]: db_allocate_row(): allocate 448 bytes for row values at 0x7efd8c5c5cc8 13(21432) DEBUG: uac [uac_reg.c:581]: reg_ht_add(): added uuid: 1002 - l_user: 1002 13(21432) DEBUG: [db_res.c:79]: db_free_columns(): freeing 14 columns 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[0] at 0x7efd8c527cb0 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[1] at 0x7efd8c527d30 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[2] at 0x7efd8c527db8 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[3] at 0x7efd8c527e40 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[4] at 0x7efd8c527ec8 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[5] at 0x7efd8c527f50 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[6] at 0x7efd8c527fd0 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[7] at 0x7efd8c528058 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[8] at 0x7efd8c5280e0 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[9] at 0x7efd8c528168 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[10] at 0x7efd8c5c59d8 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[11] at 0x7efd8c5c5a58 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[12] at 0x7efd8c5c5ad8 13(21432) DEBUG: [db_res.c:83]: db_free_columns(): freeing RES_NAMES[13] at 0x7efd8c5c5b60 13(21432) DEBUG: [db_res.c:96]: db_free_columns(): freeing result names at 0x7efd8c53c808 13(21432) DEBUG: [db_res.c:101]: db_free_columns(): freeing result types at 0x7efd8c53c8e0 13(21432) DEBUG: [db_res.c:52]: db_free_rows(): freeing 1 rows 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[0] '1002' at 0x7efd8c4fe160 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[1] '1002' at 0x7efd8c4a4160 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[2] '172.26.0.83' at 0x7efd8c4fdba8 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[3] '1002' at 0x7efd8c5c5ef0 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[4] '172.26.0.83' at 0x7efd8c5c5f60 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[5] '172.26.0.83' at 0x7efd8c5c5fd8 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[6] '1002' at 0x7efd8c5c6050 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[7] '1002' at 0x7efd8c5c60c0 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[8] '23598b35f7c629a6790d79ae72934856' at 0x7efd8c5c6130 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[9] 'sip:172.26.0.83' at 0x7efd8c597e60 13(21432) DEBUG: [db_row.c:66]: db_free_row(): free VAL_STRING[13] 'udp:172.26.0.83:5060' at 0x7efd8c597ed8 13(21432) DEBUG: [db_row.c:95]: db_free_row(): freeing row values at 0x7efd8c5c5cc8 13(21432) DEBUG: [db_res.c:60]: db_free_rows(): freeing rows at 0x7efd8c5c5be0 13(21432) DEBUG: [db_res.c:138]: db_free_result(): freeing result set at 0x7efd8c53c768 13(21432) DEBUG: [db_pool.c:100]: pool_remove(): removing connection from the pool 13(21432) DEBUG: uac [uac.c:413]: child_init(): run initial uac registration routine 13(21432) DEBUG: uac [uac_reg.c:1137]: uac_reg_update(): using custom socket udp:172.26.0.83:5060 to send request 13(21432) DEBUG: tm [uac.c:451]: t_uac_prepare(): next_hop=sip:172.26.0.83 13(21432) ERROR: [core/forward.c:182]: get_out_socket(): no socket found 13(21432) ERROR: [core/forward.c:184]: get_out_socket(): no corresponding socket found for(udp:172.26.0.83:5060) 13(21432) ERROR: tm [ut.h:317]: uri2dst2(): no corresponding socket found for "172.26.0.83" af 2 (udp:172.26.0.83:5060) 13(21432) ERROR: tm [uac.c:473]: t_uac_prepare(): no socket found 13(21432) DEBUG: tm [h_table.c:132]: free_cell_helper(): freeing transaction 0x7efd82bfafa8 from uac.c:614 13(21432) ERROR: uac [uac_reg.c:1177]: uac_reg_update(): failed to send request for [1002] 6(21425) DEBUG: ctl [../../core/io_wait.h:375]: io_watch_add(): DBG: io_watch_add(0x7efd894923e0, 9, 3, 0x25115f0), fd_no=1 6(21425) DEBUG: ctl [io_listener.c:442]: handle_new_connect(): handle_stream read: new connection (1) on /var/run/kamailio/kamailio_ctl 6(21425) INFO: [core/cfg/cfg_ctx.c:604]: cfg_set_now(): direction.sip has been changed to "172.26.0.83" 6(21425) DEBUG: ctl [io_listener.c:495]: handle_stream_read(): handle_stream read: eof on /var/run/kamailio/kamailio_ctl Additional Information version: kamailio 5.4.1 (x86_64/linux) 09fd6a 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_BLACKLIST, HAVE_RESOLV_RES 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. id: 09fd6a compiled on 19:15:47 Jan 25 2021 with gcc 4.8.5 I found that kamailio 5.4.1 included the commit a752411 above mentioned,any one meet same issue ,or maybe somewhere configured wrong in kamailio.cfg,how to selove this issue,thanksssssss ! -- 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/issues/2610

1 2

[kamailio/kamailio] Kamailio not retrieving custom or default socket when sending timed REGISTER messages (#2262)
by kdanderso 26 Jan '21

26 Jan '21

### Description When using Kamailio in a multi-homed configuration to register with another device using the uac module, it does not retrieve the custom or default socket information for timed registration attempts. It does, however, use the custom/default socket data for the initial registration. #### Reproduction The network is configured for two addresses: 10.9.9.121/24 and 10.9.7.121/24. Two floating IP addresses are also assigned (via Pacemaker/Corosync): 10.9.9.120/24 and 10.9.7.120/24. For testing, all addresses can be assigned permanently, so it is not necessary to have Pacemaker/Corosync installed. MySQL database has a user in the uacreg table with the socket column set to udp:10.9.7.120:5060. kamailio.cfg includes the following to enable sending of REGISTER messages: mhomed = 1 listen=udp:10.9.9.120 listen=udp:10.9.7.120 #!define DBURL "mysql://kamailio:kamailiorw@10.9.9.101/kamailio" modparam("rr", "append_fromtag", 1); modparam("uac", "reg_db_url", DBURL); modparam("uac", "reg_contact_addr", "10.9.7.120:5060"); modparam("uac", "reg_keep_callid", 1); modparam("uac", "default_socket", "udp:10.9.7.120:5060"); modparam("uac", "reg_timer_interval", 30); #### Log Messages The initial REGISTER outputs the following log data. The registration completes successfully. Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: uac [uac.c:407]: child_init(): run initial uac registration routine Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: uac [uac_reg.c:1172]: uac_reg_update(): using custom socket udp:10.9.7.120:5060 to send request Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/socket_info.c:628]: grep_sock_info(): checking if host==us: 10==10 && [10.9.7.120] == [10.9.9.120] Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/socket_info.c:635]: grep_sock_info(): checking if port 5060 (advertise 0) matches port 5060 Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/socket_info.c:628]: grep_sock_info(): checking if host==us: 10==10 && [10.9.7.120] == [10.9.7.120] Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/socket_info.c:635]: grep_sock_info(): checking if port 5060 (advertise 0) matches port 5060 Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: tm [uac.c:435]: t_uac_prepare(): next_hop=<sip:10.9.7.102> Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: tm [uac.c:158]: dlg2hash(): hashid 3846 Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: tm [uac.c:244]: t_run_local_req(): executing event_route[tm:local-request] Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/parser/msg_parser.c:610]: parse_msg(): SIP Request: Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/parser/msg_parser.c:612]: parse_msg(): method: <REGISTER> Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/parser/msg_parser.c:614]: parse_msg(): uri: <sip:10.9.7.102> Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/parser/msg_parser.c:616]: parse_msg(): version: <SIP/2.0> Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/parser/parse_via.c:1303]: parse_via_param(): Found param type 232, <branch> = <z9hG4bK60f.6fd7c907000000000000000000000000.0>; state=16 Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/parser/parse_via.c:2639]: parse_via(): end of header reached, state=5 Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/parser/msg_parser.c:498]: parse_headers(): Via found, flags=2 Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <core> [core/parser/msg_parser.c:500]: parse_headers(): this is the first via Mar 24 19:09:27 kam_test kamailio[8425]: exec: *** cfgtrace:local_route=[tm:local-request] c=[/usr/local/etc/kamailio/kamailio.cfg] l=229 a=26 n=xlog Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: <script>: In event_route(tm:local-request) Mar 24 19:09:27 kam_test kamailio[8425]: DEBUG: tm [uac.c:664]: send_prepared_request_impl(): uac: 0x7268e184 branch: 0 to 10.9.7.102:5060 When the timer expires, the following is logged. Since the socket is not found, the registration fails: Mar 24 19:09:57 kam_test kamailio[8425]: DEBUG: tm [uac.c:435]: t_uac_prepare(): next_hop=<sip:10.9.7.102> Mar 24 19:09:57 kam_test kamailio[8425]: ERROR: <core> [core/forward.c:181]: get_out_socket(): no socket found Mar 24 19:09:57 kam_test kamailio[8425]: ERROR: <core> [core/forward.c:183]: get_out_socket(): no corresponding socket found for(udp:10.9.7.102:5060) Mar 24 19:09:57 kam_test kamailio[8425]: ERROR: tm [ut.h:316]: uri2dst2(): no corresponding socket found for "10.9.7.102" af 2 (udp:10.9.7.102:5060) Mar 24 19:09:57 kam_test kamailio[8425]: ERROR: tm [uac.c:463]: t_uac_prepare(): no socket found Mar 24 19:09:57 kam_test kamailio[8425]: DEBUG: tm [h_table.c:132]: free_cell_helper(): freeing transaction 0x7268de10 from uac.c:604 Mar 24 19:09:57 kam_test kamailio[8425]: ERROR: uac [uac_reg.c:1196]: uac_reg_update(): failed to send request for [kamailioha] ### Additional Information version: kamailio 5.3.2 (arm6/linux) 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_BLACKLIST, 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. id: unknown compiled on 17:22:05 Mar 18 2020 with gcc 8.3.0 * **Operating System**: Linux kam_test 4.19.75-v7+ #1270 SMP Tue Sep 24 18:45:11 BST 2019 armv7l GNU/Linux -- 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/issues/2262

4 6

[kamailio/kamailio] dispatcher: Log dest sets after variable update (#2609)
by Marat 25 Jan '21

25 Jan '21

#### 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 - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] 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 - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  I found that zero servers are being logged, even though they are (ignore the line numbers, the file is slightly modified): ``` 0(15681) DEBUG: dispatcher [dispatch.c:919]: ds_load_list(): found [0] dest sets 0(15681) DEBUG: dispatcher [dispatch.c:222]: ds_log_dst_cb(): dst>> 1031 sip:127.0.0.1:5182 0 0 (,0,0,0) 0(15681) DEBUG: dispatcher [dispatch.c:222]: ds_log_dst_cb(): dst>> 1011 sip:127.0.0.1:5180 0 0 (,0,0,0) 0(15681) DEBUG: dispatcher [dispatch.c:222]: ds_log_dst_cb(): dst>> 1021 sip:127.0.0.1:5181 0 0 (,0,0,0) ``` Therefore, I moved the logging of the dest sets after updating variable and this fixed the error. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2609 -- Commit Summary -- * dispatcher: Log dest sets after variable update -- File Changes -- M src/modules/dispatcher/dispatch.c (8) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2609.patch https://github.com/kamailio/kamailio/pull/2609.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/2609

2 2

git:master:f4b8bede: dispatcher: Log dest sets after variable update
by Marat Gareev 25 Jan '21

25 Jan '21

Module: kamailio Branch: master Commit: f4b8bede6a247415eda84a7db9569747905f9fa0 URL: https://github.com/kamailio/kamailio/commit/f4b8bede6a247415eda84a7db956974… Author: Marat Gareev <maratkin94(a)gmail.com> Committer: Marat Gareev <maratkin94(a)gmail.com> Date: 2021-01-25T18:40:47+03:00 dispatcher: Log dest sets after variable update --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/f4b8bede6a247415eda84a7db956974… Patch: https://github.com/kamailio/kamailio/commit/f4b8bede6a247415eda84a7db956974… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index c0a6d22fc0..62d416620e 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -895,14 +895,14 @@ int ds_load_list(char *lfile) goto error; } - LM_DBG("found [%d] dest sets\n", _ds_list_nr); - fclose(f); f = NULL; /* Update list - should it be sync'ed? */ _ds_list_nr = setn; *crt_idx = *next_idx; + LM_DBG("found [%d] dest sets\n", _ds_list_nr); + ds_log_sets(); return 0; @@ -1143,14 +1143,14 @@ int ds_load_db(void) goto err2; } - LM_DBG("found [%d] dest sets\n", _ds_list_nr); - ds_dbf.free_result(ds_db_handle, res); /* update data - should it be sync'ed? */ _ds_list_nr = setn; *crt_idx = *next_idx; + LM_DBG("found [%d] dest sets\n", _ds_list_nr); + ds_log_sets(); if(dest_errs > 0)

1 0

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

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2021