sr-dev June 2024

sr-dev@lists.kamailio.org

14 participants
224 discussions

git:master:244fc84a: dispatcher: docs for ds_oc_set_attrs(...)
by Daniel-Constantin Mierla 10 Jun '24

10 Jun '24

Module: kamailio Branch: master Commit: 244fc84a971d3381e731dab558c19d96e0922f5f URL: https://github.com/kamailio/kamailio/commit/244fc84a971d3381e731dab558c19d9… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-10T12:49:42+02:00 dispatcher: docs for ds_oc_set_attrs(...) --- Modified: src/modules/dispatcher/doc/dispatcher_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/244fc84a971d3381e731dab558c19d9… Patch: https://github.com/kamailio/kamailio/commit/244fc84a971d3381e731dab558c19d9… --- diff --git a/src/modules/dispatcher/doc/dispatcher_admin.xml b/src/modules/dispatcher/doc/dispatcher_admin.xml index e0e8d1beed8..62e212f8744 100644 --- a/src/modules/dispatcher/doc/dispatcher_admin.xml +++ b/src/modules/dispatcher/doc/dispatcher_admin.xml @@ -2014,12 +2014,13 @@ onreply_route { </programlisting> </example> </section> - <section id="dispatcher.f.ds_oc_set_rate"> + <section id="dispatcher.f.ds_oc_set_attrs"> <title> - <function moreinfo="none">ds_oc_set_rate(setid, uri, val)</function> + <function moreinfo="none">ds_oc_set_attrs(setid, uri, rval, tval)</function> </title> <para> - Sets the overload control rate for the destination (setid, uri). + Sets the overload control rate and time internval limit + for the destination (setid, uri). </para> <para>Description of parameters:</para> <itemizedlist> @@ -2032,9 +2033,14 @@ onreply_route { <para><emphasis>uri</emphasis> - the URI of the destination.</para> </listitem> <listitem> - <para><emphasis>val</emphasis> - the value of the overload control + <para><emphasis>rval</emphasis> - the value of the overload control rate (from 1 to 100) - how many times to skip using this destination in 100 selections.</para> + </listitem> + <listitem> + <para><emphasis>tval</emphasis> - time interval limit to apply the + overload control rate (in milliseconds). If the value is 0 or + negative, then the default value 500 is used.</para> </listitem> </itemizedlist> @@ -2042,10 +2048,10 @@ onreply_route { This function can be used from ANY_ROUTE. </para> <example> - <title><function>ds_oc_set_rate()</function> usage</title> + <title><function>ds_oc_set_attrs()</function> usage</title> <programlisting format="linespecific"> ... - ds_oc_set_rate("1", "sip:127.0.0.1:5080", "30"); + ds_oc_set_attrs("1", "sip:127.0.0.1:5080", "30", "1000"); ... </programlisting> </example>

1 0

git:master:0f8b5349: dispatcher: replaced ds_oc_set_rate() with ds_oc_set_attrs()
by Daniel-Constantin Mierla 10 Jun '24

10 Jun '24

Module: kamailio Branch: master Commit: 0f8b5349e2e98333d9e3b578144bb5402d8443e0 URL: https://github.com/kamailio/kamailio/commit/0f8b5349e2e98333d9e3b578144bb54… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-10T12:41:30+02:00 dispatcher: replaced ds_oc_set_rate() with ds_oc_set_attrs() - used now to set both oc rate and time limit --- Modified: src/modules/dispatcher/dispatch.c Modified: src/modules/dispatcher/dispatch.h Modified: src/modules/dispatcher/dispatcher.c --- Diff: https://github.com/kamailio/kamailio/commit/0f8b5349e2e98333d9e3b578144bb54… Patch: https://github.com/kamailio/kamailio/commit/0f8b5349e2e98333d9e3b578144bb54…

1 0

git:master:818cced9: core: helper fixup for functions with int-str-int-int parameters
by Daniel-Constantin Mierla 10 Jun '24

10 Jun '24

