sr-dev

sr-dev@lists.kamailio.org

22 participants
45186 discussions

git:5.4:b1ce252f: keepalive: removed function name from log messages
by Daniel-Constantin Mierla 31 Aug '20

31 Aug '20

Module: kamailio Branch: 5.4 Commit: b1ce252f8090f66de290cec95dbb6e1762ee6bdf URL: https://github.com/kamailio/kamailio/commit/b1ce252f8090f66de290cec95dbb6e1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-31T12:16:05+02:00 keepalive: removed function name from log messages (cherry picked from commit 17d6a88090e2a67f5c0b06e6aae76d629595b37e) --- Modified: src/modules/keepalive/keepalive_core.c --- Diff: https://github.com/kamailio/kamailio/commit/b1ce252f8090f66de290cec95dbb6e1… Patch: https://github.com/kamailio/kamailio/commit/b1ce252f8090f66de290cec95dbb6e1… --- diff --git a/src/modules/keepalive/keepalive_core.c b/src/modules/keepalive/keepalive_core.c index 36cf81aef1..00f40c46bf 100644 --- a/src/modules/keepalive/keepalive_core.c +++ b/src/modules/keepalive/keepalive_core.c @@ -61,7 +61,7 @@ ticks_t ka_check_timer(ticks_t ticks, struct timer_ln* tl, void* param) ka_dest = (ka_dest_t *)param; - LM_DBG("ka_check_timer dest:%.*s\n", ka_dest->uri.len, ka_dest->uri.s); + LM_DBG("dest: %.*s\n", ka_dest->uri.len, ka_dest->uri.s); if(ka_counter_del > 0 && ka_dest->counter > ka_counter_del) { return (ticks_t)(0); /* stops the timer */ @@ -104,7 +104,7 @@ static void ka_options_callback( uri.s = t->to.s + 5; uri.len = t->to.len - 8; - LM_DBG("OPTIONS-Request was finished with code %d (to %.*s)\n", ps->code, + LM_DBG("OPTIONS request was finished with code %d (to %.*s)\n", ps->code, ka_dest->uri.len, ka_dest->uri.s); //uri.len, uri.s); @@ -149,7 +149,7 @@ static void ka_run_route(sip_msg_t *msg, str *uri, char *route) return; } - LM_DBG("ka_run_route event_route[%s]\n", route); + LM_DBG("run event_route[%s]\n", route); rt = route_get(&event_rt, route); if(rt < 0 || event_rt.rlist[rt] == NULL) {

1 0

git:master:8aeaee0f: pv: new transformation {uri.saor}
by Daniel-Constantin Mierla 31 Aug '20

31 Aug '20

