sr-dev February 2022

sr-dev@lists.kamailio.org

17 participants
241 discussions

[kamailio/kamailio] smsops: extend rpdata params manipulation (PR #3007)

by alexyosifov

- Add capability to set RPData originator and destination addresses flags. Default values 0x91.   #### 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  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3007 -- Commit Summary -- * smsops: extend rpdata params manipulation -- File Changes -- M src/modules/smsops/smsops_impl.c (52) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3007.patch https://github.com/kamailio/kamailio/pull/3007.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3007 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3007(a)github.com>

2 years, 3 months

git:master:9bf7601d: smsops: extend rpdata params manipulation

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 9bf7601d7340ab179619db3fe4b69bfa1373ed6e URL: https://github.com/kamailio/kamailio/commit/9bf7601d7340ab179619db3fe4b69bf… Author: Aleksandar Yosifov <alexyosifov(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-02-07T21:50:29+01:00 smsops: extend rpdata params manipulation - Add capability to set RPData originator and destination addresses flags. Default values 0x91. --- Modified: src/modules/smsops/smsops_impl.c --- Diff: https://github.com/kamailio/kamailio/commit/9bf7601d7340ab179619db3fe4b69bf… Patch: https://github.com/kamailio/kamailio/commit/9bf7601d7340ab179619db3fe4b69bf…

2 years, 3 months

[kamailio/kamailio] db_postgres: added support for async queries (PR #3010)

by Federico Cabiddu

#### 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 add support for async queries to db_postgres module. Implementation is the same as the mysql and unixodbx modules. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3010 -- Commit Summary -- * db_postgres: added support for async queries -- File Changes -- M src/modules/db_postgres/db_postgres.c (2) M src/modules/db_postgres/km_dbase.c (96) M src/modules/db_postgres/km_dbase.h (9) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3010.patch https://github.com/kamailio/kamailio/pull/3010.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3010 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3010(a)github.com>

2 years, 3 months

git:master:2ab74b2c: db_postgres: added support for async queries

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 2ab74b2cee705a97214a61936a6e940bcbf6eb75 URL: https://github.com/kamailio/kamailio/commit/2ab74b2cee705a97214a61936a6e940… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-02-07T21:49:06+01:00 db_postgres: added support for async queries --- Modified: src/modules/db_postgres/db_postgres.c Modified: src/modules/db_postgres/km_dbase.c Modified: src/modules/db_postgres/km_dbase.h --- Diff: https://github.com/kamailio/kamailio/commit/2ab74b2cee705a97214a61936a6e940… Patch: https://github.com/kamailio/kamailio/commit/2ab74b2cee705a97214a61936a6e940…

2 years, 3 months

[kamailio/kamailio] fixed format-truncation warning (PR #3017)

by sergey-safarov

#### 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 - [x] Related to issue #3013 #### Description fixed format-truncation warning You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3017 -- Commit Summary -- * presence: fixed format-truncation warning GH #3013 * rls: fixed format-truncation warning GH #3013 -- File Changes -- M src/modules/presence/presence.c (4) M src/modules/rls/rls.c (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3017.patch https://github.com/kamailio/kamailio/pull/3017.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3017 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3017(a)github.com>

2 years, 3 months

git:master:e921fc8e: rls: fixed format-truncation warning GH #3013

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: e921fc8ed7a6c9fdeb532179a77503cd67799487 URL: https://github.com/kamailio/kamailio/commit/e921fc8ed7a6c9fdeb532179a77503c… Author: Sergey Safarov <s.safarov(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-02-07T21:46:57+01:00 rls: fixed format-truncation warning GH #3013 --- Modified: src/modules/rls/rls.c --- Diff: https://github.com/kamailio/kamailio/commit/e921fc8ed7a6c9fdeb532179a77503c… Patch: https://github.com/kamailio/kamailio/commit/e921fc8ed7a6c9fdeb532179a77503c… --- diff --git a/src/modules/rls/rls.c b/src/modules/rls/rls.c index 19ecce0ca2..63c69a64c6 100644 --- a/src/modules/rls/rls.c +++ b/src/modules/rls/rls.c @@ -684,8 +684,8 @@ static int child_init(int rank) for (i = 0; i < rls_notifier_processes; i++) { - char tmp[16]; - snprintf(tmp, 16, "RLS NOTIFIER %d", i); + char tmp[25]; + snprintf(tmp, 25, "RLS NOTIFIER %d", i); rls_notifier_id[i] = i; if (fork_basic_utimer(PROC_TIMER, tmp, 1,

2 years, 3 months

git:master:c1b14f49: presence: fixed format-truncation warning GH #3013

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: c1b14f49ead9a29711cde5334155819cd1c6c90d URL: https://github.com/kamailio/kamailio/commit/c1b14f49ead9a29711cde5334155819… Author: Sergey Safarov <s.safarov(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-02-07T21:46:57+01:00 presence: fixed format-truncation warning GH #3013 --- Modified: src/modules/presence/presence.c --- Diff: https://github.com/kamailio/kamailio/commit/c1b14f49ead9a29711cde5334155819… Patch: https://github.com/kamailio/kamailio/commit/c1b14f49ead9a29711cde5334155819… --- diff --git a/src/modules/presence/presence.c b/src/modules/presence/presence.c index da253acbac..d6e287c40e 100644 --- a/src/modules/presence/presence.c +++ b/src/modules/presence/presence.c @@ -566,8 +566,8 @@ static int child_init(int rank) int i; for(i = 0; i < pres_notifier_processes; i++) { - char tmp[21]; - snprintf(tmp, 21, "PRESENCE NOTIFIER %d", i); + char tmp[30]; + snprintf(tmp, 30, "PRESENCE NOTIFIER %d", i); pres_notifier_id[i] = i; if(fork_basic_utimer(PROC_TIMER, tmp, 1, pres_timer_send_notify,

2 years, 3 months

[kamailio/kamailio] fixed misleading-indentation warning (PR #3016)

by sergey-safarov

#### 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 - [x] Related to issue #3012 #### Description fixed misleading-indentation warning You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3016 -- Commit Summary -- * sms: fixed misleading-indentation warning GH #3012 * ims_charging: fixed misleading-indentation warning GH #3012 * kazoo: fixed misleading-indentation warning GH #3012 -- File Changes -- M src/modules/ims_charging/ims_ro.c (26) M src/modules/kazoo/kz_amqp.c (21) M src/modules/sms/sms_funcs.c (29) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3016.patch https://github.com/kamailio/kamailio/pull/3016.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3016 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3016(a)github.com>

2 years, 3 months

git:master:2b9706ac: kazoo: fixed misleading-indentation warning GH #3012

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 2b9706aca33b8ca34d3d9c58b80527956c92dd7a URL: https://github.com/kamailio/kamailio/commit/2b9706aca33b8ca34d3d9c58b805279… Author: Sergey Safarov <s.safarov(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-02-07T21:45:43+01:00 kazoo: fixed misleading-indentation warning GH #3012 --- Modified: src/modules/kazoo/kz_amqp.c --- Diff: https://github.com/kamailio/kamailio/commit/2b9706aca33b8ca34d3d9c58b805279… Patch: https://github.com/kamailio/kamailio/commit/2b9706aca33b8ca34d3d9c58b805279… --- diff --git a/src/modules/kazoo/kz_amqp.c b/src/modules/kazoo/kz_amqp.c index 4d9e3dcd8a..15d5b259a1 100644 --- a/src/modules/kazoo/kz_amqp.c +++ b/src/modules/kazoo/kz_amqp.c @@ -798,9 +798,9 @@ void kz_amqp_connection_close(kz_amqp_conn_ptr rmq) { } void kz_amqp_channel_close(kz_amqp_conn_ptr rmq, amqp_channel_t channel) { - LM_DBG("Close rmq channel\n"); - if (!rmq) - return; + LM_DBG("Close rmq channel\n"); + if (!rmq) + return; LM_DBG("close channel: %d rmq(%p)->channel(%d)\n", getpid(), (void *)rmq, channel); kz_amqp_error("closing channel", amqp_channel_close(rmq->conn, channel, AMQP_REPLY_SUCCESS)); @@ -1126,11 +1126,11 @@ int kz_amqp_pipe_send(str *str_exchange, str *str_routing_key, str *str_payload, if(cmd) kz_amqp_free_pipe_cmd(cmd); - if(json_obj) - json_object_put(json_obj); + if(json_obj) + json_object_put(json_obj); - if(s_local_routing_key) - pkg_free(s_local_routing_key); + if(s_local_routing_key) + pkg_free(s_local_routing_key); return ret; } @@ -2045,14 +2045,13 @@ int kz_amqp_subscribe_simple(struct sip_msg* msg, char* exchange, char* exchange binding->bind = bind; bindings_count++; - return 1; + return 1; error: - if(binding != NULL) - shm_free(binding); + if(binding != NULL) + shm_free(binding); return -1; - }

2 years, 3 months

git:master:d94879b0: ims_charging: fixed misleading-indentation warning GH #3012

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: d94879b06f9bbb9d6ffa2dcaae5b213f169f2d5a URL: https://github.com/kamailio/kamailio/commit/d94879b06f9bbb9d6ffa2dcaae5b213… Author: Sergey Safarov <s.safarov(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-02-07T21:45:43+01:00 ims_charging: fixed misleading-indentation warning GH #3012 --- Modified: src/modules/ims_charging/ims_ro.c --- Diff: https://github.com/kamailio/kamailio/commit/d94879b06f9bbb9d6ffa2dcaae5b213… Patch: https://github.com/kamailio/kamailio/commit/d94879b06f9bbb9d6ffa2dcaae5b213… --- diff --git a/src/modules/ims_charging/ims_ro.c b/src/modules/ims_charging/ims_ro.c index d93d565a65..11b2335f51 100644 --- a/src/modules/ims_charging/ims_ro.c +++ b/src/modules/ims_charging/ims_ro.c @@ -1604,23 +1604,23 @@ static int create_cca_result_code(int result) { static int create_cca_fui_avps(int action, str* redirecturi) { int_str action_avp_val, action_avp_name, redirecturi_avp_val, redirecturi_avp_name; - action_avp_name.s.s = RO_AVP_CCA_FUI_ACTION; - action_avp_name.s.len = RO_AVP_CCA_FUI_ACTION_LENGTH; + action_avp_name.s.s = RO_AVP_CCA_FUI_ACTION; + action_avp_name.s.len = RO_AVP_CCA_FUI_ACTION_LENGTH; redirecturi_avp_name.s.s = RO_AVP_CCA_FUI_REDIRECT_URI; - redirecturi_avp_name.s.len = RO_AVP_CCA_FUI_REDIRECT_URI_LENGTH; - char buf[10]; + redirecturi_avp_name.s.len = RO_AVP_CCA_FUI_REDIRECT_URI_LENGTH; + char buf[10]; int rc; - action_avp_val.n = action; - action_avp_val.s.len = snprintf(buf, 10, "%i", action); - action_avp_val.s.s = buf; + action_avp_val.n = action; + action_avp_val.s.len = snprintf(buf, 10, "%i", action); + action_avp_val.s.s = buf; - rc = add_avp(AVP_NAME_STR|AVP_VAL_STR, action_avp_name, action_avp_val); + rc = add_avp(AVP_NAME_STR|AVP_VAL_STR, action_avp_name, action_avp_val); - if (rc < 0) - LM_ERR("Couldn't create ["RO_AVP_CCA_FUI_ACTION"] AVP\n"); - else - LM_DBG("Created AVP ["RO_AVP_CCA_FUI_ACTION"] successfully: value=[%d]\n", action); + if (rc < 0) + LM_ERR("Couldn't create ["RO_AVP_CCA_FUI_ACTION"] AVP\n"); + else + LM_DBG("Created AVP ["RO_AVP_CCA_FUI_ACTION"] successfully: value=[%d]\n", action); if (redirecturi && redirecturi->len >0 && redirecturi->s) { redirecturi_avp_val.s.len = redirecturi->len; @@ -1634,7 +1634,7 @@ static int create_cca_fui_avps(int action, str* redirecturi) { LM_DBG("Created AVP ["RO_AVP_CCA_FUI_REDIRECT_URI"] successfully: value=[%.*s]\n", redirecturi->len, redirecturi->s); } - return 1; + return 1; } static int get_mac_avp_value(struct sip_msg *msg, str *value) {

2 years, 3 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev February 2022