sr-dev August 2018

sr-dev@lists.kamailio.org

22 participants
281 discussions

git:master:ee5d2b54: ims_ipsec_pcscf: sec-agree implementation for IMS

by Tsvetomir Dimitrov

Module: kamailio Branch: master Commit: ee5d2b54145d9869c1fa92bd9365fde83f0b02d6 URL: https://github.com/kamailio/kamailio/commit/ee5d2b54145d9869c1fa92bd9365fde… Author: Tsvetomir Dimitrov <tsv.dimitrov(a)gmail.com> Committer: Tsvetomir Dimitrov <tsv.dimitrov(a)gmail.com> Date: 2018-07-30T10:59:16+03:00 ims_ipsec_pcscf: sec-agree implementation for IMS --- Added: src/modules/ims_ipsec_pcscf/Makefile Added: src/modules/ims_ipsec_pcscf/cmd.c Added: src/modules/ims_ipsec_pcscf/cmd.h Added: src/modules/ims_ipsec_pcscf/doc/Makefile Added: src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf.xml Added: src/modules/ims_ipsec_pcscf/doc/ims_ipsec_pcscf_admin.xml Added: src/modules/ims_ipsec_pcscf/ims_ipsec_pcscf_mod.c Added: src/modules/ims_ipsec_pcscf/ipsec.c Added: src/modules/ims_ipsec_pcscf/ipsec.h Added: src/modules/ims_ipsec_pcscf/run_spi_list_tests.sh Added: src/modules/ims_ipsec_pcscf/spi_gen.c Added: src/modules/ims_ipsec_pcscf/spi_gen.h Added: src/modules/ims_ipsec_pcscf/spi_list.c Added: src/modules/ims_ipsec_pcscf/spi_list.h Added: src/modules/ims_ipsec_pcscf/spi_list_tests.c --- Diff: https://github.com/kamailio/kamailio/commit/ee5d2b54145d9869c1fa92bd9365fde… Patch: https://github.com/kamailio/kamailio/commit/ee5d2b54145d9869c1fa92bd9365fde…

6 years, 4 months

git:5.0:d67652f9: presence: limit the number of subscriptions handled in timer_dbonly

by Luis Azedo

