sr-dev September 2014

sr-dev@lists.kamailio.org

33 participants
375 discussions

[tracker] Comment added: zombie dialogs in state 5
by sip-router 16 Sep '14

16 Sep '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#424 - zombie dialogs in state 5 User who did this - Ovidiu Sas (osas) ---------- No luck :( I think a fix should be implemented at tm level, so all modules using tm callbacks could benefit from it. Here are the logs: Sep 16 15:08:00 centos k[28276]: INFO: <script>: [2] BYE sip:4165552222@192.168.2.97:5064 From:sip:1234@192.168.2.62,tag=as0435f26e To:sip:4165552222@192.168.2.62,tag=f738712194561120o4 Call-ID:41a0ddf5-fb426a5c@192.168.2.97 Contact:<null> <192.168.2.2:5060->udp:192.168.2.62:5060> Sep 16 15:08:00 centos k[28276]: DEBUG: dialog [dlg_handlers.c:1163]: dlg_onroute(): route param is 'c46.6f21' (len=8) Sep 16 15:08:00 centos k[28276]: DEBUG: dialog [dlg_hash.c:654]: dlg_lookup(): ref dlg 0x7f11da013070 with 1 -> 3 Sep 16 15:08:00 centos k[28276]: DEBUG: dialog [dlg_hash.c:656]: dlg_lookup(): dialog id=4854 found on entry 1612 Sep 16 15:08:00 centos k[28276]: DEBUG: dialog [dlg_profile.c:511]: set_current_dialog(): setting current dialog [1612:4854] Sep 16 15:08:00 centos k[28276]: DEBUG: dialog [dlg_hash.c:1066]: next_state_dlg(): dialog 0x7f11da013070 changed from state 4 to state 5, due event 7 (ref 3) Sep 16 15:08:00 centos k[28276]: DEBUG: dialog [dlg_hash.c:854]: dlg_ref(): ref dlg 0x7f11da013070 with 1 -> 4 Sep 16 15:08:00 centos k[28276]: DEBUG: dialog [dlg_handlers.c:1552]: dlg_run_event_route(): executing event_route 1 on state 5 Sep 16 15:08:00 centos k[28276]: INFO: <script>: [2] entering event_route[dialog:end] Sep 16 15:08:00 centos k[28272]: INFO: <script>: [3] BYE sip:4165552222@192.168.2.97:5064 From:sip:1234@192.168.2.62,tag=as0435f26e To:sip:4165552222@192.168.2.62,tag=f738712194561120o4 Call-ID:41a0ddf5-fb426a5c@192.168.2.97 Contact:<null> <192.168.2.2:5060->udp:192.168.2.62:5060> Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_handlers.c:1163]: dlg_onroute(): route param is 'c46.6f21' (len=8) Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_hash.c:654]: dlg_lookup(): ref dlg 0x7f11da013070 with 1 -> 5 Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_hash.c:656]: dlg_lookup(): dialog id=4854 found on entry 1612 Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_profile.c:511]: set_current_dialog(): setting current dialog [1612:4854] Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_hash.c:1066]: next_state_dlg(): dialog 0x7f11da013070 changed from state 5 to state 5, due event 7 (ref 5) Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_hash.c:872]: dlg_unref(): unref dlg 0x7f11da013070 with 1 -> 4 Sep 16 15:08:00 centos k[28272]: DEBUG: tm [t_lookup.c:1373]: t_newtran(): DEBUG: t_newtran: msg id=3 , global msg id=2 , T on entrance=0xffffffffffffffff Sep 16 15:08:00 centos k[28272]: DEBUG: tm [t_lookup.c:527]: t_lookup_request(): t_lookup_request: start searching: hash=25884, isACK=0 Sep 16 15:08:00 centos k[28272]: DEBUG: tm [t_lookup.c:485]: matching_3261(): DEBUG: RFC3261 transaction matching failed Sep 16 15:08:00 centos k[28272]: DEBUG: tm [t_lookup.c:709]: t_lookup_request(): DEBUG: t_lookup_request: no transaction found Sep 16 15:08:00 centos k[28272]: DEBUG: tm [t_hooks.c:374]: run_reqin_callbacks_internal(): DBG: trans=0x7f11da013320, callback type 1, id 0 entered Sep 16 15:08:00 centos k[28272]: DEBUG: tm [t_hooks.c:374]: run_reqin_callbacks_internal(): DBG: trans=0x7f11da013320, callback type 1, id 0 entered Sep 16 15:08:00 centos k[28272]: DEBUG: tm [t_funcs.c:394]: t_relay_to(): SER: new transaction fwd'ed Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_hash.c:654]: dlg_lookup(): ref dlg 0x7f11da013070 with 1 -> 5 Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_hash.c:656]: dlg_lookup(): dialog id=4854 found on entry 1612 Sep 16 15:08:00 centos k[28272]: DEBUG: dialog [dlg_hash.c:872]: dlg_unref(): unref dlg 0x7f11da013070 with 1 -> 4 Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_lookup.c:1072]: t_check_msg(): DEBUG: t_check_msg: msg id=3 global id=2 T start=(nil) Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_lookup.c:949]: t_reply_matching(): DEBUG: t_reply_matching: hash 25884 label 0 branch 0 Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_lookup.c:1004]: t_reply_matching(): DEBUG: t_reply_matching: reply matched (T=0x7f11da013320)! Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_hooks.c:288]: run_trans_callbacks_internal(): DBG: trans=0x7f11da013320, callback type 2, id 0 entered Sep 16 15:08:00 centos k[28275]: DEBUG: dialog [dlg_hash.c:654]: dlg_lookup(): ref dlg 0x7f11da013070 with 1 -> 5 Sep 16 15:08:00 centos k[28275]: DEBUG: dialog [dlg_hash.c:656]: dlg_lookup(): dialog id=4854 found on entry 1612 Sep 16 15:08:00 centos k[28275]: DEBUG: dialog [dlg_profile.c:511]: set_current_dialog(): setting current dialog [1612:4854] Sep 16 15:08:00 centos k[28275]: DEBUG: dialog [dlg_hash.c:872]: dlg_unref(): unref dlg 0x7f11da013070 with 1 -> 4 Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_lookup.c:1141]: t_check_msg(): DEBUG: t_check_msg: msg id=3 global id=3 T end=0x7f11da013320 Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_reply.c:2210]: reply_received(): DEBUG: reply_received: org. status uas=0, uac[0]=0 local=0 is_invite=0) Sep 16 15:08:00 centos k[28275]: INFO: <script>: [3] entering onreply_route[BYE] for 200 OK Call-ID:41a0ddf5-fb426a5c@192.168.2.97 Cseq:102 Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_reply.c:1304]: t_should_relay_response(): ->>>>>>>>> T_code=0, new_code=200 Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_reply.c:1822]: relay_reply(): DEBUG: relay_reply: branch=0, save=0, relay=0 icode=0 Sep 16 15:08:00 centos k[28275]: DEBUG: tm [t_reply.c:1663]: cleanup_uac_timers(): DEBUG: cleanup_uac_timers: RETR/FR timers reset Sep 16 15:08:01 centos k[28276]: INFO: <script>: [2] exiting event_route[dialog:end] Sep 16 15:08:01 centos k[28276]: DEBUG: dialog [dlg_hash.c:872]: dlg_unref(): unref dlg 0x7f11da013070 with 1 -> 3 Sep 16 15:08:01 centos k[28276]: DEBUG: dialog [dlg_hash.c:854]: dlg_ref(): ref dlg 0x7f11da013070 with 1 -> 4 Sep 16 15:08:01 centos k[28276]: DEBUG: dialog [dlg_handlers.c:1285]: dlg_onroute(): BYE successfully processed Sep 16 15:08:01 centos k[28276]: DEBUG: dialog [dlg_hash.c:872]: dlg_unref(): unref dlg 0x7f11da013070 with 2 -> 2 Sep 16 15:08:01 centos k[28276]: DEBUG: dialog [dlg_hash.c:872]: dlg_unref(): unref dlg 0x7f11da013070 with 1 -> 1 Sep 16 15:08:01 centos k[28276]: DEBUG: tm [t_lookup.c:1373]: t_newtran(): DEBUG: t_newtran: msg id=2 , global msg id=1 , T on entrance=0xffffffffffffffff Sep 16 15:08:01 centos k[28276]: DEBUG: tm [t_lookup.c:527]: t_lookup_request(): t_lookup_request: start searching: hash=25884, isACK=0 Sep 16 15:08:01 centos k[28276]: DEBUG: tm [t_lookup.c:470]: matching_3261(): DEBUG: RFC3261 transaction matched, tid=55953831 Sep 16 15:08:01 centos k[28276]: DEBUG: tm [t_lookup.c:726]: t_lookup_request(): DEBUG: t_lookup_request: transaction found (T=0x7f11da013320) Sep 16 15:08:01 centos k[28276]: DEBUG: tm [t_reply.c:1623]: t_retransmit_reply(): DEBUG: reply retransmitted. buf=0x7f11d4dd37a0: SIP/2.0 2..., shmem=0x7f11da016a68: SIP/2.0 2 Sep 16 15:08:05 centos k[28278]: DEBUG: dialog [dlg_handlers.c:324]: dlg_iuid_sfree(): freeing dlg iuid [1612:4854] (0x7f11da015d68) ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=424#comment1628 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Comment added: zombie dialogs in state 5
by sip-router 16 Sep '14

16 Sep '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#424 - zombie dialogs in state 5 User who did this - Daniel-Constantin Mierla (miconda) ---------- I just pushed a different patch, where the modules makes a check to see if the transaction is created for BYE. If not, it releases the dialog. No time to test it yet, hopefully you can give it a try quicker and report. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=424#comment1627 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

git:master: dialog: check if the transaction is created for BYE closing the dialog
by Daniel-Constantin Mierla 16 Sep '14

16 Sep '14

Module: sip-router Branch: master Commit: 02e43f87b2fa3be61bc4c63a9f8359457e3e0484 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=02e43f8… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Sep 16 19:05:57 2014 +0200 dialog: check if the transaction is created for BYE closing the dialog - if not, release the dialog, because there will be no tm callbacks called later to release it. In this way avoid zombie dialog. - reported by Ovidiu Sas, FS#424 --- modules/dialog/dlg_handlers.c | 8 ++++++++ modules/dialog/dlg_var.c | 11 +++++++++-- modules/dialog/dlg_var.h | 3 ++- 3 files changed, 19 insertions(+), 3 deletions(-) diff --git a/modules/dialog/dlg_handlers.c b/modules/dialog/dlg_handlers.c index 5a90d71..16c711f 100644 --- a/modules/dialog/dlg_handlers.c +++ b/modules/dialog/dlg_handlers.c @@ -713,6 +713,11 @@ void dlg_onreq(struct cell* t, int type, struct tmcb_params *param) sip_msg_t *req = param->req; dlg_cell_t *dlg = NULL; + if(req->first_line.u.request.method_value == METHOD_BYE) { + _dlg_ctx.t = 1; + return; + } + if(req->first_line.u.request.method_value != METHOD_INVITE) return; @@ -1303,6 +1308,9 @@ void dlg_onroute(struct sip_msg* req, str *route_params, void *param) dlg_unref(dlg, unref); + _dlg_ctx.cpid = my_pid(); + _dlg_ctx.expect_t = 1; + if_update_stat( dlg_enable_stats, active_dlgs, -1); goto done; } diff --git a/modules/dialog/dlg_var.c b/modules/dialog/dlg_var.c index 5d82777..c61cb7f 100644 --- a/modules/dialog/dlg_var.c +++ b/modules/dialog/dlg_var.c @@ -45,11 +45,18 @@ int dlg_cfg_cb(sip_msg_t *msg, unsigned int flags, void *cbp) if(flags&POST_SCRIPT_CB) { dlg = dlg_get_ctx_dialog(); if(dlg!=NULL) { - if(_dlg_ctx.t==0 && dlg->state==DLG_STATE_UNCONFIRMED) { + if(_dlg_ctx.t==0 && (dlg->state==DLG_STATE_UNCONFIRMED + || _dlg_ctx.expect_t==1)) { if(_dlg_ctx.cpid!=0 && _dlg_ctx.cpid==my_pid()) { /* release to destroy dialog if created by this process * and request was not forwarded */ - LM_DBG("new dialog with no trasaction after config execution\n"); + if(dlg->state==DLG_STATE_UNCONFIRMED) { + LM_DBG("new dialog with no trasaction after config" + " execution\n"); + } else { + LM_DBG("dialog with no expected trasaction after" + " config execution\n"); + } dlg_release(dlg); } } diff --git a/modules/dialog/dlg_var.h b/modules/dialog/dlg_var.h index bb604b7..a8b43cd 100644 --- a/modules/dialog/dlg_var.h +++ b/modules/dialog/dlg_var.h @@ -42,7 +42,8 @@ typedef struct _dlg_ctx { int cpid; int set; unsigned int dir; - int t; + int t; /* set to 1 if tm req in callback executed */ + int expect_t; /* set to 1 if expects that t is set after config */ } dlg_ctx_t; /* A dialog-variable */

