sr-dev May 2017

sr-dev@lists.kamailio.org

32 participants
298 discussions

git:master:bca06f79: ndb_redis: remove redis_execute command with no arguments
by Claudiu Boriga 02 May '17

02 May '17

Module: kamailio Branch: master Commit: bca06f793df150fe03e83ba8c893f68b4f9bbcfb URL: https://github.com/kamailio/kamailio/commit/bca06f793df150fe03e83ba8c893f68… Author: Claudiu Boriga <paul.boriga(a)1and1.ro> Committer: Claudiu Boriga <paul.boriga(a)1and1.ro> Date: 2017-05-02T15:10:55+03:00 ndb_redis: remove redis_execute command with no arguments --- Modified: src/modules/ndb_redis/doc/ndb_redis_admin.xml Modified: src/modules/ndb_redis/ndb_redis_mod.c Modified: src/modules/ndb_redis/redis_client.c Modified: src/modules/ndb_redis/redis_client.h --- Diff: https://github.com/kamailio/kamailio/commit/bca06f793df150fe03e83ba8c893f68… Patch: https://github.com/kamailio/kamailio/commit/bca06f793df150fe03e83ba8c893f68…

1 0

[kamailio/kamailio] Redis remove redis_execute() command with no arguments (#1112)
by Paul Claudiu Boriga 02 May '17

02 May '17

Hello, We realized that redis_execute should return a negative value in case of failure (just like redis_cmd), instead of always returning 1. This introduced a problem with the behavior when calling redis_execute without giving a server name. Since this function will loop through all the servers and execute the pipelined commands for each one, in case of failure the user cannot determine for what server the error occurred, by only returning a negative value. This is why we decided to remove the possibility to call redis_execute with no arguments, and have it loop through all defined servers. The user should only call redis_execute(srv_name) and check the return value for this. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1112 -- Commit Summary -- * ndb_redis: remove redis_execute command with no arguments * ndb_redis: make redis_execute() return negative value on error -- File Changes -- M src/modules/ndb_redis/doc/ndb_redis_admin.xml (10) M src/modules/ndb_redis/ndb_redis_mod.c (22) M src/modules/ndb_redis/redis_client.c (20) M src/modules/ndb_redis/redis_client.h (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1112.patch https://github.com/kamailio/kamailio/pull/1112.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/1112

2 2

[kamailio/kamailio] core: Added IP_FREEBIND flag durring socket initialization (#1113)
by sergey-safarov 02 May '17

02 May '17

Backport of commits: 1) 7036c0ea947d6450e6a23c2e5b2428c834352de0 2) ff3cf17ff3566cb08cb0a6b9f7f7d893cd908956 3) 66595f39a4ae419b054bb433f2451a69155d580f You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1113 -- Commit Summary -- * core: Added IP_FREEBIND flag durring socket initialization -- File Changes -- M cfg.lex (2) M cfg.y (3) M globals.h (1) M modules/sctp/sctp_server.c (15) M receive.c (2) M tcp_main.c (11) M udp_server.c (10) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1113.patch https://github.com/kamailio/kamailio/pull/1113.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/1113

2 2

on reducing the number of email messages
by Juha Heinanen 02 May '17

02 May '17

Why is all K related github messages also sent to sr-dev list? There is lots of duplicate traffic because of it if sr-dev member is also watching github notifications. Or is it so that sr-dev members are not supposed to subscribe to kamailio github notifications? -- Juha

2 2

libressl support
by Timo Teras 02 May '17

02 May '17

Hi, I'm trying to fix tls.so to work with libressl. And there was already one commit handling some of the api differences. The next problem is that libressl ships the memory allocator changing function, but it's a stub that just returns "not supported". It seems that kamailio module does not load unless it can make libssl use shm_alloc. I'm wondering if this is a hard requirement? Does the tls module share libtls allocated things amongs forks? Or this purely to track memory in kamailio framework? If it's hard dependency is there any way to workaround this libressl design decision? I think one option to try is to just implement the CRYPTO_* allocation function in kamailio main executable - they should then take preference over the libcrypto ones. Thoughts? Thanks, Timo

2 1

git:master:f0609372: uac: updates to fix compile time warnings
by Daniel-Constantin Mierla 02 May '17

02 May '17

Module: kamailio Branch: master Commit: f0609372d5804865ac41ba0530e84312f59f8a45 URL: https://github.com/kamailio/kamailio/commit/f0609372d5804865ac41ba0530e8431… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-05-02T15:11:42+02:00 uac: updates to fix compile time warnings --- Modified: src/modules/uac/uac.c --- Diff: https://github.com/kamailio/kamailio/commit/f0609372d5804865ac41ba0530e8431… Patch: https://github.com/kamailio/kamailio/commit/f0609372d5804865ac41ba0530e8431… --- diff --git a/src/modules/uac/uac.c b/src/modules/uac/uac.c index c8454c9..d5ac592 100644 --- a/src/modules/uac/uac.c +++ b/src/modules/uac/uac.c @@ -98,7 +98,6 @@ static int w_uac_auth(struct sip_msg* msg, char* str, char* str2); static int w_uac_reg_lookup(struct sip_msg* msg, char* src, char* dst); static int w_uac_reg_status(struct sip_msg* msg, char* src, char* dst); static int w_uac_reg_request_to(struct sip_msg* msg, char* src, char* mode_s); -static int fixup_replace_uri(void** param, int param_no); static int mod_init(void); static void mod_destroy(void); static int child_init(int rank); @@ -734,4 +733,4 @@ int mod_register(char *path, int *dlflags, void *p1, void *p2) { sr_kemi_modules_add(sr_kemi_uac_exports); return 0; -} \ No newline at end of file +}