Module: kamailio Branch: 5.0 Commit: d67652f9688f72c91841b4ad8bc906d9957733bd URL: https://github.com/kamailio/kamailio/commit/d67652f9688f72c91841b4ad8bc906d… Author: lazedo <luis.azedo(a)factorlusitano.com> Committer: Luis Azedo <luis(a)2600hz.com> Date: 2018-08-01T23:38:51+01:00 presence: limit the number of subscriptions handled in timer_dbonly due to the way update_db_subs_timer_dbonly handles the query to the database (getting all records), if for some reason a burst of terminating subscriptions occurs, most likely there will be no package memory to process all expiring subscriptions. this commit uses the same pattern as other routines in presence by using db_fetch_query with fetch_rows parameter module. because we create the subs in the loop and then call handle_expired_subs to avoid locking issues the subscription should already be deleted from the database when it returns from handle_expired_subs, there's no reason to issue the last delete, and that was removed. (cherry picked from commit e6a7a3f37e80683d941321a6e10b636f0bfb4478) --- Modified: src/modules/presence/subscribe.c --- Diff: https://github.com/kamailio/kamailio/commit/d67652f9688f72c91841b4ad8bc906d… Patch: https://github.com/kamailio/kamailio/commit/d67652f9688f72c91841b4ad8bc906d… --- diff --git a/src/modules/presence/subscribe.c b/src/modules/presence/subscribe.c index 8b53500c18..754d1bf2ca 100644 --- a/src/modules/presence/subscribe.c +++ b/src/modules/presence/subscribe.c @@ -2010,14 +2010,14 @@ void update_db_subs_timer_dbonly(void) db_val_t qvals[1]; db_key_t result_cols[18]; int pres_uri_col, to_user_col, to_domain_col, from_user_col, from_domain_col, - callid_col, totag_col, fromtag_col, event_col, event_id_col, - local_cseq_col, expires_col, rr_col, sockinfo_col, - contact_col, lcontact_col, watcher_user_col, watcher_domain_col; + callid_col, totag_col, fromtag_col, event_col, event_id_col, + local_cseq_col, expires_col, rr_col, sockinfo_col, + contact_col, lcontact_col, watcher_user_col, watcher_domain_col; int n_result_cols = 0; db1_res_t *result= NULL; - db_row_t *row = NULL; + db_row_t *rows; db_val_t *row_vals= NULL; - int i; + int i, res; subs_t s, *s_new, *s_array = NULL, *s_del; str ev_name; pres_ev_t* event; @@ -2056,27 +2056,26 @@ void update_db_subs_timer_dbonly(void) return; } - if (pa_dbf.query(pa_db, qcols, qops, qvals, result_cols, - 1, n_result_cols, 0, &result) < 0) { + res = db_fetch_query(&pa_dbf, pres_fetch_rows, pa_db, qcols, qops, qvals, result_cols,1, n_result_cols, 0, &result ); + if (res < 0) + { LM_ERR("failed to query database for expired subscriptions\n"); - if(result) + if (result) { pa_dbf.free_result(pa_db, result); + } return; } - if(result== NULL) - return; - - if(result->n <=0 ) { - pa_dbf.free_result(pa_db, result); + if(result == NULL) { + LM_DBG("no results returned\n"); return; } - LM_DBG("found %d dialogs\n", result->n); - for(i=0; i<result->n; i++) - { - row = &result->rows[i]; - row_vals = ROW_VALUES(row); + LM_DBG("processing %d dialogs\n", RES_ROW_N(result)); + s_array = NULL; + rows = RES_ROWS(result); + for (i = 0; i < RES_ROW_N(result); i++) { + row_vals = ROW_VALUES(&rows[i]); memset(&s, 0, sizeof(subs_t)); @@ -2157,12 +2156,6 @@ void update_db_subs_timer_dbonly(void) s_new = s_new->next; pkg_free(s_del); } - - /* delete the expired subscriptions */ - if(pa_dbf.delete(pa_db, qcols, qops, qvals, 1) < 0) - { - LM_ERR("deleting expired information from database\n"); - } } void update_db_subs_timer_dbnone(int no_lock)

6 years, 4 months

git:5.1:4868a1a1: presence: limit the number of subscriptions handled in timer_dbonly

by Luis Azedo

