sr-dev

sr-dev@lists.kamailio.org

21 participants
45182 discussions

git:5.3:aa793407: permissions: fix crash if load_backends parameter is 1
by Daniel-Constantin Mierla 13 Aug '20

13 Aug '20

Module: kamailio Branch: 5.3 Commit: aa793407020874377ce7c4ea871aa591c2b4a39d URL: https://github.com/kamailio/kamailio/commit/aa793407020874377ce7c4ea871aa59… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-13T00:39:08+02:00 permissions: fix crash if load_backends parameter is 1 fixes #2151 (cherry picked from commit d7f4493da751802dc47a694d251d34a85a7b3bf1) --- Modified: src/modules/permissions/permissions.c --- Diff: https://github.com/kamailio/kamailio/commit/aa793407020874377ce7c4ea871aa59… Patch: https://github.com/kamailio/kamailio/commit/aa793407020874377ce7c4ea871aa59… --- diff --git a/src/modules/permissions/permissions.c b/src/modules/permissions/permissions.c index b69cf63922..f31cadfa23 100644 --- a/src/modules/permissions/permissions.c +++ b/src/modules/permissions/permissions.c @@ -648,8 +648,10 @@ static int mod_init(void) static int child_init(int rank) { - if (init_child_trusted(rank) == -1) - return -1; + if(_perm_load_backends&PERM_LOAD_TRUSTEDDB) { + if(init_child_trusted(rank) == -1) + return -1; + } return 0; }

1 0

git:master:99c6a68d: tls: added define condition on version functions
by Daniel-Constantin Mierla 13 Aug '20

13 Aug '20

Module: kamailio Branch: master Commit: 99c6a68dea60ca745dfdfa33085bb6acbcee846a URL: https://github.com/kamailio/kamailio/commit/99c6a68dea60ca745dfdfa33085bb6a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-13T07:19:28+02:00 tls: added define condition on version functions - libssl 1.1 variants are not in 1.0 - following the changes by 234b02236b3ad13cdaf5624d11c727ad7d804747 --- Modified: src/modules/tls/tls_domain.c Modified: src/modules/tls/tls_init.c --- Diff: https://github.com/kamailio/kamailio/commit/99c6a68dea60ca745dfdfa33085bb6a… Patch: https://github.com/kamailio/kamailio/commit/99c6a68dea60ca745dfdfa33085bb6a… --- diff --git a/src/modules/tls/tls_domain.c b/src/modules/tls/tls_domain.c index fc84033e68..2930bde2be 100644 --- a/src/modules/tls/tls_domain.c +++ b/src/modules/tls/tls_domain.c @@ -63,9 +63,11 @@ static void setup_ecdh(SSL_CTX *ctx) { EC_KEY *ecdh; - if (OpenSSL_version_num() < 0x1000005fL) { +#if OPENSSL_VERSION_NUMBER < 0x010100000L + if (SSLeay() < 0x1000005fL) { return; } +#endif ecdh = EC_KEY_new_by_curve_name(NID_X9_62_prime256v1); SSL_CTX_set_options(ctx, SSL_OP_SINGLE_ECDH_USE); diff --git a/src/modules/tls/tls_init.c b/src/modules/tls/tls_init.c index 351ed4187e..f10d0c0344 100644 --- a/src/modules/tls/tls_init.c +++ b/src/modules/tls/tls_init.c @@ -645,6 +645,7 @@ int tls_h_mod_init_f(void) { /*struct socket_info* si;*/ long ssl_version; + const char *ssl_version_txt; #if OPENSSL_VERSION_NUMBER < 0x010100000L && !defined(LIBRESSL_VERSION_NUMBER) int lib_kerberos; int lib_zlib; @@ -667,7 +668,15 @@ int tls_h_mod_init_f(void) #if OPENSSL_VERSION_NUMBER < 0x00907000L LM_WARN("You are using an old version of OpenSSL (< 0.9.7). Upgrade!\n"); #endif + +#if OPENSSL_VERSION_NUMBER < 0x010100000L + ssl_version=SSLeay(); + ssl_version_txt=SSLeay_version(SSLEAY_VERSION); +#else ssl_version=OpenSSL_version_num(); + ssl_version_txt=OpenSSL_version(OPENSSL_VERSION); +#endif + /* check if version have the same major minor and fix level * (e.g. 0.9.8a & 0.9.8c are ok, but 0.9.8 and 0.9.9x are not) * - values is represented as 0xMMNNFFPPS: major minor fix patch status @@ -679,7 +688,7 @@ int tls_h_mod_init_f(void) " compiled \"%s\" (0x%08lx).\n" " Please make sure a compatible version is used" " (tls_force_run in kamailio.cfg will override this check)\n", - OpenSSL_version(OPENSSL_VERSION), ssl_version, + ssl_version_txt, ssl_version, OPENSSL_VERSION_TEXT, (long)OPENSSL_VERSION_NUMBER); if (cfg_get(tls, tls_cfg, force_run)) LM_WARN("tls_force_run turned on, ignoring "

