[asterisk-dev] [Code Review] 3109: chan_sip: Local From tag regression fixed by eliminating dialog after 200 OK

[Matt Jordan]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3109/#review10591
-----------------------------------------------------------

Ship it!


Ship It!

- Matt Jordan


On Jan. 7, 2014, 1:04 p.m., Scott Griepentrog wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3109/
> -----------------------------------------------------------
> 
> (Updated Jan. 7, 2014, 1:04 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-22946
>     https://issues.asterisk.org/jira/browse/ASTERISK-22946
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> In ASTERISK-12117 an improvement to retain the same local from tag resulted in an undesirable behavior when registrations expire quickly (i.e. every 30 seconds).  The transmit_register() function creates a new dialog (sip_pvt) for every register attempt.  However, with the local tag no longer changing, a previous register dialog could still be matched if it had not timed out and been removed prior to the next register attempt.  When the old register dialog was found, it had the wrong sequence number and would cause repeated retransmits on timeout.  Rather than a significant rework of transmit_register(), this patch eliminates the problem by destroying the old dialog immediately after the 200 OK is received indicating a successful registration.
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/channels/chan_sip.c 405032 
> 
> Diff: https://reviewboard.asterisk.org/r/3109/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Scott Griepentrog
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140114/2e0273d0/attachment.html>