[SR-Users] Proposal for dialog: DLG_STATE_EARLY
Klaus Darilion
klaus.mailinglists at pernau.at
Mon Jun 24 13:51:29 CEST 2013
On 21.06.2013 19:20, Daniel-Constantin Mierla wrote:
> Hello,
>
> I have some comments related to the patches, as I couldn't dig much into
> sources due to traveling constraints. See them inline.
>
> On 6/14/13 2:10 PM, Halina Nowak wrote:
>> These modifications were implementated for dialogs having PRACK and UPDATE
>>
>> --- a/modules/dialog/dlg_handlers.c Fri Jun 14 13:45:41 2013 +0200
>> +++ b/modules/dialog/dlg_handlers.c Fri Jun 14 13:55:24 2013 +0200
>> @@ -1249,16 +1249,18 @@
>> }
>>
>> if ( (event==DLG_EVENT_REQ || event==DLG_EVENT_REQACK)
>> - && new_state==DLG_STATE_CONFIRMED) {
>> + && (new_state==DLG_STATE_CONFIRMED ||
>> new_state==DLG_STATE_EARLY)) {
>
> This above is to catch PRACK, right? UPDATE should be sent after 200ok,
> or is allowed also for early dialogs?
Just answering from a protocol point of view, without code review:
The main purpose of UPDATE is to update the sessions although there is a
pending INVITE transaction. Thus, yes, UPDATE can be sent although there
was no 200 OK to the INVITE yet.
regards
Klaus
More information about the sr-users
mailing list