sr-dev December 2023

sr-dev@lists.kamailio.org

14 participants
234 discussions

git:master:84f6ca9b: main: fix build warning

by Victor Seva

Module: kamailio Branch: master Commit: 84f6ca9b453b33dba321c11e12a0ebc0b9b8dab4 URL: https://github.com/kamailio/kamailio/commit/84f6ca9b453b33dba321c11e12a0ebc… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-12-22T09:49:43+01:00 main: fix build warning > CC (gcc) [kamailio] core/atomic_ops.o > main.c:34: warning: "_GNU_SOURCE" redefined > 34 | #define _GNU_SOURCE > | > <command-line>: note: this is the location of the previous definition --- Modified: src/main.c --- Diff: https://github.com/kamailio/kamailio/commit/84f6ca9b453b33dba321c11e12a0ebc… Patch: https://github.com/kamailio/kamailio/commit/84f6ca9b453b33dba321c11e12a0ebc… --- diff --git a/src/main.c b/src/main.c index 3aa79b73d91..c107f354672 100644 --- a/src/main.c +++ b/src/main.c @@ -31,7 +31,9 @@ */ #ifdef KSR_PTHREAD_MUTEX_SHARED +#ifndef _GNU_SOURCE #define _GNU_SOURCE +#endif #include <pthread.h> #include <dlfcn.h> #endif

1 year

git:master:5ebe1758: ims_qos: fix build warning

by Victor Seva

Module: kamailio Branch: master Commit: 5ebe1758fb3ff03e3a035e7121f40e2539eb2c9c URL: https://github.com/kamailio/kamailio/commit/5ebe1758fb3ff03e3a035e7121f40e2… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-12-22T09:32:10+01:00 ims_qos: fix build warning > CC (gcc) [M ims_qos.so] rx_avp.o > [...] > /usr/include/fortify/stdio.h:122:16: warning: 'int_port_rctp_a' may be used uninitialized [-Wmaybe-uninitialized] > [...] > /usr/include/fortify/stdio.h:122:16: warning: 'int_port_rctp_b' may be used uninitialized [-Wmaybe-uninitialized] --- Modified: src/modules/ims_qos/rx_avp.c --- Diff: https://github.com/kamailio/kamailio/commit/5ebe1758fb3ff03e3a035e7121f40e2… Patch: https://github.com/kamailio/kamailio/commit/5ebe1758fb3ff03e3a035e7121f40e2… --- diff --git a/src/modules/ims_qos/rx_avp.c b/src/modules/ims_qos/rx_avp.c index d7d6abd9bc2..7a64d5e9713 100644 --- a/src/modules/ims_qos/rx_avp.c +++ b/src/modules/ims_qos/rx_avp.c @@ -670,7 +670,7 @@ AAA_AVP *rx_create_media_subcomponent_avp(int number, str *proto, str *ipA, str data; int len, len2; - int int_port_rctp_a, int_port_rctp_b; + int int_port_rctp_a = 0, int_port_rctp_b = 0; str port_rtcp_a = STR_NULL, port_rtcp_b = STR_NULL; AAA_AVP *flow_description1 = 0, *flow_description2 = 0, *flow_description3 = 0, *flow_description4 = 0, *flow_number = 0;

1 year

git:master:59dc9b1b: core: fix build warning

by Victor Seva