Module: kamailio Branch: 5.1 Commit: 4868a1a18b54b0fb337453c8257b2466b78fb71b URL: https://github.com/kamailio/kamailio/commit/4868a1a18b54b0fb337453c8257b246… Author: lazedo <luis.azedo(a)factorlusitano.com> Committer: Luis Azedo <luis(a)2600hz.com> Date: 2018-08-01T23:38:20+01:00 presence: limit the number of subscriptions handled in timer_dbonly due to the way update_db_subs_timer_dbonly handles the query to the database (getting all records), if for some reason a burst of terminating subscriptions occurs, most likely there will be no package memory to process all expiring subscriptions. this commit uses the same pattern as other routines in presence by using db_fetch_query with fetch_rows parameter module. because we create the subs in the loop and then call handle_expired_subs to avoid locking issues the subscription should already be deleted from the database when it returns from handle_expired_subs, there's no reason to issue the last delete, and that was removed. (cherry picked from commit e6a7a3f37e80683d941321a6e10b636f0bfb4478) --- Modified: src/modules/presence/subscribe.c --- Diff: https://github.com/kamailio/kamailio/commit/4868a1a18b54b0fb337453c8257b246… Patch: https://github.com/kamailio/kamailio/commit/4868a1a18b54b0fb337453c8257b246… --- diff --git a/src/modules/presence/subscribe.c b/src/modules/presence/subscribe.c index 4c74ac7980..e3352b1615 100644 --- a/src/modules/presence/subscribe.c +++ b/src/modules/presence/subscribe.c @@ -2026,14 +2026,14 @@ void update_db_subs_timer_dbonly(void) db_val_t qvals[1]; db_key_t result_cols[18]; int pres_uri_col, to_user_col, to_domain_col, from_user_col, from_domain_col, - callid_col, totag_col, fromtag_col, event_col, event_id_col, - local_cseq_col, expires_col, rr_col, sockinfo_col, - contact_col, lcontact_col, watcher_user_col, watcher_domain_col; + callid_col, totag_col, fromtag_col, event_col, event_id_col, + local_cseq_col, expires_col, rr_col, sockinfo_col, + contact_col, lcontact_col, watcher_user_col, watcher_domain_col; int n_result_cols = 0; db1_res_t *result= NULL; - db_row_t *row = NULL; + db_row_t *rows; db_val_t *row_vals= NULL; - int i; + int i, res; subs_t s, *s_new, *s_array = NULL, *s_del; str ev_name; pres_ev_t* event; @@ -2072,27 +2072,26 @@ void update_db_subs_timer_dbonly(void) return; } - if (pa_dbf.query(pa_db, qcols, qops, qvals, result_cols, - 1, n_result_cols, 0, &result) < 0) { + res = db_fetch_query(&pa_dbf, pres_fetch_rows, pa_db, qcols, qops, qvals, result_cols,1, n_result_cols, 0, &result ); + if (res < 0) + { LM_ERR("failed to query database for expired subscriptions\n"); - if(result) + if (result) { pa_dbf.free_result(pa_db, result); + } return; } - if(result== NULL) - return; - - if(result->n <=0 ) { - pa_dbf.free_result(pa_db, result); + if(result == NULL) { + LM_DBG("no results returned\n"); return; } - LM_DBG("found %d dialogs\n", result->n); - for(i=0; i<result->n; i++) - { - row = &result->rows[i]; - row_vals = ROW_VALUES(row); + LM_DBG("processing %d dialogs\n", RES_ROW_N(result)); + s_array = NULL; + rows = RES_ROWS(result); + for (i = 0; i < RES_ROW_N(result); i++) { + row_vals = ROW_VALUES(&rows[i]); memset(&s, 0, sizeof(subs_t)); @@ -2173,12 +2172,6 @@ void update_db_subs_timer_dbonly(void) s_new = s_new->next; pkg_free(s_del); } - - /* delete the expired subscriptions */ - if(pa_dbf.delete(pa_db, qcols, qops, qvals, 1) < 0) - { - LM_ERR("deleting expired information from database\n"); - } } void update_db_subs_timer_dbnone(int no_lock)

6 years, 4 months

git:master:e6a7a3f3: presence: limit the number of subscriptions handled in timer_dbonly

by lazedo

Module: kamailio Branch: master Commit: e6a7a3f37e80683d941321a6e10b636f0bfb4478 URL: https://github.com/kamailio/kamailio/commit/e6a7a3f37e80683d941321a6e10b636… Author: lazedo <luis.azedo(a)factorlusitano.com> Committer: lazedo <luis.azedo(a)factorlusitano.com> Date: 2018-08-01T23:36:22+01:00 presence: limit the number of subscriptions handled in timer_dbonly due to the way update_db_subs_timer_dbonly handles the query to the database (getting all records), if for some reason a burst of terminating subscriptions occurs, most likely there will be no package memory to process all expiring subscriptions. this commit uses the same pattern as other routines in presence by using db_fetch_query with fetch_rows parameter module. because we create the subs in the loop and then call handle_expired_subs to avoid locking issues the subscription should already be deleted from the database when it returns from handle_expired_subs, there's no reason to issue the last delete, and that was removed. --- Modified: src/modules/presence/subscribe.c --- Diff: https://github.com/kamailio/kamailio/commit/e6a7a3f37e80683d941321a6e10b636… Patch: https://github.com/kamailio/kamailio/commit/e6a7a3f37e80683d941321a6e10b636… --- diff --git a/src/modules/presence/subscribe.c b/src/modules/presence/subscribe.c index 4c74ac7980..e3352b1615 100644 --- a/src/modules/presence/subscribe.c +++ b/src/modules/presence/subscribe.c @@ -2026,14 +2026,14 @@ void update_db_subs_timer_dbonly(void) db_val_t qvals[1]; db_key_t result_cols[18]; int pres_uri_col, to_user_col, to_domain_col, from_user_col, from_domain_col, - callid_col, totag_col, fromtag_col, event_col, event_id_col, - local_cseq_col, expires_col, rr_col, sockinfo_col, - contact_col, lcontact_col, watcher_user_col, watcher_domain_col; + callid_col, totag_col, fromtag_col, event_col, event_id_col, + local_cseq_col, expires_col, rr_col, sockinfo_col, + contact_col, lcontact_col, watcher_user_col, watcher_domain_col; int n_result_cols = 0; db1_res_t *result= NULL; - db_row_t *row = NULL; + db_row_t *rows; db_val_t *row_vals= NULL; - int i; + int i, res; subs_t s, *s_new, *s_array = NULL, *s_del; str ev_name; pres_ev_t* event; @@ -2072,27 +2072,26 @@ void update_db_subs_timer_dbonly(void) return; } - if (pa_dbf.query(pa_db, qcols, qops, qvals, result_cols, - 1, n_result_cols, 0, &result) < 0) { + res = db_fetch_query(&pa_dbf, pres_fetch_rows, pa_db, qcols, qops, qvals, result_cols,1, n_result_cols, 0, &result ); + if (res < 0) + { LM_ERR("failed to query database for expired subscriptions\n"); - if(result) + if (result) { pa_dbf.free_result(pa_db, result); + } return; } - if(result== NULL) - return; - - if(result->n <=0 ) { - pa_dbf.free_result(pa_db, result); + if(result == NULL) { + LM_DBG("no results returned\n"); return; } - LM_DBG("found %d dialogs\n", result->n); - for(i=0; i<result->n; i++) - { - row = &result->rows[i]; - row_vals = ROW_VALUES(row); + LM_DBG("processing %d dialogs\n", RES_ROW_N(result)); + s_array = NULL; + rows = RES_ROWS(result); + for (i = 0; i < RES_ROW_N(result); i++) { + row_vals = ROW_VALUES(&rows[i]); memset(&s, 0, sizeof(subs_t)); @@ -2173,12 +2172,6 @@ void update_db_subs_timer_dbonly(void) s_new = s_new->next; pkg_free(s_del); } - - /* delete the expired subscriptions */ - if(pa_dbf.delete(pa_db, qcols, qops, qvals, 1) < 0) - { - LM_ERR("deleting expired information from database\n"); - } } void update_db_subs_timer_dbnone(int no_lock)

