sr-dev February 2022

sr-dev@lists.kamailio.org

17 participants
242 discussions

[kamailio/kamailio] SystemError: <built-in function is_present> returned a result with an error set (Issue #3019)

by wojciechkopciewicz

### Description Intermittently the following error is observed when checking if given header is present in SIP INVITE: SystemError: <built-in function is_present> returned a result with an error set The complete stack trace is (some names redacted): 11(28) ERROR: {1 195 INVITE nMHZDN.10nOIDt4ldOwz5cqz0XnMepA9} app_python3 [python_support.c:156]: python_handle_exception(): apy_exec: modify_headers((null)): Unhandled exception in the Python code: TypeError: an integer is required (got type NoneType) The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/etc/kamailio/kamailio.py", line 25, in modify_headers self.modify_header("SomeHeader") File "/etc/kamailio/kamailio.py", line 29, in modify_header present = KSR.hdr.is_present(header_name) SystemError: <built-in function is_present> returned a result with an error set ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.4.6 (x86_64/linux) flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, 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_BLACKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_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: unknown compiled with gcc 8.3.0 ``` * **Operating System**:  ``` Linux kamailio-5c7484c74d-vhhr8 5.4.0-1056-azure #58~18.04.1-Ubuntu SMP Wed Jul 28 23:14:18 UTC 2021 x86_64 GNU/Linux ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3019 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3019(a)github.com>

2 years, 10 months

[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, 10 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, 10 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, 10 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, 10 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, 10 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, 10 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, 10 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, 10 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, 10 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