1 0

git:master:66c25bdd: etc/kamailio.cfg: set load_backends to 1 for permissions module
by Daniel-Constantin Mierla 12 Aug '20

12 Aug '20

Module: kamailio Branch: master Commit: 66c25bddcfe620794ba23d816ac55beedda494fb URL: https://github.com/kamailio/kamailio/commit/66c25bddcfe620794ba23d816ac55be… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-12T21:43:08+02:00 etc/kamailio.cfg: set load_backends to 1 for permissions module - the config uses only address table --- Modified: etc/kamailio.cfg --- Diff: https://github.com/kamailio/kamailio/commit/66c25bddcfe620794ba23d816ac55be… Patch: https://github.com/kamailio/kamailio/commit/66c25bddcfe620794ba23d816ac55be… --- diff --git a/etc/kamailio.cfg b/etc/kamailio.cfg index fed81be1df..e31d112839 100644 --- a/etc/kamailio.cfg +++ b/etc/kamailio.cfg @@ -420,7 +420,7 @@ modparam("auth_db", "use_domain", MULTIDOMAIN) # ----- permissions params ----- #!ifdef WITH_IPAUTH modparam("permissions", "db_url", DBURL) -modparam("permissions", "db_mode", 1) +modparam("permissions", "load_backends", 1) #!endif #!endif

1 0

git:master:bd0a236a: modules: readme files regenerated - ims_ipsec_pcscf ... [skip ci]
by Kamailio Dev 12 Aug '20

12 Aug '20

Module: kamailio Branch: master Commit: bd0a236a08f282916ea14a510b2fff82c062b3db URL: https://github.com/kamailio/kamailio/commit/bd0a236a08f282916ea14a510b2fff8… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2020-08-12T20:46:12+02:00 modules: readme files regenerated - ims_ipsec_pcscf ... [skip ci] --- Modified: src/modules/ims_ipsec_pcscf/README --- Diff: https://github.com/kamailio/kamailio/commit/bd0a236a08f282916ea14a510b2fff8… Patch: https://github.com/kamailio/kamailio/commit/bd0a236a08f282916ea14a510b2fff8… --- diff --git a/src/modules/ims_ipsec_pcscf/README b/src/modules/ims_ipsec_pcscf/README index 1cf499fb4b..e80d339f5b 100644 --- a/src/modules/ims_ipsec_pcscf/README +++ b/src/modules/ims_ipsec_pcscf/README @@ -268,9 +268,10 @@ ipsec_create("location"); Meaning of the parameters is as follows: * domain - Logical domain within the registrar. If a database is used then this must be name of the table which stores the contacts. - flags - Set send force socket for request messages. If 1 - set - force socket for request messages. Useful for ipsec and TCP. This - is an optional parameter, default value - 0. + flags - bitwise flag: 0x01 - set force socket for request messages. + Useful for ipsec and TCP. 0x02 - reverse search for a contact in + the memory. Useful when contact alias is disabled. This is an + optional parameter, default value - 0. Example 1.10. ipsec_forward ...

1 0