Module: kamailio Branch: master Commit: 8aeaee0fda4013e1fe0477ce1dcb31738dc18aea URL: https://github.com/kamailio/kamailio/commit/8aeaee0fda4013e1fe0477ce1dcb317… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-31T09:37:45+02:00 pv: new transformation {uri.saor} - return sip aor (sip:[user@]domain) --- Modified: src/modules/pv/pv_trans.c Modified: src/modules/pv/pv_trans.h --- Diff: https://github.com/kamailio/kamailio/commit/8aeaee0fda4013e1fe0477ce1dcb317… Patch: https://github.com/kamailio/kamailio/commit/8aeaee0fda4013e1fe0477ce1dcb317… --- diff --git a/src/modules/pv/pv_trans.c b/src/modules/pv/pv_trans.c index 91e9bca516..4de9849165 100644 --- a/src/modules/pv/pv_trans.c +++ b/src/modules/pv/pv_trans.c @@ -1409,6 +1409,7 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, val->rs = (_tr_parsed_uri.passwd.s)?_tr_parsed_uri.passwd:_tr_empty; break; case TR_URI_DURI: + case TR_URI_SAOR: case TR_URI_SURI: if(_tr_uri.len >= TR_BUFFER_SIZE) { LM_WARN("uri too long [%.*s] (%d)\n", @@ -1436,7 +1437,8 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, memcpy(_tr_buffer, sv.s, sv.len); sv.s = _tr_buffer; sv.len++; - if(subtype == TR_URI_SURI && _tr_parsed_uri.user.len > 0) { + if((_tr_parsed_uri.user.len > 0) + && (subtype != TR_URI_DURI)) { memcpy(sv.s + sv.len, _tr_parsed_uri.user.s, _tr_parsed_uri.user.len); sv.len += _tr_parsed_uri.user.len; @@ -1448,19 +1450,21 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, _tr_parsed_uri.host.len); sv.len += _tr_parsed_uri.host.len; } - if(_tr_parsed_uri.port.len > 0) { - sv.s[sv.len] = ':'; - sv.len++; - memcpy(sv.s + sv.len, _tr_parsed_uri.port.s, - _tr_parsed_uri.port.len); - sv.len += _tr_parsed_uri.port.len; - } - if(_tr_parsed_uri.transport_val.len > 0) { - memcpy(sv.s + sv.len, ";transport=", 11); - sv.len += 11; - memcpy(sv.s + sv.len, _tr_parsed_uri.transport_val.s, - _tr_parsed_uri.transport_val.len); - sv.len += _tr_parsed_uri.transport_val.len; + if(subtype != TR_URI_SAOR) { + if(_tr_parsed_uri.port.len > 0) { + sv.s[sv.len] = ':'; + sv.len++; + memcpy(sv.s + sv.len, _tr_parsed_uri.port.s, + _tr_parsed_uri.port.len); + sv.len += _tr_parsed_uri.port.len; + } + if(_tr_parsed_uri.transport_val.len > 0) { + memcpy(sv.s + sv.len, ";transport=", 11); + sv.len += 11; + memcpy(sv.s + sv.len, _tr_parsed_uri.transport_val.s, + _tr_parsed_uri.transport_val.len); + sv.len += _tr_parsed_uri.transport_val.len; + } } sv.s[sv.len] = '\0'; val->rs = sv; @@ -2845,6 +2849,9 @@ char* tr_parse_uri(str* in, trans_t *t) } else if(name.len==4 && strncasecmp(name.s, "port", 4)==0) { t->subtype = TR_URI_PORT; goto done; + } else if(name.len==4 && strncasecmp(name.s, "saor", 4)==0) { + t->subtype = TR_URI_SAOR; + goto done; } else if(name.len==4 && strncasecmp(name.s, "suri", 4)==0) { t->subtype = TR_URI_SURI; goto done; diff --git a/src/modules/pv/pv_trans.h b/src/modules/pv/pv_trans.h index d15df89057..c587464bed 100644 --- a/src/modules/pv/pv_trans.h +++ b/src/modules/pv/pv_trans.h @@ -50,7 +50,8 @@ enum _tr_uri_subtype { TR_URI_NONE=0, TR_URI_USER, TR_URI_HOST, TR_URI_PASSWD, TR_URI_PORT, TR_URI_PARAMS, TR_URI_PARAM, TR_URI_HEADERS, TR_URI_TRANSPORT, TR_URI_TTL, TR_URI_UPARAM, TR_URI_MADDR, TR_URI_METHOD, TR_URI_LR, - TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_DURI, TR_URI_SURI + TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_SAOR, TR_URI_DURI, + TR_URI_SURI }; enum _tr_param_subtype { TR_PL_NONE=0, TR_PL_VALUE, TR_PL_VALUEAT, TR_PL_NAME, TR_PL_COUNT

1 0

git:master:17d6a880: keepalive: removed function name from log messages
by Daniel-Constantin Mierla 31 Aug '20

31 Aug '20

