sr-dev December 2024

sr-dev@lists.kamailio.org

17 participants
343 discussions

[kamailio/kamailio] ims_diameter_server: fix retrieval ``$diameter_response`` value (PR #4069)

by Victor Seva

#### 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 - [ ] Tested changes locally - [x] Related to issue https://lists.kamailio.org/mailman3/hyperkitty/list/sr-users@lists.kamailio… #### Description ``current_msg_id_repl is only`` changed if ``$diameter_reponse`` is changed by via ``pv_set_response()``. The event_route is executed with a ``fake_msg`` so it will not match if is setted outside anyways. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4069 -- Commit Summary -- * ims_diameter_server: fix retrieval ``$diameter_response`` value -- File Changes -- M src/modules/ims_diameter_server/avp_helper.c (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4069.patch https://github.com/kamailio/kamailio/pull/4069.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4069 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4069(a)github.com>

1 month, 3 weeks

git:master:7c365f76: ims_diameter_server: fix retrieval ``$diameter_response`` value

by Victor Seva

Module: kamailio Branch: master Commit: 7c365f76560b18aca49096121ea55791ce8f3623 URL: https://github.com/kamailio/kamailio/commit/7c365f76560b18aca49096121ea5579… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-12-14T13:44:13+01:00 ims_diameter_server: fix retrieval ``$diameter_response`` value ``current_msg_id_repl is only`` changed if ``$diameter_reponse`` is changed by via ``pv_set_response()``. The event_route is executed with a ``fake_msg`` so it will not match if is setted outside anyways. --- Modified: src/modules/ims_diameter_server/avp_helper.c --- Diff: https://github.com/kamailio/kamailio/commit/7c365f76560b18aca49096121ea5579… Patch: https://github.com/kamailio/kamailio/commit/7c365f76560b18aca49096121ea5579… --- diff --git a/src/modules/ims_diameter_server/avp_helper.c b/src/modules/ims_diameter_server/avp_helper.c index 065a58c364d..7fa2ac66ce2 100644 --- a/src/modules/ims_diameter_server/avp_helper.c +++ b/src/modules/ims_diameter_server/avp_helper.c @@ -48,7 +48,6 @@ // ID of current message static unsigned int current_msg_id = 0; -static unsigned int current_msg_id_repl = 0; cJSON *avp2json(AAA_AVP *avp_t) { @@ -488,7 +487,7 @@ int pv_get_application(struct sip_msg *msg, pv_param_t *param, pv_value_t *res) int pv_get_response(struct sip_msg *msg, pv_param_t *param, pv_value_t *res) { - if((msg->id != current_msg_id_repl) || (responsejson.len < 0)) { + if(responsejson.len < 0) { return pv_get_null(msg, param, res); } return pv_get_strval(msg, param, res, &responsejson); @@ -504,7 +503,6 @@ int pv_set_response( val->rs.s); responsejson.s = val->rs.s; responsejson.len = val->rs.len; - current_msg_id_repl = msg->id; return 0; } return 0;

1 month, 3 weeks

git:master:bb221de0: modules: readme files regenerated - topos ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: bb221de0bb49f978d238a8aa97004bda2a7fa456 URL: https://github.com/kamailio/kamailio/commit/bb221de0bb49f978d238a8aa97004bd… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-12-14T12:01:10+01:00 modules: readme files regenerated - topos ... [skip ci] --- Modified: src/modules/topos/README --- Diff: https://github.com/kamailio/kamailio/commit/bb221de0bb49f978d238a8aa97004bd… Patch: https://github.com/kamailio/kamailio/commit/bb221de0bb49f978d238a8aa97004bd… --- diff --git a/src/modules/topos/README b/src/modules/topos/README index 70af19ae47e..9c82ff026e1 100644 --- a/src/modules/topos/README +++ b/src/modules/topos/README @@ -212,6 +212,7 @@ Chapter 1. Admin Guide Type of storage, valid types are: * db - Database Backend * redis - Redis Backend + * htable - htable module Backend Default value is ��db��.

1 month, 3 weeks

[kamailio/kamailio] ims_isc: bugfix: firstflag incorrect in isc_match_filter (PR #4018)

by Kilian Wöber

#### 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 - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  The function isc_match_filter uses a local variable "firstflag", to handle the case of an HSS based terminating unregistered service. In that case, the S-CSCF exchanges SAR/SAA with the HSS in order to learn the subscription data and the iFC for an unregistered user or for a PSI. The firstflag is necessary to make a distiction between the FAILURE ROUTE that is used for this case and the FAILURE ROUTE that is used for transaction failure towards the AS. The error is, the firstflag is set for the latter case, too, which leads to a wrong handling of communication errors towards the AS. This PR introduces a new function that looks for the old ISCMARK in the lumps of the stored message and hence can make a distinction between the SAR/SAA case and the AS failure case. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4018 -- Commit Summary -- * ims_isc: bugfix: firstflag incorrect in isc_match_filter -- File Changes -- M src/modules/ims_isc/ims_isc_mod.c (24) M src/modules/ims_isc/mark.c (51) M src/modules/ims_isc/mark.h (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4018.patch https://github.com/kamailio/kamailio/pull/4018.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4018 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4018(a)github.com>

1 month, 3 weeks

git:master:2bd70ed7: ims_isc: bugfix: firstflag incorrect in isc_match_filter

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 2bd70ed7ab8ec4cf59e12be3d561c735d84f2dea URL: https://github.com/kamailio/kamailio/commit/2bd70ed7ab8ec4cf59e12be3d561c73… Author: Kilian W��ber <Kilian.Woeber(a)kontron.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-12-14T11:53:49+01:00 ims_isc: bugfix: firstflag incorrect in isc_match_filter --- Modified: src/modules/ims_isc/ims_isc_mod.c Modified: src/modules/ims_isc/ims_isc_mod.h Modified: src/modules/ims_isc/isc.c Modified: src/modules/ims_isc/isc.h Modified: src/modules/ims_isc/mark.c Modified: src/modules/ims_isc/mark.h --- Diff: https://github.com/kamailio/kamailio/commit/2bd70ed7ab8ec4cf59e12be3d561c73… Patch: https://github.com/kamailio/kamailio/commit/2bd70ed7ab8ec4cf59e12be3d561c73…

1 month, 3 weeks

[kamailio/kamailio] htable: export table create api functions (PR #4010)

by Stefan Mititelu

#### 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  Looks like htable module already exports API functions, but currently no other module uses them. Added 2 more API functions for table creation (so other modules can be able to create their own hash tables if they want to) You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4010 -- Commit Summary -- * htable: export table create api functions -- File Changes -- M src/modules/htable/api.c (10) M src/modules/htable/api.h (4) M src/modules/htable/ht_api.c (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4010.patch https://github.com/kamailio/kamailio/pull/4010.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4010 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4010(a)github.com>

1 month, 3 weeks

git:master:90d1485c: topos_htable: Small update to doc

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 90d1485c62bdfe509ff14b1b05baf0752b1a946e URL: https://github.com/kamailio/kamailio/commit/90d1485c62bdfe509ff14b1b05baf07… Author: Stefan Mititelu <stefan.mititelu(a)net2phone.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-12-14T11:52:02+01:00 topos_htable: Small update to doc --- Modified: src/modules/topos_htable/doc/topos_htable.xml --- Diff: https://github.com/kamailio/kamailio/commit/90d1485c62bdfe509ff14b1b05baf07… Patch: https://github.com/kamailio/kamailio/commit/90d1485c62bdfe509ff14b1b05baf07… --- diff --git a/src/modules/topos_htable/doc/topos_htable.xml b/src/modules/topos_htable/doc/topos_htable.xml index 66344832065..d42066155a9 100644 --- a/src/modules/topos_htable/doc/topos_htable.xml +++ b/src/modules/topos_htable/doc/topos_htable.xml @@ -10,36 +10,22 @@ <book xmlns:xi="http://www.w3.org/2001/XInclude"> <bookinfo> - <title>TOPOS_REDIS Module</title> + <title>TOPOS_HTABLE Module</title> <productname class="trade">kamailio.org</productname> <authorgroup> - <author> - <firstname>Daniel-Constantin</firstname> - <surname>Mierla</surname> - <email>miconda(a)gmail.com</email> - </author> <editor> - <firstname>Daniel-Constantin</firstname> - <surname>Mierla</surname> - <email>miconda(a)gmail.com</email> - </editor> - <editor> - <firstname>Frederic</firstname> - <surname>Gaisnon</surname> - <email>frederic.gaisnon(a)gmail.com</email> + <firstname>Stefan</firstname> + <surname>Mititelu</surname> + <email>stefan.mititelu(a)net2phone.com</email> </editor> </authorgroup> <copyright> - <year>2017</year> + <year>2024</year> <holder>kamailio.org</holder> </copyright> <copyright> - <year>2017</year> - <holder>flowroute.com</holder> - </copyright> - <copyright> - <year>2021</year> - <holder>MomentTech</holder> + <year>2024</year> + <holder>net2phone.com</holder> </copyright> </bookinfo> <toc></toc>

1 month, 3 weeks

git:master:df48b373: topos_htable: Initial commit

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: df48b373c39d631f49cb3632f7547f65b58eda3e URL: https://github.com/kamailio/kamailio/commit/df48b373c39d631f49cb3632f7547f6… Author: Stefan Mititelu <stefan.mititelu(a)net2phone.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-12-14T11:52:02+01:00 topos_htable: Initial commit --- Added: src/modules/topos_htable/Makefile Added: src/modules/topos_htable/doc/Makefile Added: src/modules/topos_htable/doc/topos_htable.xml Added: src/modules/topos_htable/doc/topos_htable_admin.xml Added: src/modules/topos_htable/topos_htable_mod.c Added: src/modules/topos_htable/topos_htable_storage.c Added: src/modules/topos_htable/topos_htable_storage.h Modified: src/Makefile.groups --- Diff: https://github.com/kamailio/kamailio/commit/df48b373c39d631f49cb3632f7547f6… Patch: https://github.com/kamailio/kamailio/commit/df48b373c39d631f49cb3632f7547f6…

1 month, 3 weeks

git:master:bd70a78c: topos: Updates for topos_htable module

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: bd70a78ce22ab419a52e8733bc7ee27765d14cac URL: https://github.com/kamailio/kamailio/commit/bd70a78ce22ab419a52e8733bc7ee27… Author: Stefan Mititelu <stefan.mititelu(a)net2phone.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-12-14T11:52:02+01:00 topos: Updates for topos_htable module Add htable storage modparam. End storage dialog on negative replies. --- Modified: src/modules/topos/doc/topos_admin.xml Modified: src/modules/topos/topos_mod.c Modified: src/modules/topos/tps_msg.c --- Diff: https://github.com/kamailio/kamailio/commit/bd70a78ce22ab419a52e8733bc7ee27… Patch: https://github.com/kamailio/kamailio/commit/bd70a78ce22ab419a52e8733bc7ee27… --- diff --git a/src/modules/topos/doc/topos_admin.xml b/src/modules/topos/doc/topos_admin.xml index 4ccc4b7790f..f4fa47556e2 100644 --- a/src/modules/topos/doc/topos_admin.xml +++ b/src/modules/topos/doc/topos_admin.xml @@ -99,6 +99,11 @@ <emphasis>redis</emphasis> - Redis Backend </para> </listitem> + <listitem> + <para> + <emphasis>htable</emphasis> - htable module Backend + </para> + </listitem> </itemizedlist> <para> <emphasis> diff --git a/src/modules/topos/topos_mod.c b/src/modules/topos/topos_mod.c index 3214bdeb66d..c0db6ec72ec 100644 --- a/src/modules/topos/topos_mod.c +++ b/src/modules/topos/topos_mod.c @@ -288,6 +288,9 @@ static int mod_init(void) _tpsdbf.close(topos_db_con); topos_db_con = NULL; } + } else if(_tps_storage.len == 6 + && strncmp(_tps_storage.s, "htable", 6) == 0) { + // use htable module api } else { if(_tps_storage.len != 7 && strncmp(_tps_storage.s, "redis", 5) != 0) { LM_ERR("unknown storage type: %.*s\n", _tps_storage.len, diff --git a/src/modules/topos/tps_msg.c b/src/modules/topos/tps_msg.c index e0e1c211923..7fff763fe26 100644 --- a/src/modules/topos/tps_msg.c +++ b/src/modules/topos/tps_msg.c @@ -1081,6 +1081,14 @@ int tps_response_received(sip_msg_t *msg) tps_reappend_rr(msg, &btsd, &btsd.x_rr); tps_append_xbranch(msg, &mtsd.x_vbranch1); + if(msg->first_line.u.reply.statuscode > 299 + && (get_cseq(msg)->method_id + & (METHOD_INVITE | METHOD_SUBSCRIBE))) { + LM_DBG("%d reply end dialog storage\n", + msg->first_line.u.reply.statuscode); + tps_storage_end_dialog(msg, &mtsd, &stsd); + } + return 0; error:

1 month, 3 weeks

git:master:69db5823: htable: Updates for topos_htable module

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 69db5823fc9b8ee98c59d591517a1ecef91b16db URL: https://github.com/kamailio/kamailio/commit/69db5823fc9b8ee98c59d591517a1ec… Author: Stefan Mititelu <stefan.mititelu(a)net2phone.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-12-14T11:52:02+01:00 htable: Updates for topos_htable module Export table create api functions Fix ht_cell_pkg_copy str pointers. Otherwise, pkg cell str pointers will still point to shm memory. Set the pkg cell str pointers similar to how ht_cell_new sets the str pointers. --- Modified: src/modules/htable/api.c Modified: src/modules/htable/api.h Modified: src/modules/htable/ht_api.c --- Diff: https://github.com/kamailio/kamailio/commit/69db5823fc9b8ee98c59d591517a1ec… Patch: https://github.com/kamailio/kamailio/commit/69db5823fc9b8ee98c59d591517a1ec… --- diff --git a/src/modules/htable/api.c b/src/modules/htable/api.c index 57b2b4e75f1..94095d3d5d4 100644 --- a/src/modules/htable/api.c +++ b/src/modules/htable/api.c @@ -147,6 +147,16 @@ int ht_api_count_cells_re(str *hname, str *sre, int mode) return 0; } +int ht_api_table_spec(char *spec) +{ + return ht_table_spec(spec); +} + +int ht_api_init_tables(void) +{ + return ht_init_tables(); +} + /** * */ @@ -156,6 +166,8 @@ int bind_htable(htable_api_t *api) ERR("Invalid parameter value\n"); return -1; } + api->table_spec = ht_api_table_spec; + api->init_tables = ht_api_init_tables; api->set = ht_api_set_cell; api->get_clone = ht_api_get_cell_clone; api->rm = ht_api_del_cell; diff --git a/src/modules/htable/api.h b/src/modules/htable/api.h index 15e5207f1b2..8f67e5ca71a 100644 --- a/src/modules/htable/api.h +++ b/src/modules/htable/api.h @@ -25,6 +25,8 @@ #include "../../core/sr_module.h" #include "../../core/usr_avp.h" +typedef int (*ht_api_table_spec_f)(char *spec); +typedef int (*ht_api_init_tables_f)(void); typedef int (*ht_api_set_cell_f)( str *hname, str *name, int type, int_str *val, int mode); typedef ht_cell_t *(*ht_api_get_cell_clone_f)(str *hname, str *name); @@ -40,6 +42,8 @@ typedef int (*ht_api_count_cells_re_f)(str *hname, str *sre, int mode); typedef struct htable_api { + ht_api_table_spec_f table_spec; + ht_api_init_tables_f init_tables; ht_api_set_cell_f set; ht_api_get_cell_clone_f get_clone; ht_api_del_cell_f rm; diff --git a/src/modules/htable/ht_api.c b/src/modules/htable/ht_api.c index ecc49b547d6..8655c2bf443 100644 --- a/src/modules/htable/ht_api.c +++ b/src/modules/htable/ht_api.c @@ -359,6 +359,11 @@ int ht_init_tables(void) ht = _ht_root; while(ht) { + if(ht->entries != NULL) { + ht = ht->next; + continue; + } + LM_DBG("initializing htable [%.*s] with nr. of slots: %d\n", ht->name.len, ht->name.s, ht->htsize); if(ht->name.len + sizeof("htable:expired:") < HT_EVEX_NAME_SIZE) { @@ -860,8 +865,14 @@ ht_cell_t *ht_cell_pkg_copy(ht_t *ht, str *name, ht_cell_t *old) } } cell = (ht_cell_t *)pkg_malloc(it->msize); - if(cell != NULL) + if(cell != NULL) { memcpy(cell, it, it->msize); + + cell->name.s = (char *)cell + sizeof(ht_cell_t); + if(cell->flags & AVP_VAL_STR) { + cell->value.s.s = (char *)cell->name.s + cell->name.len + 1; + } + } ht_slot_unlock(ht, idx); return cell; }

1 month, 3 weeks

← Newer
1
...
12
13
14
15
16
17
18
...
35
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev December 2024