Module: kamailio Branch: master Commit: 59dc9b1ba93d4e6a06f87deabf136787a3ab1cf1 URL: https://github.com/kamailio/kamailio/commit/59dc9b1ba93d4e6a06f87deabf13678… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-12-22T09:27:01+01:00 core: fix build warning > core/parser/parse_diversion.c: In function 'parse_diversion_body': core/parser/parse_diversion.c:60:18: warning: the comparison will always evaluate as 'false' for the address of 'uri_b' will never be NULL [-Waddress] > 60 | if(uri_b == NULL) { > | ^~ > core/parser/parse_diversion.c:51:26: note: 'uri_b' declared here > 51 | static to_body_t uri_b[NUM_DIVERSION_BODIES]; /* > Temporary storage */ > | --- Modified: src/core/parser/parse_diversion.c --- Diff: https://github.com/kamailio/kamailio/commit/59dc9b1ba93d4e6a06f87deabf13678… Patch: https://github.com/kamailio/kamailio/commit/59dc9b1ba93d4e6a06f87deabf13678… --- diff --git a/src/core/parser/parse_diversion.c b/src/core/parser/parse_diversion.c index 539f6a7bc44..d60d0cbdd1f 100644 --- a/src/core/parser/parse_diversion.c +++ b/src/core/parser/parse_diversion.c @@ -55,12 +55,7 @@ int parse_diversion_body(char *buf, int len, diversion_body_t **body) int i; to_param_t *params; - /* Reserves memory max NUM_DIVERSION_BODIES times */ memset(uri_b, 0, NUM_DIVERSION_BODIES * sizeof(to_body_t)); - if(uri_b == NULL) { - LM_ERR("Error allocating memory for uri_b\n"); - return -1; - } tmp = parse_addr_spec(buf, buf + len, &uri_b[num_uri], 1); if(uri_b[num_uri].error == PARSE_ERROR) {

1 year

git:master:ed0f2e77: pkg/kamailio/alpine: remove app_sqlang module [skip ci]

by Victor Seva

Module: kamailio Branch: master Commit: ed0f2e77b80a2719957b1ac43a96dfc57b88f98e URL: https://github.com/kamailio/kamailio/commit/ed0f2e77b80a2719957b1ac43a96dfc… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-12-22T09:13:28+01:00 pkg/kamailio/alpine: remove app_sqlang module [skip ci] module was moved to kamailio-archive repository --- Modified: pkg/kamailio/alpine/APKBUILD --- Diff: https://github.com/kamailio/kamailio/commit/ed0f2e77b80a2719957b1ac43a96dfc… Patch: https://github.com/kamailio/kamailio/commit/ed0f2e77b80a2719957b1ac43a96dfc… --- diff --git a/pkg/kamailio/alpine/APKBUILD b/pkg/kamailio/alpine/APKBUILD index c69e4dca7fe..b3d5c4d64bd 100644 --- a/pkg/kamailio/alpine/APKBUILD +++ b/pkg/kamailio/alpine/APKBUILD @@ -227,9 +227,6 @@ _mod_list_jansson="acc_json jansson janssonrpcc" # - modules depending on libm _mod_list_jsdt="app_jsdt" -# - modules depending on sqlang -_mod_list_sqlang="app_sqlang" - # - modules depending on rabbitmq _mod_list_rabbitmq="rabbitmq" @@ -251,7 +248,7 @@ for _i in db postgres sqlite dbtext mysql \ cpl xml unixodbc snmpstats xmpp carrierroute \ ldap utils tls presence lua ims outbound debugger \ extras json websocket authephemeral mongodb\ - uuid ev memcached redis geoip2 jansson sqlang sipdump \ + uuid ev memcached redis geoip2 jansson sipdump \ jsdt http_async kazoo rabbitmq sctp radius perl \ python3 ruby; do @@ -557,11 +554,6 @@ ruby() { "$_mod_list_ruby" } -sqlang() { - _generic_pkg "Squirrel Language (SQLang) for Kamailio" \ - "$_mod_list_sqlang" -} - rabbitmq() { _generic_pkg "RabbitMQ related modules for Kamailio" \ "$_mod_list_rabbitmq"

1 year

git:master:268147a7: core: fix build warning

by Victor Seva