6 years, 4 months

[kamailio/kamailio] presence: limit the number of subscriptions handled in timer_dbonly (#1609)

by lazedo

#### 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) - [ ] 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 - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  due to the way update_db_subs_timer_dbonly handles the query to the database (getting all records), if for some reason a burst of terminating subscriptions occurs, most likely there will be no package memory to process all expiring subscriptions. this commit uses the same pattern as other routines in presence by using db_fetch_query with fetch_rows parameter module. because we create the subs in the loop and then call handle_expired_subs to avoid locking issues the subscription should already be deleted from the database when it returns from handle_expired_subs, there's no reason to issue the last delete, and that was removed. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1609 -- Commit Summary -- * presence: limit the number of subscriptions handled in timer_dbonly -- File Changes -- M src/modules/presence/subscribe.c (41) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1609.patch https://github.com/kamailio/kamailio/pull/1609.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/1609

6 years, 4 months

[kamailio/kamailio] Update dlg_cb.c (#1607)

by kosza

#### Pre-Submission Checklist    - [ ] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [ ] 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) - [ ] 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 - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  The load_cbs and create_cbs were interchanged in line 136. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1607 -- Commit Summary -- * Update dlg_cb.c -- File Changes -- M src/modules/dialog/dlg_cb.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1607.patch https://github.com/kamailio/kamailio/pull/1607.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/1607

6 years, 4 months

[kamailio/kamailio] Fix no cdr after restart (#1606)

by kosza

Hello! I am using kamailio 5.1.4. version: kamailio 5.1.4 (x86_64/linux) flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, 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_LISTEN 16, 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 on 15:11:36 Jul 20 2018 with gcc 7.3.0 The problem is that if kamailio is restarted during a call the cdr would not be created. The run_load_callbacks() should be call after acc module is loaded. The acc module will setup DLGCB_LOADED type callback. So run_load_callbacks() should be delayed and run only once, the child_init(PROC_MAIN) is perfect for it.   #### Pre-Submission Checklist    - [ ] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [ ] 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) - [ ] 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 - [ ] 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/1606 -- Commit Summary -- * Fix no cdr after restart bug -- File Changes -- M src/modules/dialog/dialog.c (5) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1606.patch https://github.com/kamailio/kamailio/pull/1606.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/1606

6 years, 4 months

git:master:3d0e1f79: dialog: run run_load_callbacks() on child_init for rank PROC_INIT

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 3d0e1f796eb359689b75eebbb04cd09c6ee6807a URL: https://github.com/kamailio/kamailio/commit/3d0e1f796eb359689b75eebbb04cd09… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-08-02T00:16:26+02:00 dialog: run run_load_callbacks() on child_init for rank PROC_INIT - proper test for create_cbs - based on PR #1606 and #1607 by kosza --- Modified: src/modules/dialog/dialog.c Modified: src/modules/dialog/dlg_cb.c --- Diff: https://github.com/kamailio/kamailio/commit/3d0e1f796eb359689b75eebbb04cd09… Patch: https://github.com/kamailio/kamailio/commit/3d0e1f796eb359689b75eebbb04cd09… --- diff --git a/src/modules/dialog/dialog.c b/src/modules/dialog/dialog.c index 84262ea25f..a7efe2d7e9 100644 --- a/src/modules/dialog/dialog.c +++ b/src/modules/dialog/dialog.c @@ -711,7 +711,6 @@ static int mod_init(void) LM_ERR("failed to initialize the DB support\n"); return -1; } - run_load_callbacks(); } destroy_dlg_callbacks( DLGCB_LOADED ); @@ -742,6 +741,13 @@ static int child_init(int rank) { dlg_db_mode = dlg_db_mode_param; + + if(rank==PROC_INIT) { + if (dlg_db_mode!=DB_MODE_NONE) { + run_load_callbacks(); + } + } + if(rank==PROC_MAIN) { if(dlg_timer_procs>0) { if(fork_sync_timer(PROC_TIMER, "Dialog Main Timer", 1 /*socks flag*/, diff --git a/src/modules/dialog/dlg_cb.c b/src/modules/dialog/dlg_cb.c index 6ba48513ec..12946994ba 100644 --- a/src/modules/dialog/dlg_cb.c +++ b/src/modules/dialog/dlg_cb.c @@ -133,7 +133,7 @@ int register_dlgcb(struct dlg_cell *dlg, int types, dialog_cb f, cb->callback_param_free = ff; if ( types==DLGCB_CREATED ) { - if (load_cbs==POINTER_CLOSED_MARKER) { + if (create_cbs==POINTER_CLOSED_MARKER) { LM_CRIT("DLGCB_CREATED type registered after shutdown!?!\n"); goto error; }

6 years, 4 months

git:master:3924afe2: topos_redis: execute end dialog function only for BYE request

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 3924afe2917e64066ea796597104b0e00358b180 URL: https://github.com/kamailio/kamailio/commit/3924afe2917e64066ea796597104b0e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-08-01T22:55:06+02:00 topos_redis: execute end dialog function only for BYE request - reported by GH #1608 --- Modified: src/modules/topos_redis/topos_redis_storage.c --- Diff: https://github.com/kamailio/kamailio/commit/3924afe2917e64066ea796597104b0e… Patch: https://github.com/kamailio/kamailio/commit/3924afe2917e64066ea796597104b0e… --- diff --git a/src/modules/topos_redis/topos_redis_storage.c b/src/modules/topos_redis/topos_redis_storage.c index bc810e702b..4ba31f404a 100644 --- a/src/modules/topos_redis/topos_redis_storage.c +++ b/src/modules/topos_redis/topos_redis_storage.c @@ -1314,6 +1314,10 @@ int tps_redis_end_dialog(sip_msg_t *msg, tps_data_t *md, tps_data_t *sd) int32_t liflags; unsigned long lval = 0; + if(md->s_method_id != METHOD_BYE) { + return 0; + } + if(sd->a_uuid.len<=0 && sd->b_uuid.len<=0) { LM_INFO("no uuid for this message\n"); return -1;

6 years, 4 months

[kamailio/kamailio] crash in ims_usrloc_scscf.so (#1600)

by Sergey Zyrianov

### Description IMS reregistration triggers core dump on 5.1.4. 1) Create IMS registration 2) Put IMS terminal offline 3) Put it back online to update IMS registration 4) Register requests time out on s-cscf ### Troubleshooting This seems to be related to the cleaning up contacts during registration/re-registration. Related to the https://github.com/kamailio/kamailio/pull/1562. This PR did not make it to 5.1.4 but affected code was clearly involved according to the logs. #### Reproduction #### Debugging Data #### Log Messages ``` Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [impurecord.c:451]: process_impurecord(): There are 2 contacts to expire/unlink Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [impurecord.c:454]: process_impurecord(): expiring contact 0: [sip:XXXYY4000013362@10.110.1 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [impurecord.c:1237]: unlink_contact_from_impu(): asked to unlink contact [0x7f7141cf29a0] => [sip:XXXYY4000 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [impurecord.c:1106]: remove_impucontact_from_list(): deleting mid list Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [usrloc_db.c:1130]: db_unlink_contact_from_impu(): DB: un-linking contact to IMPU Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: <core> [db_res.c:120]: db_new_result(): allocate 56 bytes for result set at 0x7f714ba6e620 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: <core> [db_res.c:79]: db_free_columns(): freeing 0 columns Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: <core> [db_res.c:138]: db_free_result(): freeing result set at 0x7f714ba6e620 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [usrloc_db.c:1155]: db_unlink_contact_from_impu(): Delete query success Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [udomain.c:852]: unref_contact_unsafe(): decrementing ref count on contact [sip:XXXYY4000013362@10.110.1.69 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [udomain.c:855]: unref_contact_unsafe(): contact [sip:XXXYY4000013362@10.110.1.69:5060;alias=10.110.1.69~50 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [impurecord.c:1263]: unlink_contact_from_impu(): unlinking contact [0x7f7141cf29a0] => [sip:XXXYY4000013362 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [impurecord.c:454]: process_impurecord(): expiring contact 1: [sip:XXXYY4000013362@10.110.1 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [impurecord.c:1237]: unlink_contact_from_impu(): asked to unlink contact [0x7f7141cfc878] => [sip:XXXYY4000 Jul 24 07:21:00 scscf scscfXXXYYY-ecs[798]: 1(6) DEBUG: ims_usrloc_scscf [impurecord.c:1103]: remove_impucontact_from_list(): **deleting tail** Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_registrar_scscf [save.c:840]: update_contacts(): updating contacts in RE-REGISTRATION state Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_registrar_scscf [save.c:519]: update_contacts_helper(): updating the contacts for IMPU <sip:XXXYY4000013362@ims.mnc006. Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: <core> [ims_getters.c:812]: cscf_get_sos_uri_param(): cscf_get_sos_uri_param: searching through the uri parameters:alias=10 Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: <core> [ims_getters.c:823]: cscf_get_sos_uri_param(): cscf_get_sos_uri_param:name: alias body: 10.110.1.71~5060~1 Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_registrar_scscf [save.c:137]: calc_contact_expires(): Calculated expires for contact is 600000 Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_registrar_scscf [save.c:548]: update_contacts_helper(): Need to update contact: <sip:XXXYY4000013362@10.110.1.71:5060;a Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_registrar_scscf [save.c:550]: update_contacts_helper(): packing contact information Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_registrar_scscf [save.c:556]: update_contacts_helper(): adding/updating contact based on prior existence Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_usrloc_scscf [impurecord.c:748]: get_scontact(): looking for contact [sip:XXXYY8000099992@10.110.1.71:5060;alias=10.110 Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_registrar_scscf [save.c:562]: update_contacts_helper(): inserting new contact Jul 24 07:22:21 scscf scscfXXXYYY-ecs[798]: 15(20) DEBUG: ims_usrloc_scscf [impurecord.c:562]: insert_scontact(): INSERTing ucontact in usrloc module Jul 24 07:22:21 scscf kernel: traps: kamailio[7186] general protection ip:7f7146511000 sp:7ffc9f3719a0 error:0 in ims_usrloc_scscf.so[7f71464e7000+8f000] ``` #### SIP Traffic ![image](https://user-images.githubusercontent.com/1636682/43138556-8dc89988-8f4f-11e8-887a-53c4fb7805e1.png) ``` (paste your sip traffic here) ``` ### Possible Solutions ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.1.4 (x86_64/linux) ``` * **Operating System**: ``` Linux scscf 4.14.48-coreos-r2 #1 SMP Thu Jun 14 08:23:03 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux ``` -- 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/1600

6 years, 4 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev August 2018