1 0

git:master: utils/kamctl: removed unused default values
by Daniel-Constantin Mierla 16 Sep '14

16 Sep '14

Module: sip-router Branch: master Commit: 1735feed8e287e92c3d81cf6a4d873460f6ee878 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=1735fee… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Sep 16 18:13:02 2014 +0200 utils/kamctl: removed unused default values - test if DEFAULT_Q is set inside the function using it, because it has single place usage --- utils/kamctl/kamctl | 4 ++++ utils/kamctl/kamctlrc | 4 ---- utils/kamctl/kamdbctl.base | 6 ------ 3 files changed, 4 insertions(+), 10 deletions(-) diff --git a/utils/kamctl/kamctl b/utils/kamctl/kamctl index 8fa8ac2..bf12fbd 100755 --- a/utils/kamctl/kamctl +++ b/utils/kamctl/kamctl @@ -2454,6 +2454,10 @@ usrloc() { exit 1 fi + if [ -z "$DEFAULT_Q" ] ; then + DEFAULT_Q = "1.0" + fi + $CTLCMD ul_add "$USRLOC_TABLE" "$OSERUSER@$OSERDOMAIN" "$2" \ "$UL_EXPIRES" "$DEFAULT_Q" "$UL_PATH" "$UL_FLAGS" "$BR_FLAGS" "$ALL_METHODS" exit $? diff --git a/utils/kamctl/kamctlrc b/utils/kamctl/kamctlrc index 3ffad11..c3e0103 100644 --- a/utils/kamctl/kamctlrc +++ b/utils/kamctl/kamctlrc @@ -56,11 +56,7 @@ # FIXME # FOREVER="2030-05-28 21:32:15" -# DEFAULT_ALIASES_EXPIRES=$FOREVER # DEFAULT_Q="1.0" -# DEFAULT_CALLID="Default-Call-ID" -# DEFAULT_CSEQ="13" -# DEFAULT_LOCATION_EXPIRES=$FOREVER # Program to calculate a message-digest fingerprint diff --git a/utils/kamctl/kamdbctl.base b/utils/kamctl/kamdbctl.base index b4ed9ad..bb74810 100644 --- a/utils/kamctl/kamdbctl.base +++ b/utils/kamctl/kamdbctl.base @@ -54,12 +54,6 @@ DBUID_TABLES=${UID_TABLES:-uid_credentials uid_domain uid_domain_attrs # FIXME FOREVER=${FOREVER:-2030-05-28 21:32:15} -DEFAULT_ALIASES_EXPIRES=${DEFAULT_ALIASES_EXPIRES:-${FOREVER}} -DEFAULT_Q=${DEFAULT_Q:-1.0} -DEFAULT_CALLID=${DEFAULT_CALLID:-Default-Call-ID} -DEFAULT_CSEQ=${DEFAULT_CSEQ:-13} -DEFAULT_LOCATION_EXPIRES=${DEFAULT_LOCATION_EXPIRES:-${FOREVER}} - # default location for config files DEFAULT_CFG_DIR=/usr/local/etc/kamailio

1 0

git:master: rtpengine: fix encoding of "direction" flag
by Richard Fuchs 16 Sep '14

16 Sep '14

Module: sip-router Branch: master Commit: 640206f068ec5c52edd4c23399a99ef8a9e3b753 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=640206f… Author: Richard Fuchs <rfuchs(a)sipwise.com> Committer: Richard Fuchs <rfuchs(a)sipwise.com> Date: Tue Sep 16 10:41:43 2014 -0400 rtpengine: fix encoding of "direction" flag --- modules/rtpengine/rtpengine.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/modules/rtpengine/rtpengine.c b/modules/rtpengine/rtpengine.c index 063e3d1..b5960b2 100644 --- a/modules/rtpengine/rtpengine.c +++ b/modules/rtpengine/rtpengine.c @@ -1236,12 +1236,13 @@ static int parse_flags(struct ng_flags_parse *ng_flags, struct sip_msg *msg, enu break; case 9: - if (str_eq(&key, "RTP/SAVPF")) { + if (str_eq(&key, "RTP/SAVPF")) ng_flags->transport = 0x103; - goto next; - } else if (str_eq(&key, "direction")) bencode_list_add_str(ng_flags->direction, &val); + else + goto generic; + goto next; break; case 10:

1 0

git:master: core: notice message about exports dlflags made debug
by Daniel-Constantin Mierla 16 Sep '14

16 Sep '14

Module: sip-router Branch: master Commit: a78764174a893131124852490f47239051892139 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a787641… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Tue Sep 16 09:27:05 2014 +0200 core: notice message about exports dlflags made debug - it is anyhow for developers, not admins --- sr_module.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/sr_module.c b/sr_module.c index 860acb1..b581635 100644 --- a/sr_module.c +++ b/sr_module.c @@ -650,7 +650,7 @@ reload: if (new_dlflags!=dlflags && new_dlflags!=DEFAULT_DLFLAGS) { /* we have to reload the module */ dlclose(handle); - NOTICE("%s: exports dlflags interface is deprecated and it will not" + DEBUG("%s: exports dlflags interface is deprecated and it will not" " be supported in newer versions; consider using" " mod_register() instead\n", path); dlflags=new_dlflags;

1 0

[tracker] Comment added: zombie dialogs in state 5
by sip-router 15 Sep '14

15 Sep '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#424 - zombie dialogs in state 5 User who did this - Ovidiu Sas (osas) ---------- We have two scenarios here: 1. a request retransmission for a transaction that is already created (and this is working fine now); 2. a request re-transmission for which a transaction hasn't been created yet (and this is the troubled case). In the second case, the callbacks set by the initial request are lost (or at least this was my interpretation by looking at the logs). The second request retransmission seems to invalidate the tm callbacks that would normally be set by the initial request. In most of the cases there's no need to manually create a transaction (t_relay does that automatically) so there will be situation when this will happen. For this situation, either we fix this case properly or we add some logs with hints about the need to manually create the transaction to avoid loosing the tm callbacks. I hope that this explains better what I was trying to say in my previous post :) ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=424#comment1626 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Comment added: zombie dialogs in state 5
by sip-router 15 Sep '14

15 Sep '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#424 - zombie dialogs in state 5 User who did this - Daniel-Constantin Mierla (miconda) ---------- What do you mean by: "If we want to just enforce the workaround (pre-create the transaction) it would be helpful to somehow notify in the logs that callbacks registration will be lost." ? Specific to this case or in general? This situation could be seen also as the common case of any retransmission coming fast enough that previous request doesn't get to a transaction. Normally, same processing is happening for both requests, the second will be dropped by tm at relay time. In such scenarios all should go fine, because both request should have same set of callbacks set independently, no matter which one gets through first to create the transaction. So here is more a dialog problem that skips processing the second bye internally because of no change to dlg state. There can be added a log message to print when the retransmission is detected that the process has callbacks in the local list, which will be no longer linked to transaction. But that could fill the log files, with logs for each late-detected retransmission. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=424#comment1625 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Comment added: zombie dialogs in state 5
by sip-router 15 Sep '14

15 Sep '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#424 - zombie dialogs in state 5 User who did this - Ovidiu Sas (osas) ---------- As I said in my previous post, if the transaction is created right after the BYE is received (using t_newtran()), then it works ok. So yes, the workaround is to pre-create the transaction before doing any blocking operations. The issue here is that the tm callbacks are lost when the transaction is not pre-created. And this is an issue that will affect all modules that are using tm callbacks. There are no ERROR logs to show that callbacks have been lost and an admin will have a hard time figuring out what's going on when weird things are happening. If we want to just enforce the workaraound (pre-create the transaction) it would be helpful to somehow notify in the logs that callbacks registration will be lost. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=424#comment1624 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Comment added: zombie dialogs in state 5
by sip-router 15 Sep '14

15 Sep '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#424 - zombie dialogs in state 5 User who did this - Daniel-Constantin Mierla (miconda) ---------- I think one safe solution is to create the transaction if not created for BYE. Looking up to see if there is a transaction after executing the event route could find one in a too late state to get appropriate callbacks still execution (e.g, if execution of event route takes too long, transaction can be in wait-for-delete state). Alternative, run the dialog code for BYE under dialog lock, so a later BYE cannot overtake a previous one. Or, if a BYE comes in a state 5, drop it from inside dialog module... ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=424#comment1623 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

← Newer
1
...
11
12
13
14
15
16
17
...
38
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev September 2014