Module: kamailio Branch: master Commit: 268147a7a7bcf68689a913a94faecc87128187fb URL: https://github.com/kamailio/kamailio/commit/268147a7a7bcf68689a913a94faecc8… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-12-22T09:00:08+01:00 core: fix build warning > CC (gcc) [M jansson.so] jansson_utils.o > jansson_utils.c:23: warning: "_GNU_SOURCE" redefined > 23 | #define _GNU_SOURCE > | > <command-line>: note: this is the location of the previous definition --- Modified: src/core/io_wait.h --- Diff: https://github.com/kamailio/kamailio/commit/268147a7a7bcf68689a913a94faecc8… Patch: https://github.com/kamailio/kamailio/commit/268147a7a7bcf68689a913a94faecc8… --- diff --git a/src/core/io_wait.h b/src/core/io_wait.h index 8a60ae280fa..14d93f5937f 100644 --- a/src/core/io_wait.h +++ b/src/core/io_wait.h @@ -47,7 +47,9 @@ #include <signal.h> /* sigprocmask, sigwait a.s.o */ #endif +#ifndef _GNU_SOURCE #define _GNU_SOURCE /* for POLLRDHUP on linux */ +#endif #include <poll.h> #include <fcntl.h>

1 year

[kamailio/kamailio] build error on Alpine: grep -P not supported in BusyBox anymore (Issue #3686)

by Victor Seva

### Description It seems PCRE is not supported anymore ``` make[2]: --libs: No such file or directory grep: unrecognized option: P BusyBox v1.36.1 (2023-12-20 14:59:51 UTC) multi-call binary. ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3686 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3686(a)github.com>

1 year

git:master:ca450901: ndb_redis: replace use of grep -P

by Victor Seva

Module: kamailio Branch: master Commit: ca4509013a123f02d6d9c09d016fce3fea3e3dbe URL: https://github.com/kamailio/kamailio/commit/ca4509013a123f02d6d9c09d016fce3… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-12-22T08:53:45+01:00 ndb_redis: replace use of grep -P PCRE option is been removed in BusyBox #3686 --- Modified: src/modules/ndb_redis/Makefile --- Diff: https://github.com/kamailio/kamailio/commit/ca4509013a123f02d6d9c09d016fce3… Patch: https://github.com/kamailio/kamailio/commit/ca4509013a123f02d6d9c09d016fce3… --- diff --git a/src/modules/ndb_redis/Makefile b/src/modules/ndb_redis/Makefile index ba085ca2735..429b69d6ea3 100644 --- a/src/modules/ndb_redis/Makefile +++ b/src/modules/ndb_redis/Makefile @@ -24,7 +24,7 @@ else HIREDISLIBS = $(shell $(HIREDIS_BUILDER) --libs) HIREDISLIBSPATH = $(shell $(HIREDIS_BUILDER) --libs-only-L | cut -c 3-) ifeq ($(HIREDISLIBSPATH),) - GCCSEARCHDIRS = $(shell $(CC) -print-search-dirs | grep -Po '^.*libraries: =.*' | cut -d "=" -f2- | tr : ' ') + GCCSEARCHDIRS = $(shell $(CC) -print-search-dirs | grep -Eo '^.*libraries: =.*' | cut -d "=" -f2- | tr : ' ') ifneq ($(shell find $(GCCSEARCHDIRS) -name libhiredis_ssl.so 2>/dev/null),) HIREDISDEFS += -DWITH_SSL HIREDISLIBS += -lhiredis_ssl

1 year

git:master:16302854: db_redis: replace use of grep -P

by Victor Seva

Module: kamailio Branch: master Commit: 163028549df3abd3173a7c0ac8a9dc666c0e4879 URL: https://github.com/kamailio/kamailio/commit/163028549df3abd3173a7c0ac8a9dc6… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2023-12-22T08:53:32+01:00 db_redis: replace use of grep -P PCRE option is been removed in BusyBox #3686 --- Modified: src/modules/db_redis/Makefile --- Diff: https://github.com/kamailio/kamailio/commit/163028549df3abd3173a7c0ac8a9dc6… Patch: https://github.com/kamailio/kamailio/commit/163028549df3abd3173a7c0ac8a9dc6… --- diff --git a/src/modules/db_redis/Makefile b/src/modules/db_redis/Makefile index 815cd056b9d..85d9b9a956f 100644 --- a/src/modules/db_redis/Makefile +++ b/src/modules/db_redis/Makefile @@ -30,7 +30,7 @@ else HIREDISLIBS = $(shell $(HIREDIS_BUILDER) --libs) HIREDISLIBSPATH = $(shell $(HIREDIS_BUILDER) --libs-only-L | cut -c 3-) ifeq ($(HIREDISLIBSPATH),) - GCCSEARCHDIRS = $(shell $(CC) -print-search-dirs | grep -Po '^.*libraries: =.*' | cut -d "=" -f2- | tr : ' ') + GCCSEARCHDIRS = $(shell $(CC) -print-search-dirs | grep -Eo '^.*libraries: =.*' | cut -d "=" -f2- | tr : ' ') ifneq ($(shell find $(GCCSEARCHDIRS) -name libhiredis_ssl.so 2>/dev/null),) HIREDISDEFS += -DWITH_SSL HIREDISLIBS += -lhiredis_ssl

1 year

git:master:4c939854: pua: fix possible copy/paste error, we should use reginfo_increase_version for reginfo events (GH #3234)

by Henning Westerholt

Module: kamailio Branch: master Commit: 4c93985437122706bc68143b2f96a9d7f35be16d URL: https://github.com/kamailio/kamailio/commit/4c93985437122706bc68143b2f96a9d… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2023-12-21T16:10:25Z pua: fix possible copy/paste error, we should use reginfo_increase_version for reginfo events (GH #3234) --- Modified: src/modules/pua/add_events.c --- Diff: https://github.com/kamailio/kamailio/commit/4c93985437122706bc68143b2f96a9d… Patch: https://github.com/kamailio/kamailio/commit/4c93985437122706bc68143b2f96a9d… --- diff --git a/src/modules/pua/add_events.c b/src/modules/pua/add_events.c index e2fc78be3ee..bf4a437e4b4 100644 --- a/src/modules/pua/add_events.c +++ b/src/modules/pua/add_events.c @@ -82,7 +82,7 @@ int pua_add_events(void) } /* add application/reginfo+xml */ - if(dlginfo_increase_version) { + if(reginfo_increase_version) { if(add_pua_event(REGINFO_EVENT, "reg", "application/reginfo+xml", reginfo_process_body) < 0) {

1 year

git:master:ac4fa112: tmx: use unsigned for bitwise operations

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: ac4fa1129f5fbd1a48aad1b7dc21a2c130ecd164 URL: https://github.com/kamailio/kamailio/commit/ac4fa1129f5fbd1a48aad1b7dc21a2c… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2023-12-21T16:05:04+01:00 tmx: use unsigned for bitwise operations --- Modified: src/modules/tmx/tmx_pretran.c --- Diff: https://github.com/kamailio/kamailio/commit/ac4fa1129f5fbd1a48aad1b7dc21a2c… Patch: https://github.com/kamailio/kamailio/commit/ac4fa1129f5fbd1a48aad1b7dc21a2c… --- diff --git a/src/modules/tmx/tmx_pretran.c b/src/modules/tmx/tmx_pretran.c index dcaebd1f41b..7600a3c7b4b 100644 --- a/src/modules/tmx/tmx_pretran.c +++ b/src/modules/tmx/tmx_pretran.c @@ -72,19 +72,20 @@ static int _tmx_ptran_size = 0; */ int tmx_init_pretran_table(void) { - int n; - int pn; + unsigned int n; + unsigned int pn; - pn = get_max_procs(); + pn = (unsigned int)get_max_procs(); - if(pn <= 0) + if(pn == 0) return -1; if(_tmx_ptran_table != NULL) return -1; + n = 1; /* get the highest power of two less than number of processes */ - n = -1; - while(pn >> ++n > 0) - ; + while((pn >> n) > 0) { + n++; + } n--; if(n <= 1) n = 2;

1 year

← Newer
1
...
8
9
10
11
12
13
14
...
24
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 2023