Module: kamailio Branch: master Commit: 17d6a88090e2a67f5c0b06e6aae76d629595b37e URL: https://github.com/kamailio/kamailio/commit/17d6a88090e2a67f5c0b06e6aae76d6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-31T12:15:00+02:00 keepalive: removed function name from log messages --- Modified: src/modules/keepalive/keepalive_core.c --- Diff: https://github.com/kamailio/kamailio/commit/17d6a88090e2a67f5c0b06e6aae76d6… Patch: https://github.com/kamailio/kamailio/commit/17d6a88090e2a67f5c0b06e6aae76d6… --- diff --git a/src/modules/keepalive/keepalive_core.c b/src/modules/keepalive/keepalive_core.c index 36cf81aef1..00f40c46bf 100644 --- a/src/modules/keepalive/keepalive_core.c +++ b/src/modules/keepalive/keepalive_core.c @@ -61,7 +61,7 @@ ticks_t ka_check_timer(ticks_t ticks, struct timer_ln* tl, void* param) ka_dest = (ka_dest_t *)param; - LM_DBG("ka_check_timer dest:%.*s\n", ka_dest->uri.len, ka_dest->uri.s); + LM_DBG("dest: %.*s\n", ka_dest->uri.len, ka_dest->uri.s); if(ka_counter_del > 0 && ka_dest->counter > ka_counter_del) { return (ticks_t)(0); /* stops the timer */ @@ -104,7 +104,7 @@ static void ka_options_callback( uri.s = t->to.s + 5; uri.len = t->to.len - 8; - LM_DBG("OPTIONS-Request was finished with code %d (to %.*s)\n", ps->code, + LM_DBG("OPTIONS request was finished with code %d (to %.*s)\n", ps->code, ka_dest->uri.len, ka_dest->uri.s); //uri.len, uri.s); @@ -149,7 +149,7 @@ static void ka_run_route(sip_msg_t *msg, str *uri, char *route) return; } - LM_DBG("ka_run_route event_route[%s]\n", route); + LM_DBG("run event_route[%s]\n", route); rt = route_get(&event_rt, route); if(rt < 0 || event_rt.rlist[rt] == NULL) {

1 0

git:5.4:e9baba91: tls: proper ifdef on libssl version for tls_init_locks()
by Daniel-Constantin Mierla 30 Aug '20

30 Aug '20

Module: kamailio Branch: 5.4 Commit: e9baba91c6c7e3d1817e8a07bbabbce3a8efbe79 URL: https://github.com/kamailio/kamailio/commit/e9baba91c6c7e3d1817e8a07bbabbce… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-30T18:05:43+02:00 tls: proper ifdef on libssl version for tls_init_locks() - avoid compile warning on unused label (cherry picked from commit 60bd3aa284a5d9eb117a9763aa590aecf276ba16) --- Modified: src/modules/tls/tls_locking.c --- Diff: https://github.com/kamailio/kamailio/commit/e9baba91c6c7e3d1817e8a07bbabbce… Patch: https://github.com/kamailio/kamailio/commit/e9baba91c6c7e3d1817e8a07bbabbce… --- diff --git a/src/modules/tls/tls_locking.c b/src/modules/tls/tls_locking.c index 3eb6f033a5..21078e82d9 100644 --- a/src/modules/tls/tls_locking.c +++ b/src/modules/tls/tls_locking.c @@ -140,7 +140,7 @@ unsigned long sr_ssl_id_f() /* returns -1 on error, 0 on success */ int tls_init_locks() { -/* OpenSSL is thread-safe since 1.1.0 */ +/* OpenSSL is no longer supporting to set locking callbacks since 1.1.0 */ #if OPENSSL_VERSION_NUMBER < 0x10100000L /* init "static" tls locks */ n_static_locks=CRYPTO_num_locks(); @@ -185,10 +185,13 @@ int tls_init_locks() * (only atomic_inc), fallback to the default use-locks mode * CRYPTO_set_add_lock_callback(atomic_add_f); */ -#endif return 0; error: tls_destroy_locks(); return -1; + +#else + return 0; +#endif }

1 0

git:master:60bd3aa2: tls: proper ifdef on libssl version for tls_init_locks()
by Daniel-Constantin Mierla 30 Aug '20

30 Aug '20

Module: kamailio Branch: master Commit: 60bd3aa284a5d9eb117a9763aa590aecf276ba16 URL: https://github.com/kamailio/kamailio/commit/60bd3aa284a5d9eb117a9763aa590ae… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-30T18:04:04+02:00 tls: proper ifdef on libssl version for tls_init_locks() - avoid compile warning on unused label --- Modified: src/modules/tls/tls_locking.c --- Diff: https://github.com/kamailio/kamailio/commit/60bd3aa284a5d9eb117a9763aa590ae… Patch: https://github.com/kamailio/kamailio/commit/60bd3aa284a5d9eb117a9763aa590ae… --- diff --git a/src/modules/tls/tls_locking.c b/src/modules/tls/tls_locking.c index 3eb6f033a5..21078e82d9 100644 --- a/src/modules/tls/tls_locking.c +++ b/src/modules/tls/tls_locking.c @@ -140,7 +140,7 @@ unsigned long sr_ssl_id_f() /* returns -1 on error, 0 on success */ int tls_init_locks() { -/* OpenSSL is thread-safe since 1.1.0 */ +/* OpenSSL is no longer supporting to set locking callbacks since 1.1.0 */ #if OPENSSL_VERSION_NUMBER < 0x10100000L /* init "static" tls locks */ n_static_locks=CRYPTO_num_locks(); @@ -185,10 +185,13 @@ int tls_init_locks() * (only atomic_inc), fallback to the default use-locks mode * CRYPTO_set_add_lock_callback(atomic_add_f); */ -#endif return 0; error: tls_destroy_locks(); return -1; + +#else + return 0; +#endif }