[kamailio/kamailio] Extend get pcontact (#2426)
by alexyosifov 12 Aug '20

12 Aug '20

#### 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/2426 -- Commit Summary -- * ims_usrloc_pcscf: added a reverse searchin get_pcontact * ims_registrar_pcscf: update_contacts get received from req * ims_qos: extend get_pcontact with the new parameter. * ims_ipsec_pcscf: extend get_pcontact with the new parameter. -- File Changes -- M src/modules/ims_ipsec_pcscf/cmd.c (7) M src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf_admin.xml (5) M src/modules/ims_qos/cdpeventprocessor.c (2) M src/modules/ims_qos/ims_qos_mod.c (2) M src/modules/ims_qos/rx_aar.c (2) M src/modules/ims_registrar_pcscf/notify.c (2) M src/modules/ims_registrar_pcscf/save.c (20) M src/modules/ims_registrar_pcscf/service_routes.c (8) M src/modules/ims_usrloc_pcscf/udomain.c (20) M src/modules/ims_usrloc_pcscf/udomain.h (2) M src/modules/ims_usrloc_pcscf/usrloc.h (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2426.patch https://github.com/kamailio/kamailio/pull/2426.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2426

2 6

git:master:e10dc0db: ims_ipsec_pcscf: extend get_pcontact with the new parameter.
by Henning Westerholt 12 Aug '20

12 Aug '20

Module: kamailio Branch: master Commit: e10dc0db1b890d6ae2067f2fbff803ba8f816b68 URL: https://github.com/kamailio/kamailio/commit/e10dc0db1b890d6ae2067f2fbff803b… Author: Aleksandar Yosifov <alexyosifov(a)gmail.com> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-08-12T20:44:08+02:00 ims_ipsec_pcscf: extend get_pcontact with the new parameter. - added a new bitwise value for _cflags in ipsec_forward. - performing a reverse search for a contact in ipsec_forward if the new flag IPSEC_REVERSE_SEARCH is set. --- Modified: src/modules/ims_ipsec_pcscf/cmd.c Modified: src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/e10dc0db1b890d6ae2067f2fbff803b… Patch: https://github.com/kamailio/kamailio/commit/e10dc0db1b890d6ae2067f2fbff803b… --- diff --git a/src/modules/ims_ipsec_pcscf/cmd.c b/src/modules/ims_ipsec_pcscf/cmd.c index 16a62f8e6e..e63e9315de 100644 --- a/src/modules/ims_ipsec_pcscf/cmd.c +++ b/src/modules/ims_ipsec_pcscf/cmd.c @@ -76,6 +76,7 @@ extern usrloc_api_t ul; extern struct tm_binds tmb; #define IPSEC_SEND_FORCE_SOCKET 0x01 /* if set - set send force socket for request messages */ +#define IPSEC_REVERSE_SEARCH 0x02 /* if set - start seraching from the last element */ int bind_ipsec_pcscf(ipsec_pcscf_api_t* api) { if(!api){ @@ -663,7 +664,7 @@ int ipsec_create(struct sip_msg* m, udomain_t* d) ul.lock_udomain(d, &ci.via_host, ci.via_port, ci.via_prot); - if (ul.get_pcontact(d, &ci, &pcontact) != 0) { + if (ul.get_pcontact(d, &ci, &pcontact, 0) != 0) { LM_ERR("Contact doesn't exist\n"); goto cleanup; } @@ -802,7 +803,7 @@ int ipsec_forward(struct sip_msg* m, udomain_t* d, int _cflags) ul.lock_udomain(d, &ci.via_host, ci.via_port, ci.via_prot); - if (ul.get_pcontact(d, &ci, &pcontact) != 0) { + if (ul.get_pcontact(d, &ci, &pcontact, _cflags & IPSEC_REVERSE_SEARCH) != 0) { LM_ERR("Contact doesn't exist\n"); goto cleanup; } @@ -946,7 +947,7 @@ int ipsec_destroy(struct sip_msg* m, udomain_t* d) ul.lock_udomain(d, &ci.via_host, ci.via_port, ci.via_prot); - if (ul.get_pcontact(d, &ci, &pcontact) != 0) { + if (ul.get_pcontact(d, &ci, &pcontact, 0) != 0) { LM_ERR("Contact doesn't exist\n"); goto cleanup; } diff --git a/src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf_admin.xml b/src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf_admin.xml index 95b44f2c90..3d2a19efc9 100644 --- a/src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf_admin.xml +++ b/src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf_admin.xml @@ -243,8 +243,9 @@ ipsec_create("location"); stores the contacts. </para> <para> - <emphasis>flags</emphasis> - Set send force socket for request messages. - If 1 - set force socket for request messages. Useful for ipsec and TCP. + <emphasis>flags</emphasis> - bitwise flag: + 0x01 - set force socket for request messages. Useful for ipsec and TCP. + 0x02 - reverse search for a contact in the memory. Useful when contact alias is disabled. This is an optional parameter, default value - 0. </para> </listitem>

1 0

git:master:2e112825: ims_registrar_pcscf: fix extraction of received host in save()
by Henning Westerholt 12 Aug '20

12 Aug '20

Module: kamailio Branch: master Commit: 2e11282502dce2b222d2884f308edf31fdd42d05 URL: https://github.com/kamailio/kamailio/commit/2e11282502dce2b222d2884f308edf3… Author: alexyosifov <47529095+alexyosifov(a)users.noreply.github.com> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-08-12T20:44:08+02:00 ims_registrar_pcscf: fix extraction of received host in save() --- Modified: src/modules/ims_registrar_pcscf/save.c --- Diff: https://github.com/kamailio/kamailio/commit/2e11282502dce2b222d2884f308edf3… Patch: https://github.com/kamailio/kamailio/commit/2e11282502dce2b222d2884f308edf3… --- diff --git a/src/modules/ims_registrar_pcscf/save.c b/src/modules/ims_registrar_pcscf/save.c index a16ba48091..9432574005 100644 --- a/src/modules/ims_registrar_pcscf/save.c +++ b/src/modules/ims_registrar_pcscf/save.c @@ -128,6 +128,7 @@ static inline int update_contacts(struct sip_msg *req,struct sip_msg *rpl, udoma char *alias_start, *p, *port_s, *proto_s; char portbuf[5]; str alias_s; + char srcip[50]; pcscf_act_time(); local_time_now = time_now; @@ -197,13 +198,7 @@ static inline int update_contacts(struct sip_msg *req,struct sip_msg *rpl, udoma } } } else { - char* srcip = NULL; - if((srcip = pkg_malloc(50)) == NULL) { - LM_ERR("Error allocating memory for source IP address\n"); - return -1; - } - - ci.received_host.len = ip_addr2sbuf(&req->rcv.src_ip, srcip, 50); + ci.received_host.len = ip_addr2sbuf(&req->rcv.src_ip, srcip, sizeof(srcip)); ci.received_host.s = srcip; ci.received_port = req->rcv.src_port; ci.received_proto = req->rcv.proto;

1 0

git:master:89833d74: ims_ipsec_pcscf: fixed spelling issue
by Henning Westerholt 12 Aug '20

12 Aug '20

Module: kamailio Branch: master Commit: 89833d74d6c978512495e7e03b21e9f50e9f7af8 URL: https://github.com/kamailio/kamailio/commit/89833d74d6c978512495e7e03b21e9f… Author: alexyosifov <47529095+alexyosifov(a)users.noreply.github.com> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-08-12T20:44:08+02:00 ims_ipsec_pcscf: fixed spelling issue --- Modified: src/modules/ims_ipsec_pcscf/cmd.c --- Diff: https://github.com/kamailio/kamailio/commit/89833d74d6c978512495e7e03b21e9f… Patch: https://github.com/kamailio/kamailio/commit/89833d74d6c978512495e7e03b21e9f… --- diff --git a/src/modules/ims_ipsec_pcscf/cmd.c b/src/modules/ims_ipsec_pcscf/cmd.c index e63e9315de..f283528b1a 100644 --- a/src/modules/ims_ipsec_pcscf/cmd.c +++ b/src/modules/ims_ipsec_pcscf/cmd.c @@ -76,7 +76,7 @@ extern usrloc_api_t ul; extern struct tm_binds tmb; #define IPSEC_SEND_FORCE_SOCKET 0x01 /* if set - set send force socket for request messages */ -#define IPSEC_REVERSE_SEARCH 0x02 /* if set - start seraching from the last element */ +#define IPSEC_REVERSE_SEARCH 0x02 /* if set - start searching from the last element */ int bind_ipsec_pcscf(ipsec_pcscf_api_t* api) { if(!api){

1 0

git:master:3ee011fa: ims_registrar_pcscf: update_contacts get received from req
by Henning Westerholt 12 Aug '20

12 Aug '20

Module: kamailio Branch: master Commit: 3ee011fa21e2d24f0c9d110ec09ccb731c5859c7 URL: https://github.com/kamailio/kamailio/commit/3ee011fa21e2d24f0c9d110ec09ccb7… Author: Aleksandar Yosifov <alexyosifov(a)gmail.com> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-08-12T20:44:08+02:00 ims_registrar_pcscf: update_contacts get received from req - update_contacts - if the reply doesn't include contact alias get received info from the request. - extend get_pcontact eith the new parameter. --- Modified: src/modules/ims_registrar_pcscf/notify.c Modified: src/modules/ims_registrar_pcscf/save.c Modified: src/modules/ims_registrar_pcscf/service_routes.c --- Diff: https://github.com/kamailio/kamailio/commit/3ee011fa21e2d24f0c9d110ec09ccb7… Patch: https://github.com/kamailio/kamailio/commit/3ee011fa21e2d24f0c9d110ec09ccb7…

1 0

git:master:45ebf8bd: ims_qos: extend get_pcontact with the new parameter.
by Henning Westerholt 12 Aug '20

12 Aug '20

Module: kamailio Branch: master Commit: 45ebf8bd87c7ef23aed0461808cad25d8127db7f URL: https://github.com/kamailio/kamailio/commit/45ebf8bd87c7ef23aed0461808cad25… Author: Aleksandar Yosifov <alexyosifov(a)gmail.com> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-08-12T20:44:08+02:00 ims_qos: extend get_pcontact with the new parameter. --- Modified: src/modules/ims_qos/cdpeventprocessor.c Modified: src/modules/ims_qos/ims_qos_mod.c Modified: src/modules/ims_qos/rx_aar.c --- Diff: https://github.com/kamailio/kamailio/commit/45ebf8bd87c7ef23aed0461808cad25… Patch: https://github.com/kamailio/kamailio/commit/45ebf8bd87c7ef23aed0461808cad25… --- diff --git a/src/modules/ims_qos/cdpeventprocessor.c b/src/modules/ims_qos/cdpeventprocessor.c index 8c31ef05ac..352c39396a 100644 --- a/src/modules/ims_qos/cdpeventprocessor.c +++ b/src/modules/ims_qos/cdpeventprocessor.c @@ -275,7 +275,7 @@ void cdp_cb_event_process() contact_info.aor = p_session_data->registration_aor; contact_info.reg_state = PCONTACT_ANY; - if (ul.get_pcontact(domain, &contact_info, &pcontact) != 0) { + if (ul.get_pcontact(domain, &contact_info, &pcontact, 0) != 0) { LM_DBG("no contact found for terminated Rx reg session..... ignoring\n"); } else { LM_DBG("Updating contact [%.*s] after Rx reg session terminated, setting state to PCONTACT_DEREG_PENDING_PUBLISH\n", pcontact->aor.len, pcontact->aor.s); diff --git a/src/modules/ims_qos/ims_qos_mod.c b/src/modules/ims_qos/ims_qos_mod.c index e58dd2c140..5fa584bcd7 100644 --- a/src/modules/ims_qos/ims_qos_mod.c +++ b/src/modules/ims_qos/ims_qos_mod.c @@ -1314,7 +1314,7 @@ static int w_rx_aar_register(struct sip_msg *msg, char* route, char* str1, char* contact_info.received_host.len = 0; contact_info.reg_state = PCONTACT_ANY; //search for any state - if (ul.get_pcontact(domain_t, &contact_info, &pcontact) != 0) { + if (ul.get_pcontact(domain_t, &contact_info, &pcontact, 0) != 0) { LM_ERR("This contact does not exist in PCSCF usrloc - error in cfg file\n"); ul.unlock_udomain(domain_t, &vb->host, vb->port, vb->proto); lock_release(saved_t_data->lock); diff --git a/src/modules/ims_qos/rx_aar.c b/src/modules/ims_qos/rx_aar.c index b1e8df51b8..987b5c22bd 100644 --- a/src/modules/ims_qos/rx_aar.c +++ b/src/modules/ims_qos/rx_aar.c @@ -321,7 +321,7 @@ void async_aar_reg_callback(int is_timeout, void *param, AAAMessage *aaa, long e contact_info.via_prot = local_data->via_proto; contact_info.reg_state = PCONTACT_ANY; - if (ul.get_pcontact(domain_t, &contact_info, &pcontact) != 0) { + if (ul.get_pcontact(domain_t, &contact_info, &pcontact, 0) != 0) { LM_ERR("Shouldn't get here, can't find contact....\n"); ul.unlock_udomain(domain_t, &local_data->via_host, local_data->via_port, local_data->via_proto); goto error;

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev