Hello Everyone, Daniel,
This is directly related to the previous emails on this subject, sent by Guillaume. I am adresing here the exact same issue, we work together.
The following piece of code is supposed to disconnect the previous call placed by a user:
We save call-id,totag (if we have 180,183),fromtag,cseq of the INVITE of the previous dialog in memcache and we retrieve them for each call the user is placing.
if ($avp(saved_totag_of_previous_call)!=$null){ # the above check should be equivalent with hastotag() on the other call . if(dlg_get("$avp(saved_previous_call_callid)", "$avp(saved_totag_of_previous_call)", "$avp(saved_fromtag_of_previous_call)"))
{ # sending BYEs to the previous dialog originated by this user if (dlg_bye("all")) { xlog("L_INFO","BYE sent to caller&callee");
} else { xlog("L_INFO","could not send locally generated BYEs\n");
} dlg_get("$ci","$tt","$ft"); # this BYE will be caught in event_route[tm:local-request] where we do some accounting } else { # # dialog should be in early state - eg: 183 or 180 has been received
xlog("dlg_get() failed - because the dialog was not confirmed\n"); if (t_cancel_callid("$avp(saved_previous_call_callid)", "$avp(saved_previous_call_invite_cseq)", "22")) { xlog("maxcall:transaction cancelled - i am in call with call-id: $ci \n"); # do stuff here } else {
xlog("maxcall:cannot send forced cancel - the previous dialog had a totag - early state \n"); xlog("maxcall:i am in call with call-id: $ci \n"); }
# do stuff here } else {
# dialog should be in preceeding state - `100 Trying` was received (`100 Trying` does not have a totag) if (t_cancel_callid("$avp(saved_previous_call_callid)", "$avp(saved_previous_call_invite_cseq)", "22")) { xlog("maxcall:transaction cancelled . i am in call with call-id: $ci \n"); # do stuff here
} else { xlog("maxcall:cannot send forced cancel - the previous dialog did not have a totag - preceeding state\n"); xlog("maxcall:I am in call with call-id: $ci \n"); } }
We see this in the logs, for two quick successive calls from the same UA (same user) so we could reach the condition when t_cancel_callid() is called , with $avp(saved_totag_of_previous_call) being null :
So this is when the the user makes the second call:
0(21429) INFO: <script>: cancelling call 3e20b1db528ce041 (entry= 2776, id= 2165 ) 0(21429) DEBUG: tm [t_lookup.c:1742]: t_lookup_callid(): created comparable call_id header field: >Call-ID: 3e20b1db528ce0$ < 0(21429) DEBUG: tm [t_lookup.c:1746]: t_lookup_callid(): created comparable cseq header field: >CSeq: 20 INVITE< 0(21429) DEBUG: tm [t_lookup.c:1749]: t_lookup_callid(): just locked hash index 57175, looking for transactions there: 0(21429) DEBUG: tm [t_lookup.c:1762]: t_lookup_callid(): we have a match: callid=>>Call-ID: 3e20b1db528ce041 << cseq=>>CSeq: 20<< 0(21429) DEBUG: tm [t_lookup.c:1767]: t_lookup_callid(): DEBUG: t_lookup_callid: transaction found. 0(21429) DEBUG: tmx [tmx_mod.c:364]: t_cancel_callid(): Now calling cancel_uacs 0(21429) DEBUG: tm [t_cancel.c:329]: cancel_branch(): DEBUG: cancel_branch: sending cancel... 0(21429) ERROR: <script>: maxcall:transaction cancelled . i am in call with call-id: 2bf9efb7528ce04d # <- this is our custom logging 0(21429) ERROR: <script>: the other call (previous) had call-id: 3e20b1db528ce041
[...]
then:
0(21429) DEBUG: dialog [dlg_hash.c:600]: dlg_lookup(): ref dlg 0x7f4179801d08 with 1 -> 2 0(21429) DEBUG: dialog [dlg_hash.c:602]: dlg_lookup(): dialog id=9721 found on entry 2505 0(21429) DEBUG: dialog [dlg_var.c:49]: dlg_cfg_cb(): new dialog with no trasaction after config execution 0(21429) DEBUG: dialog [dlg_hash.c:765]: dlg_unref(): unref dlg 0x7f4179801d08 with 1 -> 1 0(21429) DEBUG: dialog [dlg_hash.c:765]: dlg_unref(): unref dlg 0x7f4179801d08 with 1 -> 0 0(21429) DEBUG: dialog [dlg_hash.c:765]: dlg_unref(): ref <=0 for dialog 0x7f4179801d08 0(21429) DEBUG: dialog [dlg_hash.c:316]: destroy_dlg(): destroying dialog 0x7f4179801d08 (ref 0) 0(21429) DEBUG: dialog [dlg_hash.c:332]: destroy_dlg(): removed timer for dlg 0x7f4179801d08 [2505:9721] with clid '2bf9efb7528ce04d' and tags '76bc4edf528ce04d' ''
What happens is that kamailio seems that it destroys the second dialog , instead of the first one. This leads to other inconsitency :
DEBUG: dialog [dlg_hash.c:664]: internal_get_dlg(): no dialog callid='2bf9efb7528ce04d' found # <- it cannot match the ACK for the second call.
[...]
0(21429) WARNING: dialog [dlg_handlers.c:1127]: dlg_onroute(): unable to find dialog for BYE with route param '9c9.9f52' [2505:9721] # <- it cannot match the BYE for the second call 0(21429) DEBUG: dialog [dlg_hash.c:664]: internal_get_dlg(): no dialog callid='2bf9efb7528ce04d' found 0(21429) DEBUG: dialog [dlg_hash.c:695]: get_dlg(): no dialog callid='2bf9efb7528ce04d' found 0(21429) DEBUG: dialog [dlg_handlers.c:1173]: dlg_onroute(): Callid '2bf9efb7528ce04d' not found
[...]
1(21430) INFO: <script>: event_route[local-request]: call-id=3e20b1db528ce041 from=15909901 to=+33630815276 : local_route (method= BYE) :: dlg flag: 1 1(21430) DEBUG: pv [pv_trans.c:315]: tr_eval_string(): i=0 j=1 0(21429) WARNING: dialog [dlg_req_within.c:177]: bye_reply_cb(): inconsitent dlg timer data on dlg 0x7f41797e7048 [2776:2165] with clid '3e20b1db528ce041' and tags '522612bb528ce041' 'as1b2c7ace' 0(21429) DEBUG: dialog [dlg_hash.c:332]: destroy_dlg(): removed timer for dlg 0x7f41797e7048 [2776:2165] with clid '3e20b1db528ce041' and tags '522612bb528ce041' 'as1b2c7ace'
It it also mixing the timeout of the first dialog with the one of the second. If we put a default timeout of 15 seconds for all calls and we cancel the first call,kamailio will generate a BYEs in event_route[tm:local-request] after 15 seconds after the call other call was answered, but with the call-id of the first call - it is mixing dialogs as we can see with kamctl.
We suspect that t_cancel_callid() is not destroying the proper dialog at the right time. We use dlg_manage() on every INVITE.
It is the same behaviour with kamailio 4.0.2 and kamailio 4.0.3 .
Any hints / ideas about this problem ? Is the problem in the dialog module, or t_cancel_callid() is doing something wrong ?
Thank you.
Regards, Dragos
On Wednesday, November 20, 2013 10:57 PM, Daniel-Constantin Mierla miconda@gmail.com wrote:
Hello,
are you dropping replies? I don't see the 'SIP/2.0 487 Request Terminated' being sent to caller, it looks ok and has two Via headers.
Cheers, Daniel
On 11/20/13 4:06 PM, Guillaume Bour wrote:
On 20/11/2013 12:01, Daniel-Constantin Mierla wrote:
Hello,
On 11/20/13 11:50 AM, Guillaume Bour wrote:
Hi All
We wan't to prevent our users to make more than one call at time, so we choose to disconnect the previous call. When the previous call is established, we use dlg_bye(), and its ok. But when it is in early state, we use t_cancel_callid() to cancel its INVITE transaction.
We face 2 issues: 1) we use local-request event route to account calls on timeout. Sometimes this route is called for the cancelled call (after default timeout of 1 hour)
what is in the local-request in this case? Is it a BYE?
2) t_cancel_callid() cancel previous call, but also make current dialog disappear: call is still ongoing and we can answer and talk to each other, but the dialog does not appear in 'kamctl stats dialog' and 'kamctl mi dlg_list' commands
Is there a known limitation, or do we misuse t_cancel_callid()
?
Can you send the log with debug=3 in kamailio.cfg? It will help to see what happens. Otherwise, if the call id is different for current dialog, it should not happen. The ngrep output in this situation (for both first and second invite) will help.
Cheers, Daniel
Hi Daniel,
local-request is triggered by a BYE I have attached sample log and trace
There is some kind of dialogs mixing. Here is the 1st call dialog as reported by "kamctl mi dlg_list" before and after the 2d call is answered:
# kamctl mi dlg_list dialog:: hash=2790:3231 state:: 2 ref_count:: 1 timestart:: 0 timeout:: 0 callid:: GoXhk8GfkIEEqFyFNcySEjSOOpVKg4Uq from_uri:: sip:15909901@staging.voip from_tag:: swYh88AkicGbSHpK.D1z7uo3EX9Q-.AZ caller_contact:: sip:37984520-gch2kindtioq8@10.0.1.10:5060;transport=udp caller_cseq:: 24899 caller_route_set:: caller_bind_addr:: udp:10.0.1.10:5060 callee_bind_addr:: to_uri:: sip:+3360000011@staging.voip to_tag:: callee_contact:: callee_cseq:: callee_route_set::
# kamctl mi dlg_list dialog:: hash=2790:3231 state:: 3 ref_count:: 2 timestart:: 1384952191 timeout:: 20242152 callid:: GoXhk8GfkIEEqFyFNcySEjSOOpVKg4Uq from_uri:: sip:15909901@staging.voip from_tag:: swYh88AkicGbSHpK.D1z7uo3EX9Q-.AZ caller_contact:: sip:37984520-gch2kindtioq8@10.0.1.20:5060;transport=udp caller_cseq:: 24899 caller_route_set:: caller_bind_addr:: udp:10.0.1.10:5060 callee_bind_addr:: udp:10.0.1.10:5060 to_uri:: sip:+3360000011@staging.voip to_tag:: as6c8b935a callee_contact:: sip:+3360000022@10.0.1.11:5060 callee_cseq:: callee_route_set::
_______________________________________________ SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list sr-users@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users