1 0

git:master:564940f5: pv: new transformation {uri.duri}
by Daniel-Constantin Mierla 29 Aug '20

29 Aug '20

Module: kamailio Branch: master Commit: 564940f5ef923df7935a7b737a70e25c0d5f6a7b URL: https://github.com/kamailio/kamailio/commit/564940f5ef923df7935a7b737a70e25… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-29T19:45:20+02:00 pv: new transformation {uri.duri} - return uri needed for destination address (e.g., sip:host:port;transport=tcp) --- Modified: src/modules/pv/pv_trans.c Modified: src/modules/pv/pv_trans.h --- Diff: https://github.com/kamailio/kamailio/commit/564940f5ef923df7935a7b737a70e25… Patch: https://github.com/kamailio/kamailio/commit/564940f5ef923df7935a7b737a70e25… --- diff --git a/src/modules/pv/pv_trans.c b/src/modules/pv/pv_trans.c index b9eb291b99..91e9bca516 100644 --- a/src/modules/pv/pv_trans.c +++ b/src/modules/pv/pv_trans.c @@ -1408,6 +1408,7 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, case TR_URI_PASSWD: val->rs = (_tr_parsed_uri.passwd.s)?_tr_parsed_uri.passwd:_tr_empty; break; + case TR_URI_DURI: case TR_URI_SURI: if(_tr_uri.len >= TR_BUFFER_SIZE) { LM_WARN("uri too long [%.*s] (%d)\n", @@ -1435,7 +1436,7 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, memcpy(_tr_buffer, sv.s, sv.len); sv.s = _tr_buffer; sv.len++; - if(_tr_parsed_uri.user.len > 0) { + if(subtype == TR_URI_SURI && _tr_parsed_uri.user.len > 0) { memcpy(sv.s + sv.len, _tr_parsed_uri.user.s, _tr_parsed_uri.user.len); sv.len += _tr_parsed_uri.user.len; @@ -2847,6 +2848,9 @@ char* tr_parse_uri(str* in, trans_t *t) } else if(name.len==4 && strncasecmp(name.s, "suri", 4)==0) { t->subtype = TR_URI_SURI; goto done; + } else if(name.len==4 && strncasecmp(name.s, "duri", 4)==0) { + t->subtype = TR_URI_DURI; + goto done; } else if(name.len==6 && strncasecmp(name.s, "params", 6)==0) { t->subtype = TR_URI_PARAMS; goto done; diff --git a/src/modules/pv/pv_trans.h b/src/modules/pv/pv_trans.h index bf4c922243..d15df89057 100644 --- a/src/modules/pv/pv_trans.h +++ b/src/modules/pv/pv_trans.h @@ -50,7 +50,7 @@ enum _tr_uri_subtype { TR_URI_NONE=0, TR_URI_USER, TR_URI_HOST, TR_URI_PASSWD, TR_URI_PORT, TR_URI_PARAMS, TR_URI_PARAM, TR_URI_HEADERS, TR_URI_TRANSPORT, TR_URI_TTL, TR_URI_UPARAM, TR_URI_MADDR, TR_URI_METHOD, TR_URI_LR, - TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_SURI + TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_DURI, TR_URI_SURI }; enum _tr_param_subtype { TR_PL_NONE=0, TR_PL_VALUE, TR_PL_VALUEAT, TR_PL_NAME, TR_PL_COUNT

1 0

git:master:4a3ad2bf: pv: new transformation {uri.suri}
by Daniel-Constantin Mierla 28 Aug '20

28 Aug '20

