[sr-dev] [kamailio] dialog: force cleanup of aged terminated dialogs (23a3481)

[Dmitri Savolainen]

I  cherry-picked it to 4.2.7 
Dialog was  deleted a after 5mins (according to tdlg->end_ts<tm-300 ),  but it still may confuse configs with current call limit check (get_profile_size calling can't get dialog status as parameter )
May be unref dialog after DLG_EVENT_RPL3xx  at once? I can't dive into dialog module source code now, so not sure is this safe, but it works:
diff --git a/modules/dialog/dlg_hash.c b/modules/dialog/dlg_hash.c
index a305468..202f0b9 100644
--- a/modules/dialog/dlg_hash.c
+++ b/modules/dialog/dlg_hash.c
@@ -965,6 +965,7 @@ void next_state_dlg(dlg_cell_t *dlg, int event,
                                case DLG_STATE_UNCONFIRMED:
                                case DLG_STATE_EARLY:
                                        dlg->state = DLG_STATE_DELETED;
+                                       unref_dlg_unsafe(dlg,1,d_entry);
                                        *unref = 1;
                                        break;
                                default:


---
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/commit/23a3481302d5aca04be0e8d10f6d4ce23dbc36aa#commitcomment-16765433
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20160318/3bfc8ef4/attachment-0001.html>