sr-dev April 2025

sr-dev@lists.kamailio.org

14 participants
201 discussions

git:master:c8217350: lost: clean allocated memory also for length 0
by Daniel-Constantin Mierla 28 Apr '25

28 Apr '25

Module: kamailio Branch: master Commit: c8217350f44a9ab897e3acdffbce1eb2a57fe1a2 URL: https://github.com/kamailio/kamailio/commit/c8217350f44a9ab897e3acdffbce1eb… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-04-28T11:17:12+02:00 lost: clean allocated memory also for length 0 --- Modified: src/modules/lost/response.c --- Diff: https://github.com/kamailio/kamailio/commit/c8217350f44a9ab897e3acdffbce1eb… Patch: https://github.com/kamailio/kamailio/commit/c8217350f44a9ab897e3acdffbce1eb… --- diff --git a/src/modules/lost/response.c b/src/modules/lost/response.c index 64cd55170aa..ac1728b62ab 100644 --- a/src/modules/lost/response.c +++ b/src/modules/lost/response.c @@ -731,10 +731,10 @@ p_lost_list_t lost_get_response_list( new->next = list; list = new; - lost_free_string(&tmp); /* clean up */ } else { lost_delete_response_list(&new); /* clean up */ } + lost_free_string(&tmp); /* clean up */ } } } else {

1 0

Implementation of ED-137B
by Sandeep Nayak 27 Apr '25

27 Apr '25

Hi Kamailio Team, We have started inhouse implementation of ED-137B for VVCS project. I need your guidance in identifying credible source for deployment of this protocol. Regards, Sandeep Nayak (VP - Engineering) +91 80952 41019 [https://ci3.googleusercontent.com/mail-sig/AIorK4woqONsX5aNx6yhTdY0_mcz3Zal…] Applied Electro Magnetics Pvt Ltd. B-106, Block B, Sector 67, Noida, Uttar Pradesh-201301 Website: https://www.aemindia.com/

1 0

[kamailio/kamailio] Handling Duplicate Values in secfilter Module's kamcmd Commands (Issue #4091)
by AsedMorteza 26 Apr '25

26 Apr '25

**Description** The `secfilter` module in `kamcmd` currently allows duplicate entries when adding values to certain lists (e.g., blacklists, whitelists). This can lead to unexpected behavior and potential inconsistencies. Could you clarify the reason behind accepting duplicate values? Recently, I submitted a pull request to add delete commands to the `secfilter` module #4089. When a user, for example, executes: `kamcmd secfilter.del_bl user 1005` I iterate through the entire list to ensure all occurrences of `1005` are removed, as I realized duplicates might exist. This is inefficient and could be avoided by preventing duplicates in the first place. **Proposed Solutions** 1. **Prevent Duplicates in Existing `add` Commands:** * I can modify existing `add` commands to directly prevent the addition of duplicate values. This would simplify future operations and improve performance. 2. **Introduce New `add_unique` Commands:** * I can maintain the current `add` commands for backward compatibility. * I can introduce new `add_unique` commands (e.g., `secfilter.add_bl_unique`) that explicitly reject duplicate entries. **Recommendation** I recommend implementing the first solution (preventing duplicates in existing `add` commands) as it provides the most straightforward and efficient approach. **Further Considerations** * Consider adding a check for duplicates during the `add` operation and returning an appropriate error message if a duplicate is encountered. * Document the behavior of duplicate values in the `secfilter` module's documentation. **I would appreciate your feedback and guidance on the best course of action.** -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4091 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4091(a)github.com>

6 9

[kamailio/kamailio] prefix_route - prefix longer than 12 characters ignored (Issue #4218)
by Barry Flanagan 25 Apr '25

25 Apr '25

flantel created an issue (kamailio/kamailio#4218) ### Description If I add a prefix of greater than 12 characters in to prefix_route it does not return the correct route but instead acts as if no record was found. ### Troubleshooting #### Reproduction Create a prefix_route prefix of more than 12 characters and the route will not be chosen. #### Debugging Data There is none. #### Log Messages prefix_route does not seem to produce DEBUG level messages. ### Additional Information ``` kamailio 5.8.6 (x86_64/linux) ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4218 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4218(a)github.com>

2 2

[kamailio/kamailio] Pepelux/secfilter Allow single quotes in From name and To name in secf_check_sqli_all(); (PR #4224)
by Jose Luis Verdeguer 25 Apr '25

25 Apr '25

1 1

git:master:e02eb4b2: add secf_remove_rule in header file
by Jose Luis Verdeguer 25 Apr '25

25 Apr '25

Module: kamailio Branch: master Commit: e02eb4b2ce854fc0f64a887d15588630d67a663b URL: https://github.com/kamailio/kamailio/commit/e02eb4b2ce854fc0f64a887d1558863… Author: AsedMorteza <mortezatorabi001(a)gmail.com> Committer: Jose Luis Verdeguer <pepeluxx(a)gmail.com> Date: 2025-04-25T17:05:48+02:00 add secf_remove_rule in header file --- Modified: src/modules/secfilter/secfilter.h --- Diff: https://github.com/kamailio/kamailio/commit/e02eb4b2ce854fc0f64a887d1558863… Patch: https://github.com/kamailio/kamailio/commit/e02eb4b2ce854fc0f64a887d1558863… --- diff --git a/src/modules/secfilter/secfilter.h b/src/modules/secfilter/secfilter.h index 92d96f5916e..8317fac5453 100644 --- a/src/modules/secfilter/secfilter.h +++ b/src/modules/secfilter/secfilter.h @@ -58,6 +58,7 @@ extern int *secf_stats; void secf_reset_stats(void); int secf_append_rule(int action, int type, str *value); +int secf_remove_rule(int action, int type, str *value); /* Get header values from message */ int secf_get_ua(struct sip_msg *msg, str *ua);

1 0

git:master:d2d90af5: Fixing problems
by Jose Luis Verdeguer 25 Apr '25

25 Apr '25

Module: kamailio Branch: master Commit: d2d90af5c7177bbfdcd10d39fa70d6ddd481d583 URL: https://github.com/kamailio/kamailio/commit/d2d90af5c7177bbfdcd10d39fa70d6d… Author: AsedMorteza <mortezatorabi001(a)gmail.com> Committer: Jose Luis Verdeguer <pepeluxx(a)gmail.com> Date: 2025-04-25T17:05:48+02:00 Fixing problems --- Modified: src/modules/secfilter/secfilter_db.c --- Diff: https://github.com/kamailio/kamailio/commit/d2d90af5c7177bbfdcd10d39fa70d6d… Patch: https://github.com/kamailio/kamailio/commit/d2d90af5c7177bbfdcd10d39fa70d6d… --- diff --git a/src/modules/secfilter/secfilter_db.c b/src/modules/secfilter/secfilter_db.c index e4dfc4ad77c..1c0138b0193 100644 --- a/src/modules/secfilter/secfilter_db.c +++ b/src/modules/secfilter/secfilter_db.c @@ -284,7 +284,7 @@ int secf_remove_rule(int action, int type, str *value) LM_DBG("Total matches removed: %d", total); return 0; // Return the total number of removed items } else { - LM_DGB("No matching values found in the list."); + LM_DBG("No matching values found in the list."); return -1; // Return -1 on no matches } }

1 0

git:master:350c6dc9: secfilterâ: support remove rule in Whitelist and Blacklist
by Jose Luis Verdeguer 25 Apr '25

25 Apr '25

Module: kamailio Branch: master Commit: 350c6dc9ac559987d627935f43743b62f95bd350 URL: https://github.com/kamailio/kamailio/commit/350c6dc9ac559987d627935f43743b6… Author: AsedMorteza <mortezatorabi001(a)gmail.com> Committer: Jose Luis Verdeguer <pepeluxx(a)gmail.com> Date: 2025-04-25T17:05:48+02:00 secfilterâ: support remove rule in Whitelist and Blacklist --- Modified: src/modules/secfilter/secfilter.c Modified: src/modules/secfilter/secfilter.h Modified: src/modules/secfilter/secfilter_db.c Modified: src/modules/secfilter/secfilter_rpc.c --- Diff: https://github.com/kamailio/kamailio/commit/350c6dc9ac559987d627935f43743b6… Patch: https://github.com/kamailio/kamailio/commit/350c6dc9ac559987d627935f43743b6…

1 0

[kamailio/kamailio] Kam fixes (PR #4219)
by Paul Donald 25 Apr '25

25 Apr '25

#### Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] 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 spelling/grammar two version number bumps for ctl tools. Should the commits be separated into different PRs? You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4219 -- Commit Summary -- * kamdbctl: version set to 6.0.0 * kamctl: version set to 6.0.0 * treewide: spelling/grammar fixes -- File Changes -- M doc/README.md (2) M doc/tutorials/ser_radius/ser_radius.xml (2) M pkg/kamailio/deb/bionic/kamailio.README.Debian (2) M pkg/kamailio/deb/bookworm/kamailio.README.Debian (2) M pkg/kamailio/deb/bullseye/kamailio.README.Debian (2) M pkg/kamailio/deb/buster/kamailio.README.Debian (2) M pkg/kamailio/deb/debian/kamailio.README.Debian (2) M pkg/kamailio/deb/focal/kamailio.README.Debian (2) M pkg/kamailio/deb/jammy/kamailio.README.Debian (2) M pkg/kamailio/deb/jessie/kamailio.README.Debian (2) M pkg/kamailio/deb/noble/kamailio.README.Debian (2) M pkg/kamailio/deb/precise/kamailio.README.Debian (2) M pkg/kamailio/deb/sid/kamailio.README.Debian (2) M pkg/kamailio/deb/stretch/kamailio.README.Debian (2) M pkg/kamailio/deb/trusty/kamailio.README.Debian (2) M pkg/kamailio/deb/wheezy/kamailio.README.Debian (2) M pkg/kamailio/deb/xenial/kamailio.README.Debian (2) M src/core/atomic/atomic_sparc.h (2) M src/core/atomic/atomic_sparc64.h (2) M src/core/atomic/atomic_x86.h (2) M src/core/cfg/cfg.h (2) M src/core/cfg/cfg_script.c (2) M src/core/cfg/cfg_script.h (2) M src/core/cfg/cfg_select.c (2) M src/core/cfg/cfg_struct.c (6) M src/core/cfg/cfg_struct.h (16) M src/core/parser/hf.c (2) M src/core/parser/hf.h (2) M src/core/sr_module.h (6) M src/core/udp_server.c (2) M src/lib/srdb1/db_query.h (20) M src/lib/trie/dtrie.c (2) M src/lib/trie/dtrie.h (2) M src/modules/app_java/README-draft (4) M src/modules/app_java/kamailio_java_folder/java-untested/siprouter_src/NativeInterface.java (4) M src/modules/app_java/kamailio_java_folder/java/siprouter_src/NativeInterface.java (4) M src/modules/app_jsdt/duktape.c (26) M src/modules/avpops/doc/avpops_admin.xml (2) M src/modules/call_control/doc/call_control_admin.xml (2) M src/modules/carrierroute/doc/carrierroute_admin.xml (2) M src/modules/cplc/doc/cplc_admin.xml (2) M src/modules/db_berkeley/doc/db_berkeley_admin.xml (4) M src/modules/db_berkeley/km_doc/db_berkeley_admin.xml (4) M src/modules/db_postgres/km_res.c (2) M src/modules/http_async_client/http_multi.c (2) M src/modules/imc/doc/imc_admin.xml (2) M src/modules/ims_charging/doc/ims_charging_admin.xml (2) M src/modules/ims_qos/doc/ims_qos_admin.xml (2) M src/modules/ims_registrar_pcscf/doc/ims_registrar_pcscf_admin.xml (2) M src/modules/ims_usrloc_pcscf/pcontact.c (2) M src/modules/ims_usrloc_scscf/ims_usrloc_scscf_mod.c (2) M src/modules/ims_usrloc_scscf/ucontact.c (2) M src/modules/janssonrpcc/janssonrpc_connect.c (2) M src/modules/janssonrpcc/janssonrpcc_mod.c (2) M src/modules/keepalive/doc/keepalive_devel.xml (2) M src/modules/matrix/doc/matrix_admin.xml (2) M src/modules/memcached/doc/memcached_admin.xml (2) M src/modules/mohqueue/mohq_db.c (6) M src/modules/nsq/nsq_mod.c (2) M src/modules/p_usrloc/doc/p_usrloc_admin.xml (4) M src/modules/p_usrloc/doc/p_usrloc_devel.xml (2) M src/modules/presence/presentity.c (4) M src/modules/qos/qos.c (2) M src/modules/qos/qos_handlers.c (4) M src/modules/rabbitmq/doc/rabbitmq_admin.xml (2) M src/modules/rr/doc/rr_admin.xml (2) M src/modules/rtpengine/doc/rtpengine_admin.xml (2) M src/modules/secfilter/doc/secfilter_admin.xml (2) M src/modules/sipcapture/doc/sipcapture_admin.xml (2) M src/modules/siptrace/doc/siptrace_admin.xml (2) M src/modules/sms/libsms_modem.c (2) M src/modules/snmpstats/snmpstats.c (4) M src/modules/snmpstats/sub_agent.c (2) M src/modules/sst/sst.c (2) M src/modules/sst/sst_handlers.c (22) M src/modules/tm/doc/tm.xml (4) M src/modules/userblocklist/doc/userblocklist_admin.xml (6) M src/modules/xmpp/xode.c (2) M test/unit/1.sh (2) M test/unit/10.sh (2) M test/unit/13.sh (2) M test/unit/17.sh (2) M test/unit/18.sh (2) M test/unit/23.sh (6) M test/unit/24.sh (2) M test/unit/25.sh (2) M test/unit/26.sh (4) M test/unit/3.sh (4) M test/unit/30.sh (2) M test/unit/31.sh (2) M test/unit/32.sh (2) M test/unit/37.sh (6) M test/unit/39.sh (4) M test/unit/4.sh (2) M test/unit/42.sh (6) M test/unit/43.sh (2) M test/unit/8.sh (4) M test/unit/9.sh (4) M utils/kamctl/CMakeLists.txt (2) M utils/kamctl/kamctl (2) M utils/kamctl/kamctlrc (2) M utils/kamctl/kamdbctl (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4219.patch https://github.com/kamailio/kamailio/pull/4219.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4219 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4219(a)github.com>

2 3

git:master:9a2c70a3: modules: readme files regenerated - avpops ... [skip ci]
by Kamailio Dev 25 Apr '25

25 Apr '25

Module: kamailio Branch: master Commit: 9a2c70a3d30005ff0f3af7e366641b87e2aec6cc URL: https://github.com/kamailio/kamailio/commit/9a2c70a3d30005ff0f3af7e366641b8… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-04-25T11:01:17+02:00 modules: readme files regenerated - avpops ... [skip ci] --- Modified: src/modules/avpops/README Modified: src/modules/call_control/README Modified: src/modules/carrierroute/README Modified: src/modules/cplc/README Modified: src/modules/db_berkeley/README Modified: src/modules/imc/README Modified: src/modules/ims_charging/README Modified: src/modules/ims_qos/README Modified: src/modules/ims_registrar_pcscf/README Modified: src/modules/keepalive/README Modified: src/modules/matrix/README Modified: src/modules/memcached/README Modified: src/modules/p_usrloc/README Modified: src/modules/rabbitmq/README Modified: src/modules/rr/README Modified: src/modules/rtpengine/README Modified: src/modules/secfilter/README Modified: src/modules/sipcapture/README Modified: src/modules/siptrace/README Modified: src/modules/tm/README Modified: src/modules/userblocklist/README --- Diff: https://github.com/kamailio/kamailio/commit/9a2c70a3d30005ff0f3af7e366641b8… Patch: https://github.com/kamailio/kamailio/commit/9a2c70a3d30005ff0f3af7e366641b8…

1 0

← Newer
1
...
4
5
6
7
8
9
10
...
21
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev April 2025