sr-dev April 2025

sr-dev@lists.kamailio.org

14 participants
201 discussions

git: new commits in branch master
by Daniel-Constantin Mierla 25 Apr '25

25 Apr '25

- URL: https://github.com/kamailio/kamailio/commit/d54641a86656b782e3c8901c1c3ba65… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:46:44+02:00 kamctl: version set to 6.0.0 - URL: https://github.com/kamailio/kamailio/commit/607d713acfaadbb3b60ec1ba25c6eec… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:47:21+02:00 kamdbclt: version set to 6.0.0 - spell fix - URL: https://github.com/kamailio/kamailio/commit/92917b5d3e69798ddb64c8cc40d3c4a… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:48:01+02:00 kamctl: spell fixes - URL: https://github.com/kamailio/kamailio/commit/c876f3f753cd48184bc1f11d6354a66… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:48:41+02:00 doc: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/7dbb45a1a8d4a98f439054ace33f041… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-04-25T10:50:04+02:00 core: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/3f1f75bf48a6c6675d86866751d616b… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:50:40+02:00 test/unit: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/cb55765834b24da779d381531d1bc70… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:51:53+02:00 pkg/kamailio/deb: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/eefee7f917795bd4e6ffd049fbdfc0e… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:52:45+02:00 lib/srdb1: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/bbd27a61781d751af22b0b4c4d9fd04… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:53:17+02:00 lib/trie: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/6548a1d86f8089758521d23ebdfc07a… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 app_java: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/cbeb777d3e9eeab55179756c0f526ce… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 app_jsdt: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/32bc708242fcecaddb6e4e50eca3a18… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 avpops: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/66c0dcf47b1f79a6a6f06b53a767e47… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 call_control: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/16900d46a0ae4d5fed1c8ed2af303b8… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 carrierroute: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/7a93d58fd03eff6b3615571f5887ad2… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 cplc: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/2e7b0100eb5b44890236a00ef290b60… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 db_berkeley: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/fe3d4ff577a4e52b4c58b3c33c6fd69… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 db_postgres: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/1929dc6451f726dc27f441e2acc382d… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 http_async_client: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/ced010db51e2e8b58145266b96a619f… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 imc: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/63cf35bebb6b9250fbdfd699f74d93c… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 ims_charging: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/ea2ad94474d7250a08885734c12b4d4… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 ims_qos: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/753068aadd954ff49c646659381201c… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 ims_registrar_pcscf: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/a22438751dbb169d33279e86825b6d2… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 ims_usrloc_pcscf: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/adbd17e5ca0d17c10779cc5cdd02988… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 ims_usrloc_scscf: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/44e37b9de0b6f4fb464509b50226271… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 janssonrpcc: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/668d455c7162f38f73dd559eb7754d6… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 keepalive: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/a22b6fd36aff95439321a881967afab… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 matrix: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/8064c53993cf934ed38f6dff9abfbac… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 memcached: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/354acb45e78995e988600c129d40af5… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 mohqueue: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/468ad7b72c1e468f99ac97282ccf6bf… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 nsq: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/b476fcefcccbddd15b2b18158ee7670… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 p_usrloc: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/251c9e9b530e2b192abceb0b739bcb9… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 presence: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/b669d4a0a33404c8ec9ae8429dc5e93… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 qos: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/b611f6d1403a6d9217d153d47880fd2… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:06+02:00 rabbitmq: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/4bac980f5f8efbe502b2a757e06f6c6… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 rr: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/4971c7c1e5d759fd4f2cb0e035c8b27… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 rtpengine: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/a13cf9ab687fcfdbaa6b099050bdd64… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 secfilter: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/df5d2224ac3643d49df126d244e80c1… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 sipcapture: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/059b9493ac044cdd142a2e6f37dea7d… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 siptrace: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/f449e5894f05f4c4dc105a2f5accced… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 sms: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/a1ed61f26fe4aadbccd90b0a78a85b7… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 snmpstats: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/7ac33cfc100cff251bbfc4fc7c01b64… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 sst: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/11bcbc534b9b6c51d1837855c2d9327… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 tm: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/a807b14ece749c0c688b8b9619567f5… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 userblocklist: spell fixes - mainly setup(n) <-> set(v) up distinction - URL: https://github.com/kamailio/kamailio/commit/3ff36b84d4d50cb5e5bcf62406e715e… Author: Paul Donald <newtwen+github(a)gmail.com> Date: 2025-04-25T10:56:07+02:00 xmpp: spell fixes - mainly setup(n) <-> set(v) up distinction

1 0