1 0

git:master:15883fef: tm: fix warnings for compilation
by Daniel-Constantin Mierla 02 May '17

02 May '17

Module: kamailio Branch: master Commit: 15883fef8f5b7205bbf68b765e856dd887c42954 URL: https://github.com/kamailio/kamailio/commit/15883fef8f5b7205bbf68b765e856dd… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-05-02T15:11:23+02:00 tm: fix warnings for compilation --- Modified: src/modules/tm/tm.c --- Diff: https://github.com/kamailio/kamailio/commit/15883fef8f5b7205bbf68b765e856dd… Patch: https://github.com/kamailio/kamailio/commit/15883fef8f5b7205bbf68b765e856dd… --- diff --git a/src/modules/tm/tm.c b/src/modules/tm/tm.c index 90c41f7..7d3ab19 100644 --- a/src/modules/tm/tm.c +++ b/src/modules/tm/tm.c @@ -996,7 +996,6 @@ static int ki_t_lookup_request(struct sip_msg* msg) static int ki_t_lookup_cancel_flags(sip_msg_t* msg, int flags) { struct cell *ret; - int i=0; if (msg->REQ_METHOD==METHOD_CANCEL) { ret = t_lookupOriginalT( msg ); LM_DBG("lookup_original: t_lookupOriginalT returned: %p\n", ret);

1 0

git:master:17123659: textopsx: remove unused variables
by Daniel-Constantin Mierla 02 May '17

02 May '17

Module: kamailio Branch: master Commit: 171236590aab6a7dbf5a79fdcc110745af891e19 URL: https://github.com/kamailio/kamailio/commit/171236590aab6a7dbf5a79fdcc11074… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-05-02T15:10:53+02:00 textopsx: remove unused variables --- Modified: src/modules/textopsx/textopsx.c --- Diff: https://github.com/kamailio/kamailio/commit/171236590aab6a7dbf5a79fdcc11074… Patch: https://github.com/kamailio/kamailio/commit/171236590aab6a7dbf5a79fdcc11074… --- diff --git a/src/modules/textopsx/textopsx.c b/src/modules/textopsx/textopsx.c index 9b78ea5..7f99a39 100644 --- a/src/modules/textopsx/textopsx.c +++ b/src/modules/textopsx/textopsx.c @@ -314,8 +314,6 @@ static int change_reply_status_f( { int code; str reason; - struct lump *l; - char *ch; if(get_int_fparam(&code, msg, (fparam_t *)_code) || get_str_fparam(&reason, msg, (fparam_t *)_reason)) { @@ -2163,4 +2161,4 @@ int mod_register(char *path, int *dlflags, void *p1, void *p2) { sr_kemi_modules_add(sr_kemi_textopsx_exports); return 0; -} \ No newline at end of file +}

1 0

git:master:cf7ed18c: registrar: fix compilation warnings
by Daniel-Constantin Mierla 02 May '17

02 May '17

Module: kamailio Branch: master Commit: cf7ed18c74343ae928ec57a7b76835eedd202dcc URL: https://github.com/kamailio/kamailio/commit/cf7ed18c74343ae928ec57a7b76835e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-05-02T15:10:17+02:00 registrar: fix compilation warnings --- Modified: src/modules/registrar/regpv.c --- Diff: https://github.com/kamailio/kamailio/commit/cf7ed18c74343ae928ec57a7b76835e… Patch: https://github.com/kamailio/kamailio/commit/cf7ed18c74343ae928ec57a7b76835e… --- diff --git a/src/modules/registrar/regpv.c b/src/modules/registrar/regpv.c index aba0eee..35bdd74 100644 --- a/src/modules/registrar/regpv.c +++ b/src/modules/registrar/regpv.c @@ -427,7 +427,6 @@ int pv_fetch_contacts_helper(sip_msg_t* msg, udomain_t* dt, str* uri, ucontact_t* c0; regpv_profile_t *rpp; str aor = {0, 0}; - str u = {0, 0}; int res; int olen; int ilen;

1 0

git:master:8b210310: evrexec: fixed the compile time warnings
by Daniel-Constantin Mierla 02 May '17

02 May '17

Module: kamailio Branch: master Commit: 8b210310e31ad1083d4fa9a83910b104c77b0a01 URL: https://github.com/kamailio/kamailio/commit/8b210310e31ad1083d4fa9a83910b10… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-05-02T15:09:49+02:00 evrexec: fixed the compile time warnings --- Modified: src/modules/evrexec/evrexec_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/8b210310e31ad1083d4fa9a83910b10… Patch: https://github.com/kamailio/kamailio/commit/8b210310e31ad1083d4fa9a83910b10… --- diff --git a/src/modules/evrexec/evrexec_mod.c b/src/modules/evrexec/evrexec_mod.c index 5d45e6f..a91ab4d 100644 --- a/src/modules/evrexec/evrexec_mod.c +++ b/src/modules/evrexec/evrexec_mod.c @@ -183,7 +183,6 @@ int evrexec_param(modparam_t type, void *val) evrexec_task_t tmp; sr_kemi_eng_t *keng = NULL; str s; - char c; if(val==NULL) return -1;

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 May 2017