Module: kamailio Branch: master Commit: 818cced90cdbb524acbc423c79fc6c2b696da77c URL: https://github.com/kamailio/kamailio/commit/818cced90cdbb524acbc423c79fc6c2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-10T12:22:58+02:00 core: helper fixup for functions with int-str-int-int parameters --- Modified: src/core/mod_fix.c Modified: src/core/mod_fix.h --- Diff: https://github.com/kamailio/kamailio/commit/818cced90cdbb524acbc423c79fc6c2… Patch: https://github.com/kamailio/kamailio/commit/818cced90cdbb524acbc423c79fc6c2… --- diff --git a/src/core/mod_fix.c b/src/core/mod_fix.c index ec4e016ed88..fa7e2c4b47a 100644 --- a/src/core/mod_fix.c +++ b/src/core/mod_fix.c @@ -1061,3 +1061,37 @@ int fixup_free_isi(void **param, int param_no) return E_UNSPEC; } } + +/** + * + */ +int fixup_isii(void **param, int param_no) +{ + switch(param_no) { + case 1: + case 3: + case 4: + return fixup_igp_null(param, 1); + case 2: + return fixup_spve_null(param, 1); + default: + return E_UNSPEC; + } +} + +/** + * + */ +int fixup_free_isii(void **param, int param_no) +{ + switch(param_no) { + case 1: + case 3: + case 4: + return fixup_free_igp_null(param, 1); + case 2: + return fixup_free_spve_null(param, 1); + default: + return E_UNSPEC; + } +} diff --git a/src/core/mod_fix.h b/src/core/mod_fix.h index aafb3a4cbff..76c57bc9bf6 100644 --- a/src/core/mod_fix.h +++ b/src/core/mod_fix.h @@ -187,4 +187,7 @@ int fixup_free_ssii(void **param, int param_no); int fixup_isi(void **param, int param_no); int fixup_free_isi(void **param, int param_no); +int fixup_isii(void **param, int param_no); +int fixup_free_isii(void **param, int param_no); + #endif

1 0

git:5.8:955c6e02: modules: readme files regenerated - modules ... [skip ci]
by Kamailio Dev 08 Jun '24

08 Jun '24

Module: kamailio Branch: 5.8 Commit: 955c6e02ed0e31a4c7f148518bf44edb12c37a07 URL: https://github.com/kamailio/kamailio/commit/955c6e02ed0e31a4c7f148518bf44ed… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-06-08T20:31:20+02:00 modules: readme files regenerated - modules ... [skip ci] --- Modified: src/modules/ims_ipsec_pcscf/README --- Diff: https://github.com/kamailio/kamailio/commit/955c6e02ed0e31a4c7f148518bf44ed… Patch: https://github.com/kamailio/kamailio/commit/955c6e02ed0e31a4c7f148518bf44ed… --- diff --git a/src/modules/ims_ipsec_pcscf/README b/src/modules/ims_ipsec_pcscf/README index bce99b791da..3d831eb2c47 100644 --- a/src/modules/ims_ipsec_pcscf/README +++ b/src/modules/ims_ipsec_pcscf/README @@ -361,7 +361,7 @@ ipsec_destroy("location"); recv_host - received host to match the record. recv_port - received port to match the record. - The last three parameters have to be string valies and can contain + The last three parameters have to be string values and can contain variables. Example 1.14. ipsec_destroy_by_contact()

1 0

git: new commits in branch 5.8
by Daniel-Constantin Mierla 08 Jun '24

08 Jun '24

- URL: https://github.com/kamailio/kamailio/commit/f9bd2c5a4341913ac3e887e0324a19d… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:28:02+02:00 tcpops: reformat exported structures (cherry picked from commit 9b087fd618ab6c5cbd6a29c352b09c7198f1b594) - URL: https://github.com/kamailio/kamailio/commit/5c4f687d0ed06d6c7a5696917982b1c… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:28:19+02:00 sipcapture: use local static buffers for ip address and port - GH #3835 (cherry picked from commit d1f377ff23f491bfd5162cb7bf510759440dd8e5) - URL: https://github.com/kamailio/kamailio/commit/d53843c75da9115a9c3cb6b1b909fd3… Author: Sergey Safarov <s.safarov(a)gmail.com> Date: 2024-06-08T12:28:32+02:00 db_mysql: fixed build on CentOS 7 (cherry picked from commit 767b3faee3af450d6482b919c81431ea6d707807) - URL: https://github.com/kamailio/kamailio/commit/deb11ee814e1591698d120a113630aa… Author: Rick Barenthin <rick(a)ng-voice.com> Date: 2024-06-08T12:29:15+02:00 usrloc: delete location_attrs in db_only mode on delete_urecord When calling delete_urecord in DB_ONLY mode the location_attrs are not deleted. This fix makes sure to delete location_attrs when calling db_delete_urecord is called. (cherry picked from commit 6b8f6d3b5d563d91234b8d4b7e155b7e74b9d313) - URL: https://github.com/kamailio/kamailio/commit/9a7e3e54f1c52ee69c68b1165de1158… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:30:01+02:00 htable: error on not finding htable in pv set (cherry picked from commit 82cfea317fd8d02a6c94347bc1be18c8a06e580f) - URL: https://github.com/kamailio/kamailio/commit/7087ce6aa496de9b6015e5054d8fd70… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:30:14+02:00 htable: error on not finding htable in pv get (cherry picked from commit 24b27214231166366cceb88fcb3724298cc53b32) - URL: https://github.com/kamailio/kamailio/commit/f4eea76a15b220b15337b60a77f7978… Author: Bastian Triller <bastian.triller(a)gmail.com> Date: 2024-06-08T12:30:26+02:00 rtpengine: Fix extra process in pkg.stats When DTMF events socket is not configured, pkg.stats RPC method returns an empty entry. { entry: 13 pid: 0 rank: -128 used: 0 free: 0 real_used: 0 total_size: 67108864 total_frags: 0 desc: } Fixes: 26f6e57c8f ("Register a new worker process for dtmf event listener") (cherry picked from commit e67f6048f4883d446df2d88fe7a7d7f2ff6daf8c) - URL: https://github.com/kamailio/kamailio/commit/833b93563f43fa95e4bfdd175b14467… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:30:35+02:00 siputils: free head of params list for get_uri_param() - GH #3857 (cherry picked from commit d72a3fbc903da02ae3b26f3aae2ea228c9f8b255) - URL: https://github.com/kamailio/kamailio/commit/ebc8acef7617a69dcc06b163c47e88a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:30:46+02:00 siputils: free head of params list for get_uri_param() - GH #3857 (cherry picked from commit 8363208fff7c101a2779ff64783e5b7dbf8ffd7c) - URL: https://github.com/kamailio/kamailio/commit/5c9c13080be2b15252492dca388777d… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-06-08T12:31:20+02:00 regex: fix pcre2 migration instead of reserving an array in pkg_mem and later copy the pointers to shm_mem just use shm_mem in the first place. fixes #3812 (cherry picked from commit d89333748662c0e7c72e3579d41fd14ef05c0fee) - URL: https://github.com/kamailio/kamailio/commit/614b5601170578c2d0eb79e7e5c64e6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:31:31+02:00 geoip2: init MMDB_s global pointer and check on destroy (cherry picked from commit 8ba4ee4e00798ca015adc48542d0e4de8ed9daad) - URL: https://github.com/kamailio/kamailio/commit/2cc901e8be63791432a49f87d434eeb… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:31:39+02:00 geoip2: do not close global MMDB_s on destroy - GH #3861 (cherry picked from commit e066613cac308d1129baebc47d7c245b36eba4b0) - URL: https://github.com/kamailio/kamailio/commit/ae3be6af4ea8d50c29fe8e523c95482… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:31:50+02:00 geoip2: rename global lock variable (cherry picked from commit 4bd619d3517f129362de3ce50eed2620d5002b42) - URL: https://github.com/kamailio/kamailio/commit/45f2e2670a3eac768b97b58eeec228b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:32:00+02:00 geoip2: keep global structure per process - parsing db file allocates private memory - reworked reload to rely on a global counter - GH #3861 (cherry picked from commit baccbe0298f9fa46db285505bc4386f997de5a47) - URL: https://github.com/kamailio/kamailio/commit/c27094b8288b7a949fbfc766eb9223c… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:32:08+02:00 geoip2: clang format module file (cherry picked from commit 293193c55caaf5d0c39dc456babe92f574b58b3b) - URL: https://github.com/kamailio/kamailio/commit/54f9adb03ebbacf35eb0785dc46b55c… Author: Bastian Triller <bastian.triller(a)gmail.com> Date: 2024-06-08T12:32:16+02:00 geoip2: Reload database before accessing it Re-add reloading of database. GH #3861 Fixes: 293193c55c ("geoip2: clang format module file") (cherry picked from commit 51dcab019a42e5b1f3b81247ca71ffcaf46180de) - URL: https://github.com/kamailio/kamailio/commit/eb4a8e326b10d0c3e922cc524d8f46d… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:32:55+02:00 iims_registrar_scscf: include header for common c functions - GH #3855 (cherry picked from commit b508ef253cf996a5efd8bf13ded0c804f3557ada) - URL: https://github.com/kamailio/kamailio/commit/5999529be9ac2187dad465518bea3de… Author: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2024-06-08T12:33:18+02:00 core/resolve: Check dns_cache_init and choose appropriate functions (cherry picked from commit 06d583e356351ae9d8a559c9f5de3e57fb128a38) - URL: https://github.com/kamailio/kamailio/commit/eb4ec6f9305e334d9749e1fd56ad195… Author: codenot <codenot(a)msn.com> Date: 2024-06-08T12:33:34+02:00 nathelper: fixed handle_ruri_alias_mode(1) cannot proper handle multi alias (cherry picked from commit 8e0b2e4b6450fa50bd573fa013dc888aaaccd2bf) - URL: https://github.com/kamailio/kamailio/commit/e8dc8506dbc4d8964e02c3be175d7f1… Author: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2024-06-08T12:33:43+02:00 nathelper: remove magic constant (cherry picked from commit 3d187336f22e30800acefd28e4d0f98e0f677a49) - URL: https://github.com/kamailio/kamailio/commit/5571ea0ff8877a8a19b3fb76030af19… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-06-08T12:34:06+02:00 dmq_usrloc: reformat exported structures (cherry picked from commit e94812149c6f2867a5b23f1218a9af139590bf51) - URL: https://github.com/kamailio/kamailio/commit/ed0cdf3a3c85a8d05aacec8917fe35a… Author: Bastian Triller <bastian.triller(a)gmail.com> Date: 2024-06-08T12:35:18+02:00 ims_ipsec_pcscf: Fix typo (cherry picked from commit 16e1b38fcc1593ad09eb03d86cd056ab8ff7f7f7) - URL: https://github.com/kamailio/kamailio/commit/8ef8db05dbc3a4cf5246354ce31dff1… Author: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2024-06-08T12:35:24+02:00 pua_dialoginfo: Fix ruri length (cherry picked from commit 2c7c2d215ec9277fa706053579a38f058f15ae85)