[kamailio/kamailio] core: added support for multiple tcp_closed callbacks (PR #4215)
by sergey-safarov 25 Apr '25

25 Apr '25

#### 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 enables the ability to define an array of callbacks for closed TCP connections. Use full when new module development. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4215 -- Commit Summary -- * core: added support for multiple tcp_closed callbacks -- File Changes -- M src/core/events.c (21) M src/core/events.h (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4215.patch https://github.com/kamailio/kamailio/pull/4215.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4215 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4215(a)github.com>

2 3

git:master:2873637a: core: added support for multiple tcp_closed callbacks
by Daniel-Constantin Mierla 25 Apr '25

25 Apr '25

Module: kamailio Branch: master Commit: 2873637af8a58ab5a694a86226d16284a7938460 URL: https://github.com/kamailio/kamailio/commit/2873637af8a58ab5a694a86226d1628… Author: Nikolay Ivanuschak <ivanuschak.nikolay(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-04-25T10:36:22+02:00 core: added support for multiple tcp_closed callbacks --- Modified: src/core/events.c Modified: src/core/events.h --- Diff: https://github.com/kamailio/kamailio/commit/2873637af8a58ab5a694a86226d1628… Patch: https://github.com/kamailio/kamailio/commit/2873637af8a58ab5a694a86226d1628… --- diff --git a/src/core/events.c b/src/core/events.c index 4274bce58e0..b985bccc6c2 100644 --- a/src/core/events.c +++ b/src/core/events.c @@ -209,9 +209,13 @@ int sr_event_register_cb(int type, sr_event_cb_f f) return -1; break; case SREV_TCP_CLOSED: - if(_sr_events_list.tcp_closed == 0) - _sr_events_list.tcp_closed = f; - else + for(i = 0; i < SREV_CB_LIST_SIZE; i++) { + if(_sr_events_list.tcp_closed[i] == 0) { + _sr_events_list.tcp_closed[i] = f; + break; + } + } + if(i == SREV_CB_LIST_SIZE) return -1; break; case SREV_NET_DATA_RECV: @@ -372,8 +376,13 @@ int sr_event_exec(int type, sr_event_param_t *evp) } else return 1; case SREV_TCP_CLOSED: - if(unlikely(_sr_events_list.tcp_closed != 0)) { - ret = _sr_events_list.tcp_closed(evp); + if(unlikely(_sr_events_list.tcp_closed[0] != 0)) { + ret = 0; + for(i = 0; + i < SREV_CB_LIST_SIZE && _sr_events_list.tcp_closed[i]; + i++) { + ret |= _sr_events_list.tcp_closed[i](evp); + } return ret; } else return 1; @@ -449,7 +458,7 @@ int sr_event_enabled(int type) case SREV_RCV_NOSIP: return (_sr_events_list.rcv_nosip != 0) ? 1 : 0; case SREV_TCP_CLOSED: - return (_sr_events_list.tcp_closed != 0) ? 1 : 0; + return (_sr_events_list.tcp_closed[0] != 0) ? 1 : 0; case SREV_NET_DATA_RECV: return (_sr_events_list.net_data_recv != 0) ? 1 : 0; case SREV_NET_DATA_SENT: diff --git a/src/core/events.h b/src/core/events.h index cf1ec91d554..297abba798e 100644 --- a/src/core/events.h +++ b/src/core/events.h @@ -70,7 +70,7 @@ typedef struct sr_event_cb sr_event_cb_f tcp_ws_frame_out; sr_event_cb_f stun_in; sr_event_cb_f rcv_nosip; - sr_event_cb_f tcp_closed; + sr_event_cb_f tcp_closed[SREV_CB_LIST_SIZE]; sr_event_cb_f net_data_recv; sr_event_cb_f net_data_sent; sr_event_cb_f sip_reply_out[SREV_CB_LIST_SIZE];

1 0

[kamailio/kamailio] topoh: fix bug in hiding record routes in reply messages (PR #4170)
by Farzaneh Soltanzadeh 25 Apr '25

25 Apr '25

#### Pre-Submission Checklist    - [ ] 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) - [ ] 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) - [ ] 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  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4170 -- Commit Summary -- * hide all record routes except last one in reply messages -- File Changes -- M src/modules/topoh/th_msg.c (16) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4170.patch https://github.com/kamailio/kamailio/pull/4170.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4170 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4170(a)github.com>

2 11

[kamailio/kamailio] Update PCSCF IMS example config (PR #4177)
by Tobias Funke 25 Apr '25

25 Apr '25

#### Pre-Submission Checklist    - [ ] 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) - [ ] 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 - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  The name of the `location` database for the `pcscf` was changed to `pcscf_location` here: https://github.com/kamailio/kamailio/commit/360bccbca1fe7515384065651034374…. So the example IMS pcscf config is not working right now. This simple PR provides the fix. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4177 -- Commit Summary -- * Update example pcscf ims config -- File Changes -- M misc/examples/ims/pcscf/kamailio.cfg (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4177.patch https://github.com/kamailio/kamailio/pull/4177.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4177 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4177(a)github.com>

3 3

git:master:dc269b33: misc/examples/ims/pcscf: location renamed to pcscf_location
by Daniel-Constantin Mierla 25 Apr '25

25 Apr '25

Module: kamailio Branch: master Commit: dc269b336c863940514e538bc71a0ef3eb5961ff URL: https://github.com/kamailio/kamailio/commit/dc269b336c863940514e538bc71a0ef… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-04-25T08:57:08+02:00 misc/examples/ims/pcscf: location renamed to pcscf_location - GH #4177 - table has different name to avoid conflict with usrloc - pre-commit whitespaces fixes --- Modified: misc/examples/ims/pcscf/kamailio.cfg --- Diff: https://github.com/kamailio/kamailio/commit/dc269b336c863940514e538bc71a0ef… Patch: https://github.com/kamailio/kamailio/commit/dc269b336c863940514e538bc71a0ef…

1 0

git:master:6d658fe6: sca: free in case of parse headers error
by Daniel-Constantin Mierla 25 Apr '25

25 Apr '25

Module: kamailio Branch: master Commit: 6d658fe6c1af84368d093942356cf2b116ff17e5 URL: https://github.com/kamailio/kamailio/commit/6d658fe6c1af84368d093942356cf2b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-04-25T08:51:23+02:00 sca: free in case of parse headers error --- Modified: src/modules/sca/sca_call_info.c --- Diff: https://github.com/kamailio/kamailio/commit/6d658fe6c1af84368d093942356cf2b… Patch: https://github.com/kamailio/kamailio/commit/6d658fe6c1af84368d093942356cf2b… --- diff --git a/src/modules/sca/sca_call_info.c b/src/modules/sca/sca_call_info.c index 3d5471178c0..0bf57b6f0a0 100644 --- a/src/modules/sca/sca_call_info.c +++ b/src/modules/sca/sca_call_info.c @@ -609,6 +609,7 @@ int sca_call_info_seize_held_call(sip_msg_t *msg, sca_call_info *call_info, // all headers must be parsed before using lump functions if(parse_headers(msg, HDR_EOH_F, 0) < 0) { LM_ERR("Failed to parse_headers\n"); + pkg_free(replaces_hdr.s); goto done; }

1 0

rpm builder docker image
by Sergey Safarov 24 Apr '25

24 Apr '25

Hello I have refactored docker builder image for RPM packages. Added multiarch support https://github.com/sergey-safarov/kamailio-builder Resulting docker images available https://github.com/sergey-safarov/kamailio-builder/pkgs/container/kamailio-… Before creating a pull request to https://github.com/kamailio/pkg-kamailio-docker I want to discuss this tool with the community. Maybe need to change the tags naming format. Or separate into different repos "manifest" and arch-specific images. Sergey

2 1

Kamailio Redis Integration
by Timothy Delin 24 Apr '25

24 Apr '25

Hey team, I have a question related to the Kamailio - Redis integration. I am currently using a VM with RHEL 9 OS, and I have created two separated docker files in my VM for the Kamailio and Redis. Both docker files are up and running as individual applications. However, when I tried to integrate the two containers, I keep facing issue where it doesn't want to connect between one and the other. Can you please help to advise on what I should do next? Thanks Best regards, Timothy

3 3

git:master:d965b924: jsonrpcs: reworked closing tcp socket
by Daniel-Constantin Mierla 23 Apr '25

23 Apr '25

Module: kamailio Branch: master Commit: d965b9242b313cb19f75baac12786d202a9987eb URL: https://github.com/kamailio/kamailio/commit/d965b9242b313cb19f75baac12786d2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-04-23T21:04:08+02:00 jsonrpcs: reworked closing tcp socket --- Modified: src/modules/jsonrpcs/jsonrpcs_sock.c --- Diff: https://github.com/kamailio/kamailio/commit/d965b9242b313cb19f75baac12786d2… Patch: https://github.com/kamailio/kamailio/commit/d965b9242b313cb19f75baac12786d2… --- diff --git a/src/modules/jsonrpcs/jsonrpcs_sock.c b/src/modules/jsonrpcs/jsonrpcs_sock.c index 366569ea3d9..ebd040e6509 100644 --- a/src/modules/jsonrpcs/jsonrpcs_sock.c +++ b/src/modules/jsonrpcs/jsonrpcs_sock.c @@ -830,6 +830,9 @@ int jsonrpc_tcp_process(void) LM_DBG("waiting for client connections\n"); while(1) { cfg_update(); + if(csock >= 0) { + close(csock); + } csock = accept( _jsonrpc_tcp_address.tsock, (struct sockaddr *)&caddr, &clen); @@ -845,12 +848,10 @@ int jsonrpc_tcp_process(void) n = read(csock, jsonrpc_tcp_buf, JSONRPC_DGRAM_BUF_SIZE - 1); if(n < 0) { LM_ERR("failed reading from tcp socket\n"); - close(csock); continue; } if(n == 0) { LM_DBG("no data received\n"); - close(csock); continue; } LM_DBG("data received - size: %d\n", n); @@ -892,7 +893,6 @@ int jsonrpc_tcp_process(void) if(n < 0) { LM_ERR("failed to send the response\n"); } - close(csock); continue; } @@ -900,7 +900,6 @@ int jsonrpc_tcp_process(void) if(n < 0) { LM_ERR("failed to send the response\n"); } - close(csock); } return 0;

1 0

← Newer
1
...
5
6
7
8
9
10
11
...
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