Module: kamailio Branch: master Commit: 4a3ad2bf58d6c18f522bb73330030403a50cafbf URL: https://github.com/kamailio/kamailio/commit/4a3ad2bf58d6c18f522bb7333003040… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-28T20:42:15+02:00 pv: new transformation {uri.suri} - get the simple routing uri - keep only transport parameter, if exists --- Modified: src/modules/pv/pv_trans.c Modified: src/modules/pv/pv_trans.h --- Diff: https://github.com/kamailio/kamailio/commit/4a3ad2bf58d6c18f522bb7333003040… Patch: https://github.com/kamailio/kamailio/commit/4a3ad2bf58d6c18f522bb7333003040… --- diff --git a/src/modules/pv/pv_trans.c b/src/modules/pv/pv_trans.c index d07e55e719..b9eb291b99 100644 --- a/src/modules/pv/pv_trans.c +++ b/src/modules/pv/pv_trans.c @@ -1408,6 +1408,62 @@ int tr_eval_uri(struct sip_msg *msg, tr_param_t *tp, int subtype, case TR_URI_PASSWD: val->rs = (_tr_parsed_uri.passwd.s)?_tr_parsed_uri.passwd:_tr_empty; break; + case TR_URI_SURI: + if(_tr_uri.len >= TR_BUFFER_SIZE) { + LM_WARN("uri too long [%.*s] (%d)\n", + _tr_uri.len, _tr_uri.s, _tr_uri.len); + val->rs = _tr_empty; + break; + } + + tr_set_crt_buffer(); + sv.s = _tr_uri.s; + sv.len = 0; + while(sv.len<_tr_uri.len) { + if(_tr_uri.s[sv.len]==':') { + break; + } + sv.len++; + } + if(_tr_uri.s[sv.len]!=':') { + LM_WARN("uri schema not found [%.*s] (%d)\n", + _tr_uri.len, _tr_uri.s, _tr_uri.len); + val->rs = _tr_empty; + break; + } + sv.len++; + memcpy(_tr_buffer, sv.s, sv.len); + sv.s = _tr_buffer; + sv.len++; + if(_tr_parsed_uri.user.len > 0) { + memcpy(sv.s + sv.len, _tr_parsed_uri.user.s, + _tr_parsed_uri.user.len); + sv.len += _tr_parsed_uri.user.len; + sv.s[sv.len] = '@'; + sv.len++; + } + if(_tr_parsed_uri.host.len > 0) { + memcpy(sv.s + sv.len, _tr_parsed_uri.host.s, + _tr_parsed_uri.host.len); + sv.len += _tr_parsed_uri.host.len; + } + if(_tr_parsed_uri.port.len > 0) { + sv.s[sv.len] = ':'; + sv.len++; + memcpy(sv.s + sv.len, _tr_parsed_uri.port.s, + _tr_parsed_uri.port.len); + sv.len += _tr_parsed_uri.port.len; + } + if(_tr_parsed_uri.transport_val.len > 0) { + memcpy(sv.s + sv.len, ";transport=", 11); + sv.len += 11; + memcpy(sv.s + sv.len, _tr_parsed_uri.transport_val.s, + _tr_parsed_uri.transport_val.len); + sv.len += _tr_parsed_uri.transport_val.len; + } + sv.s[sv.len] = '\0'; + val->rs = sv; + break; case TR_URI_PORT: val->flags |= PV_TYPE_INT|PV_VAL_INT; val->rs = (_tr_parsed_uri.port.s)?_tr_parsed_uri.port:_tr_empty; @@ -2788,6 +2844,9 @@ char* tr_parse_uri(str* in, trans_t *t) } else if(name.len==4 && strncasecmp(name.s, "port", 4)==0) { t->subtype = TR_URI_PORT; goto done; + } else if(name.len==4 && strncasecmp(name.s, "suri", 4)==0) { + t->subtype = TR_URI_SURI; + goto done; } else if(name.len==6 && strncasecmp(name.s, "params", 6)==0) { t->subtype = TR_URI_PARAMS; goto done; diff --git a/src/modules/pv/pv_trans.h b/src/modules/pv/pv_trans.h index 6494852246..bf4c922243 100644 --- a/src/modules/pv/pv_trans.h +++ b/src/modules/pv/pv_trans.h @@ -50,7 +50,7 @@ enum _tr_uri_subtype { TR_URI_NONE=0, TR_URI_USER, TR_URI_HOST, TR_URI_PASSWD, TR_URI_PORT, TR_URI_PARAMS, TR_URI_PARAM, TR_URI_HEADERS, TR_URI_TRANSPORT, TR_URI_TTL, TR_URI_UPARAM, TR_URI_MADDR, TR_URI_METHOD, TR_URI_LR, - TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET + TR_URI_R2, TR_URI_SCHEME, TR_URI_TOSOCKET, TR_URI_SURI }; enum _tr_param_subtype { TR_PL_NONE=0, TR_PL_VALUE, TR_PL_VALUEAT, TR_PL_NAME, TR_PL_COUNT

