restore_uri_reply() called twice in 200OK to BYE - sr-users

5 Nov 2015

Hi Folks,

I have a problem with To and From headers in 200 OK to BYE request. Their
URI is doubled when leaving Kamailio.

Incoming 200 OK:
  From : <sip:00421111@IP_ADDRESS>;tag=as017a2986
  To : <sip:00421222@IP_ADDRESS0>;tag=K7ryDgH2g7gFm
Outgoing 200 OK:
  From : <sip:+421111@IP_ADDRESS><sip:+421111@IP_ADDRESS>;tag=as017a2986
  To : <sip:+421222@IP_ADDRESS><sip:+421222@IP_ADDRESS>;tag=K7ryDgH2g7gFm

I'm using uac module to replace these headers in auto and dialog mode:

modparam("uac", "restore_mode", "auto")
modparam("uac", "restore_dlg", 1)

append_fromtag is also set:

modparam("rr", "append_fromtag", 1)

When I run kamailio in debug mode I can see the function
restore_uri_reply() is called twice:

.
.
.
 0(3133) DEBUG: <core> [parser/parse_via.c:1284]: parse_via_param(): Found
param type 234, <received> = <IP_ADDRESS>; state=6
 0(3133) DEBUG: <core> [parser/parse_via.c:1284]: parse_via_param(): Found
param type 232, <branch> = <z9hG4bK0bb8c5b2>; state=6
 0(3133) DEBUG: <core> [parser/parse_via.c:1284]: parse_via_param(): Found
param type 235, <rport> = <5060>; state=16
 0(3133) DEBUG: <core> [parser/parse_via.c:2672]: parse_via(): end of
header reached, state=5
 0(3133) DEBUG: <core> [parser/msg_parser.c:513]: parse_headers():
parse_headers: Via found, flags=62
 0(3133) DEBUG: <core> [parser/msg_parser.c:526]: parse_headers():
parse_headers: this is the second via
 0(3133) DEBUG: <core> [parser/parse_addr_spec.c:176]: parse_to_param():
DEBUG: add_param: tag=teHD72X67aQUm
 0(3133) DEBUG: <core> [parser/parse_addr_spec.c:898]: parse_addr_spec():
end of header reached, state=29
 0(3133) DEBUG: <core> [parser/msg_parser.c:190]: get_hdr_field(): DEBUG:
get_hdr_field: <To> [60]; uri=[sip:421222@IP_ADDRESS:5060]
 0(3133) DEBUG: <core> [parser/msg_parser.c:192]: get_hdr_field(): DEBUG:
to body [<sip:421222@IP_ADDRESS:5060>]
 0(3133) DEBUG: <core> [parser/msg_parser.c:170]: get_hdr_field():
get_hdr_field: cseq <CSeq>: <103> <BYE>
 0(3133) DEBUG: tm [t_lookup.c:949]: t_reply_matching(): DEBUG:
t_reply_matching: hash 10806 label 0 branch 0
 0(3133) DEBUG: tm [t_lookup.c:1004]: t_reply_matching(): DEBUG:
t_reply_matching: reply matched (T=0x220071b4)!
 0(3133) DEBUG: tm [t_hooks.c:288]: run_trans_callbacks_internal(): DBG:
trans=0x220071b4, callback type 2, id 0 entered
 0(3133) DEBUG: <core> [parser/parse_addr_spec.c:176]: parse_to_param():
DEBUG: add_param: tag=as46bccb83
 0(3133) DEBUG: <core> [parser/parse_addr_spec.c:898]: parse_addr_spec():
end of header reached, state=29
 0(3133) DEBUG: uac [replace.c:705]: restore_uri_reply(): removing
<<sip:00421111@IP_ADDRESS>>
 0(3133) DEBUG: uac [replace.c:714]: restore_uri_reply(): inserting
<<sip:+421111@IP_ADDRESS>>
 0(3133) DEBUG: uac [replace.c:705]: restore_uri_reply(): removing
<<sip:00421222@IP_ADDRESS>>
 0(3133) DEBUG: uac [replace.c:714]: restore_uri_reply(): inserting
<<sip:+421222@IP_ADDRESS>>
 0(3133) DEBUG: tm [t_hooks.c:288]: run_trans_callbacks_internal(): DBG:
trans=0x220071b4, callback type 2, id 0 entered
 0(3133) DEBUG: uac [replace.c:705]: restore_uri_reply(): removing
<<sip:00421111@IP_ADDRESS>>
 0(3133) DEBUG: uac [replace.c:714]: restore_uri_reply(): inserting
<<sip:+421111@IP_ADDRESS>>
 0(3133) DEBUG: uac [replace.c:705]: restore_uri_reply(): removing
<<sip:00421222@IP_ADDRESS>>
 0(3133) DEBUG: uac [replace.c:714]: restore_uri_reply(): inserting
<<sip:+421222@IP_ADDRESS>>
.
.
.

I think the problem is in registering a function restore_uris_reply() in
callbacks in functions (both in replace.c of uac module)
* void rr_checker(struct sip_msg *msg, str *r_param, void *cb_param)
* static void replace_callback(struct dlg_cell *dlg, int type, struct
dlg_cb_params *_params)

Both of these callbacks are registered for BYE and ACK requests (INVITE
request is using different callback), therefore running restore_uri_reply()
twice.
For testing purposes after I have removed the callback in rr_checker and
recompile the uac module, To and From headers were OK.

In my configuration I'm calling uac_replace_to in RELAY route:
route[RELAY]
{

        if ( !has_totag() ) {
                dlg_manage();
                uac_replace_to("$ru");
        }

        if (!t_relay())
        {
                sl_reply_error();
        }
}

So I think there is a bug in uac module, when one function is registered
twice in callbacks for NON-INVITE requests.

Or is there a problem with my configuration script?

Can anyone point me to the direction how to solve this issue?

Thank you

Efelin

Kamailio version: kamailio 4.4.0-dev6 (i386/linux) a66e22-dirty