1 0

[kamailio/kamailio] pua_dialoginfo fails to publish the caller when not using avps (Issue #3838)
by michelpelletier07 08 Jun '24

08 Jun '24

### Description What I did: I enabled the pua_dialoginfo module to publish changes for both the caller and the callee. What I expected: PUBLISH to be generated for the caller and the callee What actually happened: PUBLISH is only generated for the callee. ### Troubleshooting I troubleshooted the issue by adding LM_ERR messages at various point in the pua_dialoginfo.c #### Reproduction Add a db-url as modparam to pua Do not add any modparam to pua_dialoginfo. Then make a phone call while making sure dialog.dlg_manage() is called to track the call. #### Debugging Data Not applicable #### Log Messages pua_dialoginfo [dialog_publish.c:303]: dialog_publish(): failed to parse the PUBLISH R-URI ### Possible Solutions I tracked the issue to line 840 of pua_dialoginfo.c (see enclosed patch file), where dlginfo->pubruris_caller->s.s is set to the value of dlginfo->from_uri but dlginfo->pubruris_caller->s.len is not set to anything (left at zero). So I added a line to set the len value to the len value of dlginfo->from_uri.len See patch below: ``` diff -ru a/src/modules/pua_dialoginfo/pua_dialoginfo.c b/src/modules/pua_dialoginfo/pua_dialoginfo.c --- a/src/modules/pua_dialoginfo/pua_dialoginfo.c 2024-05-02 11:14:34.150097771 -0600 +++ b/src/modules/pua_dialoginfo/pua_dialoginfo.c 2024-05-02 11:09:30.855960974 -0600 @@ -838,6 +838,8 @@ } memset(dlginfo->pubruris_caller, 0, sizeof(struct str_list)); dlginfo->pubruris_caller->s.s = shm_str2char_dup(&dlginfo->from_uri); + dlginfo->pubruris_caller->s.len = dlginfo->from_uri.len; + if(!dlginfo->pubruris_caller->s.s) { free_dlginfo_cell(dlginfo); return NULL; ``` ### Additional Information version: kamailio 5.8.1 (x86_64/linux) 384843 flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, MEM_JOIN_FREE, 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_SEND_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: 384843 compiled on 13:45:08 May 2 2024 with gcc 10.2.1 * **Operating System**: Linux yto11-stg-smf-vtr1.vcn.ms 5.10.0-27-amd64 #1 SMP Debian 5.10.205-2 (2023-12-31) x86_64 GNU/Linux -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3838 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3838(a)github.com>

3 4

[kamailio/kamailio] pua_dialoginfo: Fix ruri length (PR #3871)
by Xenofon Karamanos 08 Jun '24

08 Jun '24

#### 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:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [x] Related to issue #3838 #### Description  This PR fixes a bug where R-URI couldn't be parsed correctly by assing the correct length to the str variable You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3871 -- Commit Summary -- * pua_dialoginfo: Fix ruri length -- File Changes -- M src/modules/pua_dialoginfo/pua_dialoginfo.c (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3871.patch https://github.com/kamailio/kamailio/pull/3871.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3871 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3871(a)github.com>

2 2

git:master:2c7c2d21: pua_dialoginfo: Fix ruri length
by Henning Westerholt 08 Jun '24

08 Jun '24

Module: kamailio Branch: master Commit: 2c7c2d215ec9277fa706053579a38f058f15ae85 URL: https://github.com/kamailio/kamailio/commit/2c7c2d215ec9277fa706053579a38f0… Author: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-06-08T08:52:02+02:00 pua_dialoginfo: Fix ruri length --- Modified: src/modules/pua_dialoginfo/pua_dialoginfo.c --- Diff: https://github.com/kamailio/kamailio/commit/2c7c2d215ec9277fa706053579a38f0… Patch: https://github.com/kamailio/kamailio/commit/2c7c2d215ec9277fa706053579a38f0… --- diff --git a/src/modules/pua_dialoginfo/pua_dialoginfo.c b/src/modules/pua_dialoginfo/pua_dialoginfo.c index 22a2fb11da7..c8a8a6f31a3 100644 --- a/src/modules/pua_dialoginfo/pua_dialoginfo.c +++ b/src/modules/pua_dialoginfo/pua_dialoginfo.c @@ -838,6 +838,7 @@ struct dlginfo_cell *get_dialog_data(struct dlg_cell *dlg, int type, } memset(dlginfo->pubruris_caller, 0, sizeof(struct str_list)); dlginfo->pubruris_caller->s.s = shm_str2char_dup(&dlginfo->from_uri); + dlginfo->pubruris_caller->s.len = dlginfo->from_uri.len; if(!dlginfo->pubruris_caller->s.s) { free_dlginfo_cell(dlginfo); return NULL;

1 0

[kamailio/kamailio] Less predictive algorithm for carrierroute hashing (Issue #3786)
by Ben Kaufman 08 Jun '24

08 Jun '24

### Description Carrierroute's CRC32 algorithm over call_id tends to produce poor call distribution under real world examples. Having 4 records for a given carrier/domain combo (assuming no flags, mask, or scan_prefix) and a probability of `0.25` using a hash source of `call_id` I observed over 800 hundred consecutive calls from a carrier (bandwidth.com) where `CRC32(call_id) % 1000` was less than 250. To be fair, the module does what it says - it uses a CRC32 hash over the call id, however the data from the UA generating the call is yielding poor results. Would it be possible to change to an alternate hashing mechanism (or add another hashing mechanism, selectable via `modparam`) such as MD5? ### Expected behavior Even distribution across all hosts provided. #### Actual observed behavior Uneven distribution under real-world conditions. #### Debugging Data I can provide a list of > 800 call-ids observed that all routed to the same endpoint. These are pulled from real world calls, and while there's not much danger in just having call-id's I would still prefer not to post it into github for posterity. #### Log Messages Log messages end up looking similar to this. I validated that the number is the CRC32 hash of the call id, so it's not a bug in the module. ``` proxy-1 | 1(7) DEBUG: carrierroute [cr_func.c:449]: rewrite_on_rule(): searching for matching routing rules 1(7) DEBUG: carrierroute [prime_hash.c:66]: hash_func(): hash: 2417021133 % 1000 = 133 proxy-1 | 1(7) INFO: carrierroute [cr_func.c:689]: ki_cr_do_route_helper(): uri service was rewritten to sip:service@host-d, carrier 1, domain 1 ``` ### Possible Solutions Ability to use MD5, kamailio's core hash function or other such hashing function ### Additional Information Problem validate in 5.7.3, but since it's *correct* by module design, it shouldn't have changed in 5.8 -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3786 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3786(a)github.com>

3 4

[kamailio/kamailio] microhttpd cannot be build on CentOS 8 (Issue #3504)
by sergey-safarov 08 Jun '24

08 Jun '24

``` LD (gcc) [M memcached.so] memcached.so CC (gcc) [M microhttpd.so] microhttpd_mod.o microhttpd_mod.c:374:24: error: return type is an incomplete type static enum MHD_Result ksr_microhttpd_request(void *cls, ^~~~~~~~~~~~~~~~~~~~~~ microhttpd_mod.c: In function 'ksr_microhttpd_request': microhttpd_mod.c:390:10: warning: 'return' with a value, in function returning void return MHD_YES; ^~~~~~~ microhttpd_mod.c:374:24: note: declared here static enum MHD_Result ksr_microhttpd_request(void *cls, ^~~~~~~~~~~~~~~~~~~~~~ microhttpd_mod.c:411:10: warning: 'return' with a value, in function returning void return MHD_NO; ^~~~~~ microhttpd_mod.c:374:24: note: declared here static enum MHD_Result ksr_microhttpd_request(void *cls, ^~~~~~~~~~~~~~~~~~~~~~ microhttpd_mod.c:433:10: warning: 'return' with a value, in function returning void return MHD_NO; ^~~~~~ microhttpd_mod.c:374:24: note: declared here static enum MHD_Result ksr_microhttpd_request(void *cls, ^~~~~~~~~~~~~~~~~~~~~~ microhttpd_mod.c:436:9: warning: 'return' with a value, in function returning void return MHD_YES; ^~~~~~~ microhttpd_mod.c:374:24: note: declared here static enum MHD_Result ksr_microhttpd_request(void *cls, ^~~~~~~~~~~~~~~~~~~~~~ microhttpd_mod.c: In function 'microhttpd_server_run': microhttpd_mod.c:462:17: warning: passing argument 5 of 'MHD_start_daemon' from incompatible pointer type [-Wincompatible-p ointer-types] NULL, NULL, &ksr_microhttpd_request, KSR_MICROHTTPD_PAGE, ^~~~~~~~~~~~~~~~~~~~~~~ In file included from microhttpd_mod.c:29: /usr/include/microhttpd.h:2148:1: note: expected 'MHD_AccessHandlerCallback' {aka 'int (*)(void *, struct MHD_Connection *, const char *, const char *, const char *, const char *, long unsigned int *, void **)'} but argument is of type 'void (*)( void *, struct MHD_Connection *, const char *, const char *, const char *, const char *, size_t *, void **)' {aka 'void (*) (void *, struct MHD_Connection *, const char *, const char *, const char *, const char *, long unsigned int *, void **)'} MHD_start_daemon (unsigned int flags, ^~~~~~~~~~~~~~~~ microhttpd_mod.c:467:17: warning: passing argument 5 of 'MHD_start_daemon' from incompatible pointer type [-Wincompatible-p ointer-types] NULL, NULL, &ksr_microhttpd_request, KSR_MICROHTTPD_PAGE, ^~~~~~~~~~~~~~~~~~~~~~~ In file included from microhttpd_mod.c:29: /usr/include/microhttpd.h:2148:1: note: expected 'MHD_AccessHandlerCallback' {aka 'int (*)(void *, struct MHD_Connection *, const char *, const char *, const char *, const char *, long unsigned int *, void **)'} but argument is of type 'void (*)( void *, struct MHD_Connection *, const char *, const char *, const char *, const char *, size_t *, void **)' {aka 'void (*) (void *, struct MHD_Connection *, const char *, const char *, const char *, const char *, long unsigned int *, void **)'} MHD_start_daemon (unsigned int flags, ^~~~~~~~~~~~~~~~ make[2]: *** [../../Makefile.rules:100: microhttpd_mod.o] Error 1 make[1]: Leaving directory '/root/rpmbuild/BUILD/kamailio-5.8.0-dev1/src' make[1]: *** [Makefile:508: modules] Error 1 make: *** [Makefile:34: every-module] Error 2 error: Bad exit status from /var/tmp/rpm-tmp.K7wkd7 (%build) Bad exit status from /var/tmp/rpm-tmp.K7wkd7 (%build) ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3504 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3504(a)github.com>

3 3

← Newer
1
...
14
15
16
17
18
19
20
...
23
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev June 2024