1 0

[kamailio/kamailio] TCP : queue full, x requests queued (total handled y) (#2392)
by ThomasSevestre 28 Aug '20

28 Aug '20

Hello, Do you know what can cause this issue ? Do you need more information ? ### Description I had an issue on a kamailio instance with TLS connections. User agents where not able to register to kamailio using TLS With the follonwing setup : UDP => Kamailio => TLS Kamailio was able to route from UDP to TLS but not from TLS to UDP. ### Troubleshooting #### Log Messages Here are some unusual logs during the issue : ``` ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 27 , queue entry 0, retries 17, connection 0x7f913e1d7fd8, tcp socket 134, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 23 , queue entry 1, retries 13, connection 0x7f913e1a0eb0, tcp socket 105, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 25 , queue entry 2, retries 11, connection 0x7f913e1ada08, tcp socket 137, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 27 , queue entry 3, retries 9, connection 0x7f913e1c2cf0, tcp socket 170, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 23 , queue entry 0, retries 7, connection 0x7f913e1cb480, tcp socket 281, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 25 , queue entry 1, retries 5, connection 0x7f913e1e4b30, tcp socket 398, errno=11 (Resource temporarily unavailable) ERROR: <core> [core/tcp_main.c:3504]: send_fd_queue_run(): send_fd failed on socket 27 , queue entry 2, retries 3, connection 0x7f913e1e8ef8, tcp socket 406, errno=11 (Resource temporarily unavailable) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 1, socket 23: queue full, 304 requests queued (total handled 425) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 2, socket 25: queue full, 304 requests queued (total handled 4940) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 3, socket 27: queue full, 304 requests queued (total handled 310) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 1, socket 23: queue full, 305 requests queued (total handled 426) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 2, socket 25: queue full, 305 requests queued (total handled 4941) CRITICAL: <core> [core/tcp_main.c:4216]: send2child(): tcp child 3, socket 27: queue full, 305 requests queued (total handled 311) ``` Just before the CRITICAL log, there is this comment in sources : /* FIXME: remove after debugging */ ### Additional Information ``` version: kamailio 5.3.5 (x86_64/linux) 9e70e8 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 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: 9e70e8 compiled on 03:49:09 Jun 24 2020 with /usr/bin/gcc 5.4.0 ``` -- 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/issues/2392

3 7

[kamailio/kamailio] pkg/kamailio/deb: libxmlrpc-c3-dev becomes libxmlrpc-core-c3-dev on debian buster (#2432)
by Karsten Horsmann 27 Aug '20

27 Aug '20

### Description Hi all, make deb on debian buster should make kamailio packages. The libxmlrpc-c3-dev seems renamed to libxmlrpc-core-c3-dev. ### Troubleshooting #### Reproduction git clone kamailio cd kamailio git checkout 5.4 make deb #### Debugging Data ``` pi@pi:~/src/kamailio (5.4=)$ make deb make -C src/ deb make[1]: Entering directory '/home/pi/src/kamailio/src' target architecture <arm6>, host architecture <arm7> making config... (cd ..; \ if [ -d debian ]; then \ dpkg-buildpackage -rfakeroot -tc -uc -us; \ rm debian; \ else \ ln -s pkg/kamailio/deb/debian debian; \ dpkg-buildpackage -rfakeroot -tc -uc -us; \ rm debian; \ fi) dpkg-buildpackage: info: source package kamailio dpkg-buildpackage: info: source version 5.4.0 dpkg-buildpackage: info: source distribution unstable dpkg-buildpackage: info: source changed by Victor Seva <vseva(a)debian.org> dpkg-buildpackage: info: host architecture armhf dpkg-source --before-build . dpkg-checkbuilddeps: error: Unmet build dependencies: libxmlrpc-c3-dev dpkg-buildpackage: warning: build dependencies/conflicts unsatisfied; aborting dpkg-buildpackage: warning: (Use -d flag to override.) make[1]: Leaving directory '/home/pi/src/kamailio/src' ``` #### Log Messages ``` (paste your log messages here) ``` #### SIP Traffic ``` (paste your sip traffic here) ``` ### Possible Solutions ``` pi@pi:~/src/kamailio (5.4-patch-libxmlrpc-core)$ git diff HEAD HEAD^ diff --git a/pkg/kamailio/deb/debian/backports/buster b/pkg/kamailio/deb/debian/backports/buster index 8ccfda1db2..82a41a8d4f 100755 --- a/pkg/kamailio/deb/debian/backports/buster +++ b/pkg/kamailio/deb/debian/backports/buster @@ -7,7 +7,7 @@ rm -rf ${DIST} cp -r debian ${DIST} # No mi_xmlrpc module -sed -i -e '/libxmlrpc-core-c3-dev/d' -e '/^Package: kamailio-xmlrpc-modules/,/^$/d' \ +sed -i -e '/libxmlrpc-c3-dev/d' -e '/^Package: kamailio-xmlrpc-modules/,/^$/d' \ ${DIST}/control sed -i -e 's/ mi_xmlrpc[ ,$]*/ /' ${DIST}/rules sed -i -e '/^EXTRA_EXCLUDED_MODULES=/s/$/ mi_xmlrpc/' ${DIST}/rules diff --git a/pkg/kamailio/deb/debian/control b/pkg/kamailio/deb/debian/control index a423d6c70c..52957ddabc 100644 --- a/pkg/kamailio/deb/debian/control +++ b/pkg/kamailio/deb/debian/control @@ -49,7 +49,7 @@ Build-Depends: libunistring-dev, libval-dev, libxml2-dev, - libxmlrpc-core-c3-dev, + libxmlrpc-c3-dev, openssl, pkg-config, python, diff --git a/utils/db_berkeley/kambdb_recover b/utils/db_berkeley/kambdb_recover deleted file mode 100755 index 88d06da872..0000000000 ``` ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` kamailio -v version: kamailio 5.4.0 (arm6/linux) a24d6c-dirty 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: a24d6c -dirty compiled with gcc 8.3.0 ``` * **Operating System**: ``` pi@pi:~/src/kamailio (5.4-patch-libxmlrpc-core)$ lsb_release -a No LSB modules are available. Distributor ID: Raspbian Description: Raspbian GNU/Linux 10 (buster) Release: 10 Codename: buster lsb_release -a No LSB modules are available. Distributor ID: Debian Description: Debian GNU/Linux 10 (buster) Release: 10 Codename: buster ``` -- 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/issues/2432

3 4

git:5.4:ac56dfde: usrloc: get_udomain() - safety checks for location table name parameter
by Daniel-Constantin Mierla 27 Aug '20

27 Aug '20

Module: kamailio Branch: 5.4 Commit: ac56dfde20c5e689fb911debdd33a6155f7b130c URL: https://github.com/kamailio/kamailio/commit/ac56dfde20c5e689fb911debdd33a61… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-08-27T12:36:11+02:00 usrloc: get_udomain() - safety checks for location table name parameter (cherry picked from commit 29dde9fea1eb21b4bb484286f7443dd32f611bd2) --- Modified: src/modules/usrloc/dlist.c --- Diff: https://github.com/kamailio/kamailio/commit/ac56dfde20c5e689fb911debdd33a61… Patch: https://github.com/kamailio/kamailio/commit/ac56dfde20c5e689fb911debdd33a61… --- diff --git a/src/modules/usrloc/dlist.c b/src/modules/usrloc/dlist.c index cba8ec425c..96a95fe3c9 100644 --- a/src/modules/usrloc/dlist.c +++ b/src/modules/usrloc/dlist.c @@ -792,13 +792,24 @@ int get_udomain(const char* _n, udomain_t** _d) dlist_t* d; str s; + if(_n == NULL) { + LM_ERR("null location table name\n"); + goto notfound; + } + s.s = (char*)_n; s.len = strlen(_n); + if(s.len <= 0) { + LM_ERR("empty location table name\n"); + goto notfound; + } if (find_dlist(&s, &d) == 0) { *_d = d->d; return 0; } + +notfound: *_d = 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