[sr-dev] Re: [kamailio/kamailio] dialog: failover database loading behaviour in shared DB scenario, expiration of dialogs (#2080)

2024-01-05 Thread github-actions[bot] via sr-dev
This issue is stale because it has been open 6 weeks with no activity. Remove 
stale label or comment or this will be closed in 2 weeks.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/2080#issuecomment-1879505889
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] git:master:95d8d937: msilo: make static and add prefix to global variables with short name

2024-01-05 Thread Daniel-Constantin Mierla via sr-dev
Module: kamailio
Branch: master
Commit: 95d8d9374a0e4385fd549078fcf56ecadef424a0
URL: 
https://github.com/kamailio/kamailio/commit/95d8d9374a0e4385fd549078fcf56ecadef424a0

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2024-01-05T21:36:29+01:00

msilo: make static and add prefix to global variables with short name

---

Modified: src/modules/msilo/msilo.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/95d8d9374a0e4385fd549078fcf56ecadef424a0.diff
Patch: 
https://github.com/kamailio/kamailio/commit/95d8d9374a0e4385fd549078fcf56ecadef424a0.patch

___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] Dispatcher: added two new flags to mode parameter of ds_is_from_list function for more strictly matching (PR #3699)

2024-01-05 Thread Dennis via sr-dev
> I've not done any testing myself, but by my understanding the 
> DS_MATCH_STRICTEST flag is basically a flag to indicate it should not return 
> on the first match but scan through all. Then sort and return the one that is 
> matching one most criterias?

Right, but not exactly, there is no sorting as separate entity, during the 
traverse of tree the gobal variable filled with strictess flags (which are the 
bitmask of sip uri components - addr/proto/port and local socket) and more 
strictess node is saved in process global variable, if full set of possible 
flags is found, the cycle is stopped (we found full match and no more 
iterations needed), when traverse pocess finished (resursive method in original 
code) and we are not found full match, the global vars will contains more 
strictness node (if found at least address match), from this global vars then 
making out the final result of ds_is_from_list.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3699#issuecomment-1879215548
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] Dispatcher: added two new flags to mode parameter of ds_is_from_list function for more strictly matching (PR #3699)

2024-01-05 Thread Dennis via sr-dev
> Maybe DS_MATCH_FULLADDRSOCK.

OK, i will change the flag name, need some time to think about adequate name, 
writing docs hope can help with this.



-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3699#issuecomment-1879191053
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] sip_trace(address, "", "t") uses the duplicate_uri for replies from auth_challenge instead of address argument in sip_trace() (Issue #3700)

2024-01-05 Thread Markus Monka via sr-dev
Hi,

407 is a reply, so i guess, kamailio sends the duplicate message to default
target configured in modpraram.

You should configure something like

onreply_route {

  if 407
sip_trace(sip:Y.Y.Y.Y)
}

Markus

On Fri, Jan 5, 2024 at 2:23 PM Niklas Larsson via sr-dev <
sr-dev@lists.kamailio.org> wrote:

> Description
>
> Using sip_trace() with different address arguments:
>
> modparam("siptrace", "duplicate_uri", "sip:X.X.X.X:9060")
>
> # siptrace till Homer
> if(method == "REGISTER"){
>   sip_trace("sip:X.X.X.X:9060", "", "t");
> }
>
> # siptrace till Voipmonitor
> if (is_method("INVITE|BYE|UPDATE|CANCEL")) {
>   sip_trace("sip:Y.Y.Y.Y:9060", "", "t");
> }
>
> Will send replies from auth_challenge to the address defined in
> duplicate_uri. So in the case above - no 407 are sent to
> "sip:Y.Y.Y.Y:9060", instead the are sent to "sip:X.X.X.X:9060" when
> challenging an Invite.
> Additional Information
>
>- *Kamailio Version* - output of kamailio -v
>
> version: kamailio 5.6.5 (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_BLOCKLIST, 
> 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
>
>
> —
> Reply to this email directly, view it on GitHub
> , or unsubscribe
> 
> .
> You are receiving this because you are subscribed to this thread.Message
> ID: 
> ___
> Kamailio (SER) - Development Mailing List
> To unsubscribe send an email to sr-dev-le...@lists.kamailio.org
>
___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] [kamailio/kamailio] sip_trace(address, "", "t") uses the duplicate_uri for replies from auth_challenge instead of address argument in sip_trace() (Issue #3700)

2024-01-05 Thread Niklas Larsson via sr-dev


### Description

Using sip_trace() with different address arguments:
```
modparam("siptrace", "duplicate_uri", "sip:X.X.X.X:9060")

# siptrace till Homer
if(method == "REGISTER"){
sip_trace("sip:X.X.X.X:9060", "", "t");
}

# siptrace till Voipmonitor
if (is_method("INVITE|BYE|UPDATE|CANCEL")) {
sip_trace("sip:Y.Y.Y.Y:9060", "", "t");
}
```
Will send replies from auth_challenge to the address defined in duplicate_uri. 
So in the case above - no 407 are sent to "sip:Y.Y.Y.Y:9060", instead the are 
sent to "sip:X.X.X.X:9060" when challenging an Invite.

### Additional Information

  * **Kamailio Version** - output of `kamailio -v`

```
version: kamailio 5.6.5 (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_BLOCKLIST, 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

```


-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/3700
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] git:master:4708f537: outbound: build, fix missing argument name

2024-01-05 Thread S-P Chan via sr-dev
Module: kamailio
Branch: master
Commit: 4708f537d7f5d28123b48cd89474a4931dd698ad
URL: 
https://github.com/kamailio/kamailio/commit/4708f537d7f5d28123b48cd89474a4931dd698ad

Author: S-P Chan 
Committer: S-P Chan 
Date: 2024-01-05T20:56:39+08:00

outbound: build, fix missing argument name

---

Modified: src/modules/outbound/outbound_mod.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/4708f537d7f5d28123b48cd89474a4931dd698ad.diff
Patch: 
https://github.com/kamailio/kamailio/commit/4708f537d7f5d28123b48cd89474a4931dd698ad.patch

---

diff --git a/src/modules/outbound/outbound_mod.c 
b/src/modules/outbound/outbound_mod.c
index a797eb8132e..32f10f6cdb9 100644
--- a/src/modules/outbound/outbound_mod.c
+++ b/src/modules/outbound/outbound_mod.c
@@ -75,7 +75,7 @@ struct module_exports exports = {
destroy  /* destroy 
function */
 };
 
-static void *mod_init_openssl(void *) {
+static void *mod_init_openssl(void *arg) {
 if(flow_token_secret.s) {
 assert(ob_key.len == SHA_DIGEST_LENGTH);
 LM_DBG("flow_token_secret mod param set. use persistent ob_key");

___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] Dispatcher: added two new flags to mode parameter of ds_is_from_list function for more strictly matching (PR #3699)

2024-01-05 Thread Morten Tryfoss via sr-dev
I've not done any testing myself, but by my understanding the 
DS_MATCH_STRICTEST flag is basically a flag to indicate it should not return on 
the first match but scan through all. Then sort and return the one that is 
matching one most criterias?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3699#issuecomment-1878505345
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] git:master:ef45b7b8: tm: free new uac dlg in case of errors

2024-01-05 Thread Daniel-Constantin Mierla via sr-dev
Module: kamailio
Branch: master
Commit: ef45b7b804a3ba79880afd218f6b3f3962c85ed7
URL: 
https://github.com/kamailio/kamailio/commit/ef45b7b804a3ba79880afd218f6b3f3962c85ed7

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2024-01-05T11:55:03+01:00

tm: free new uac dlg in case of errors

---

Modified: src/modules/tm/dlg.c
Modified: src/modules/tm/rpc_uac.c
Modified: src/modules/tm/t_funcs.c
Modified: src/modules/tm/t_lookup.c
Modified: src/modules/tm/tm.c
Modified: src/modules/tm/uac.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/ef45b7b804a3ba79880afd218f6b3f3962c85ed7.diff
Patch: 
https://github.com/kamailio/kamailio/commit/ef45b7b804a3ba79880afd218f6b3f3962c85ed7.patch

___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] dmq_usrloc: DMQ also copies and sends attributes related to contacts (PR #3679)

2024-01-05 Thread Henning Westerholt via sr-dev
Thanks, yes this was basically the question also @xkaraman had earlier. The 
XAVPs will be used in the cfg script context later on I think. He will do 
another review.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3679#issuecomment-1878409382
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] dmq_usrloc: DMQ also copies and sends attributes related to contacts (PR #3679)

2024-01-05 Thread Daniel-Constantin Mierla via sr-dev
I haven't gotten the time to review if the list of XAVPs can be just linked or 
needs to be cloned. If it is just used temporarily in a function, then linking 
should be fine, but if it needs to be kept for long, then it could be that 
clone has to be done, otherwise the list stays linked from two places and 
attempted to be destroyed twice.

If the above is clarified, I am fine to merge it.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3679#issuecomment-1878386870
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] Dispatcher: added two new flags to mode parameter of ds_is_from_list function for more strictly matching (PR #3699)

2024-01-05 Thread Daniel-Constantin Mierla via sr-dev
Thanks for the PR!

I haven't done a full review, but first, you have to add documentation to the 
xml fine in the doc/ subfolder. It helps to understand how is supposed to be 
used and see if follows the code.

Also, the DS_MATCH_STRICTEST should be changed to try to reflect better what 
kind of match is done, I am sure one can think of another match mode more 
stricter, to include more than ip/port/proto/socket. Maybe 
DS_MATCH_FULLADDRSOCK.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3699#issuecomment-1878381418
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] git:master:8038fba2: pv_core: Fix negative index bug for hfl()

2024-01-05 Thread Daniel-Constantin Mierla via sr-dev
Module: kamailio
Branch: master
Commit: 8038fba2dd056193b37a5c2571f2d4d910c47e73
URL: 
https://github.com/kamailio/kamailio/commit/8038fba2dd056193b37a5c2571f2d4d910c47e73

Author: Xenofon Karamanos 
Committer: Daniel-Constantin Mierla 
Date: 2024-01-05T10:25:02+01:00

pv_core: Fix negative index bug for hfl()

Before: -1 yielded null and -2 the last element of a header

---

Modified: src/modules/pv/pv_core.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/8038fba2dd056193b37a5c2571f2d4d910c47e73.diff
Patch: 
https://github.com/kamailio/kamailio/commit/8038fba2dd056193b37a5c2571f2d4d910c47e73.patch

---

diff --git a/src/modules/pv/pv_core.c b/src/modules/pv/pv_core.c
index d98abbf975d..129247f2688 100644
--- a/src/modules/pv/pv_core.c
+++ b/src/modules/pv/pv_core.c
@@ -2149,7 +2149,7 @@ int pv_get_hfl(sip_msg_t *msg, pv_param_t *param, 
pv_value_t *res)
return pv_get_null(msg, param, res);
}
if(idx < 0) {
-   n = 1;
+   n = 0;
/* count Via header bodies */
for(hf = msg->h_via1; hf != NULL; hf = hf->next) {
if(hf->type == HDR_VIA_T) {
@@ -2207,7 +2207,7 @@ int pv_get_hfl(sip_msg_t *msg, pv_param_t *param, 
pv_value_t *res)
}
 
if(idx < 0) {
-   n = 1;
+   n = 0;
/* count Record-Route/Route header bodies */
for(; hf != NULL; hf = hf->next) {
if(hf->type == tv.ri) {
@@ -2290,7 +2290,7 @@ int pv_get_hfl(sip_msg_t *msg, pv_param_t *param, 
pv_value_t *res)
return pv_get_null(msg, param, res);
}
if(idx < 0) {
-   n = 1;
+   n = 0;
/* count Contact header bodies */
for(hf = msg->contact; hf != NULL; hf = hf->next) {
if(hf->type == HDR_CONTACT_T) {

___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] pv_core: Fix negative index bug for hfl() (PR #3697)

2024-01-05 Thread Daniel-Constantin Mierla via sr-dev
Merged #3697 into master.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3697#event-11399179201
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] pv_core: Fix negative index bug for hfl() (PR #3697)

2024-01-05 Thread Daniel-Constantin Mierla via sr-dev
Thanks!

I thought it was based on the same approach as for `$(hdr(name)[index])`, but 
that works fine.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3697#issuecomment-1878365467
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] dmq_usrloc: DMQ also copies and sends attributes related to contacts (PR #3679)

2024-01-05 Thread Henning Westerholt via sr-dev
Are there more comments or objections? If not I would merge it in the next days.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3679#issuecomment-1878352646
You are receiving this because you are subscribed to this thread.

Message ID: ___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Re: [kamailio/kamailio] parser/contact: Change addition order to the end of list (PR #3698)

2024-01-05 Thread Xenofon Karamanos via sr-dev
@xkaraman pushed 1 commit.

2cd4b4dfcc4e9a642a9d02de7cc2541e913586b3  parser/contact: Change addition order 
to the end of list

-- 
View it on GitHub:
https://github.com/kamailio/kamailio/pull/3698/files/cf39918a7d22c1f55b74c22fa659e79f7eccff3e..2cd4b4dfcc4e9a642a9d02de7cc2541e913586b3
You are receiving this because you are subscribed to this thread.

Message ID: 
___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org


[sr-dev] Build failed in Jenkins: kamailiodev-nightly-binaries » amd64,xenial #2567

2024-01-05 Thread kamailio--- via sr-dev
See 


Changes:


--
[...truncated 1.45 MB...]
make[2]: Entering directory 
'/build/kamailio-5.8.0~dev2+ubuntu16.04.20240105005443.2590/src'
config.mak loaded
Makefile.defs defs skipped
Makefile.defs defs skipped
Makefile.defs defs skipped
gcc -fPIC -DPIC -funroll-loops -Wcast-align -m64 -minline-all-stringops 
-falign-loops -ftree-vectorize -fno-strict-overflow -mtune=generic -pthread 
-DKSR_PTHREAD_MUTEX_SHARED -Wall -Wdate-time -D_FORTIFY_SOURCE=2 -g -O2 
-fstack-protector-strong -Wformat -Werror=format-security -DVERSION_NODATE   
-DNAME='"kamailio"' -DVERSION='"5.8.0-dev2"' -DARCH='"x86_64"' -DOS='linux_' 
-DOS_QUOTED='"linux"' -DCOMPILER='"gcc 5.4.0"' -D__CPU_x86_64 -D__OS_linux 
-DVERSIONVAL=5008000 -DCFG_DIR='"/etc/kamailio/"' 
-DSHARE_DIR='"/usr/share/kamailio/"' -DRUN_DIR='"/var/run/kamailio/"' 
-DPKG_MALLOC -DSHM_MMAP -DDNS_IP_HACK -DUSE_MCAST -DUSE_TCP -DDISABLE_NAGLE 
-DHAVE_RESOLV_RES -DUSE_DNS_CACHE -DUSE_DNS_FAILOVER -DUSE_DST_BLOCKLIST 
-DUSE_NAPTR -DWITH_XAVP -DMEM_JOIN_FREE -DF_MALLOC -DQ_MALLOC -DTLSF_MALLOC 
-DDBG_SR_MEMORY -DUSE_TLS -DTLS_HOOKS -DUSE_CORE_STATS -DSTATISTICS 
-DMALLOC_STATS -DUSE_SCTP -DFAST_LOCK -DADAPTIVE_WAIT 
-DADAPTIVE_WAIT_LOOPS=1024 -DCC_GCC_LIKE_ASM -DHAVE_GETHOSTBYNAME2 
-DHAVE_UNION_SEMUN -DHAVE_SCHED_YIELD -DHAVE_MSG_NOSIGNAL 
-DHAVE_MSGHDR_MSG_CONTROL -DHAVE_ALLOCA_H -DHAVE_TIMEGM 
-DHAVE_SCHED_SETSCHEDULER -DHAVE_IP_MREQN -DUSE_RAW_SOCKS -DHAVE_EPOLL 
-DHAVE_SIGIO_RT -DSIGINFO64_WORKAROUND -DUSE_FUTEX -DHAVE_SELECT  
-DMOD_NAME='"geoip"' -DMOD_NAMEID='geoip' -c geoip_pv.c -o geoip_pv.o -MMD -MP
gcc -shared -g -m64 -Wl,-O2 -Wl,-E  -Wl,-Bsymbolic-functions -Wl,-z,relro 
-pthread -rdynamic "-ldl" -Wl,-Bsymbolic-functions redis_client.o 
ndb_redis_mod.o  -lhiredis   -o ndb_redis.so
Makefile.defs defs skipped
Makefile.defs defs skipped
gcc -fPIC -DPIC -funroll-loops -Wcast-align -m64 -minline-all-stringops 
-falign-loops -ftree-vectorize -fno-strict-overflow -mtune=generic -pthread 
-DKSR_PTHREAD_MUTEX_SHARED -Wall -Wdate-time -D_FORTIFY_SOURCE=2 -g -O2 
-fstack-protector-strong -Wformat -Werror=format-security -DVERSION_NODATE   
-DNAME='"kamailio"' -DVERSION='"5.8.0-dev2"' -DARCH='"x86_64"' -DOS='linux_' 
-DOS_QUOTED='"linux"' -DCOMPILER='"gcc 5.4.0"' -D__CPU_x86_64 -D__OS_linux 
-DVERSIONVAL=5008000 -DCFG_DIR='"/etc/kamailio/"' 
-DSHARE_DIR='"/usr/share/kamailio/"' -DRUN_DIR='"/var/run/kamailio/"' 
-DPKG_MALLOC -DSHM_MMAP -DDNS_IP_HACK -DUSE_MCAST -DUSE_TCP -DDISABLE_NAGLE 
-DHAVE_RESOLV_RES -DUSE_DNS_CACHE -DUSE_DNS_FAILOVER -DUSE_DST_BLOCKLIST 
-DUSE_NAPTR -DWITH_XAVP -DMEM_JOIN_FREE -DF_MALLOC -DQ_MALLOC -DTLSF_MALLOC 
-DDBG_SR_MEMORY -DUSE_TLS -DTLS_HOOKS -DUSE_CORE_STATS -DSTATISTICS 
-DMALLOC_STATS -DUSE_SCTP -DFAST_LOCK -DADAPTIVE_WAIT 
-DADAPTIVE_WAIT_LOOPS=1024 -DCC_GCC_LIKE_ASM -DHAVE_GETHOSTBYNAME2 
-DHAVE_UNION_SEMUN -DHAVE_SCHED_YIELD -DHAVE_MSG_NOSIGNAL 
-DHAVE_MSGHDR_MSG_CONTROL -DHAVE_ALLOCA_H -DHAVE_TIMEGM 
-DHAVE_SCHED_SETSCHEDULER -DHAVE_IP_MREQN -DUSE_RAW_SOCKS -DHAVE_EPOLL 
-DHAVE_SIGIO_RT -DSIGINFO64_WORKAROUND -DUSE_FUTEX -DHAVE_SELECT  
-D_FILE_OFFSET_BITS=64 -I/usr/include/hiredis -DMOD_NAME='"topos_redis"' 
-DMOD_NAMEID='topos_redis' -c topos_redis_mod.c -o topos_redis_mod.o -MMD -MP
gcc -fPIC -DPIC -funroll-loops -Wcast-align -m64 -minline-all-stringops 
-falign-loops -ftree-vectorize -fno-strict-overflow -mtune=generic -pthread 
-DKSR_PTHREAD_MUTEX_SHARED -Wall -Wdate-time -D_FORTIFY_SOURCE=2 -g -O2 
-fstack-protector-strong -Wformat -Werror=format-security -DVERSION_NODATE   
-DNAME='"kamailio"' -DVERSION='"5.8.0-dev2"' -DARCH='"x86_64"' -DOS='linux_' 
-DOS_QUOTED='"linux"' -DCOMPILER='"gcc 5.4.0"' -D__CPU_x86_64 -D__OS_linux 
-DVERSIONVAL=5008000 -DCFG_DIR='"/etc/kamailio/"' 
-DSHARE_DIR='"/usr/share/kamailio/"' -DRUN_DIR='"/var/run/kamailio/"' 
-DPKG_MALLOC -DSHM_MMAP -DDNS_IP_HACK -DUSE_MCAST -DUSE_TCP -DDISABLE_NAGLE 
-DHAVE_RESOLV_RES -DUSE_DNS_CACHE -DUSE_DNS_FAILOVER -DUSE_DST_BLOCKLIST 
-DUSE_NAPTR -DWITH_XAVP -DMEM_JOIN_FREE -DF_MALLOC -DQ_MALLOC -DTLSF_MALLOC 
-DDBG_SR_MEMORY -DUSE_TLS -DTLS_HOOKS -DUSE_CORE_STATS -DSTATISTICS 
-DMALLOC_STATS -DUSE_SCTP -DFAST_LOCK -DADAPTIVE_WAIT 
-DADAPTIVE_WAIT_LOOPS=1024 -DCC_GCC_LIKE_ASM -DHAVE_GETHOSTBYNAME2 
-DHAVE_UNION_SEMUN -DHAVE_SCHED_YIELD -DHAVE_MSG_NOSIGNAL 
-DHAVE_MSGHDR_MSG_CONTROL -DHAVE_ALLOCA_H -DHAVE_TIMEGM 
-DHAVE_SCHED_SETSCHEDULER -DHAVE_IP_MREQN -DUSE_RAW_SOCKS -DHAVE_EPOLL 
-DHAVE_SIGIO_RT -DSIGINFO64_WORKAROUND -DUSE_FUTEX -DHAVE_SELECT  
-D_FILE_OFFSET_BITS=64 -I/usr/include/hiredis -DMOD_NAME='"topos_redis"' 
-DMOD_NAMEID='topos_redis' -c topos_redis_storage.c -o topos_redis_storage.o 
-MMD -MP
gcc -fPIC -DPIC -funroll-loops -Wcast-align -m64 -minline-all-stringops 
-falign-loops -ftree-vectorize -fno-strict-overflow -mtune=generic -pthread 
-DKSR_PTHREAD_MUTEX_SHARED -Wall -Wdate-time 

[sr-dev] git:master:aef9cec3: core: avoid C99 error

2024-01-05 Thread Morten Tryfoss via sr-dev
Module: kamailio
Branch: master
Commit: aef9cec329893f58b0c7df5aec2050b6c3cecc60
URL: 
https://github.com/kamailio/kamailio/commit/aef9cec329893f58b0c7df5aec2050b6c3cecc60

Author: Morten Tryfoss 
Committer: Morten Tryfoss 
Date: 2024-01-04T13:26:26+01:00

core: avoid C99 error

---

Modified: src/core/parser/parse_diversion.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/aef9cec329893f58b0c7df5aec2050b6c3cecc60.diff
Patch: 
https://github.com/kamailio/kamailio/commit/aef9cec329893f58b0c7df5aec2050b6c3cecc60.patch

---

diff --git a/src/core/parser/parse_diversion.c 
b/src/core/parser/parse_diversion.c
index d60d0cbdd1f..92690296935 100644
--- a/src/core/parser/parse_diversion.c
+++ b/src/core/parser/parse_diversion.c
@@ -210,7 +210,8 @@ int parse_diversion_header(struct sip_msg *msg)
 
 int free_diversion_body(diversion_body_t *div_b)
 {
-   for(int i = 0; i < div_b->num_ids; i++) {
+   int i = 0;
+   for(i = 0; i < div_b->num_ids; i++) {
/* Free to_body pointer parameters */
if(div_b->id[i].param_lst) {
free_to_params(&(div_b->id[i]));

___
Kamailio (SER) - Development Mailing List
To unsubscribe send an email to sr-dev-le...@lists.kamailio.org