<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/3109/">https://reviewboard.asterisk.org/r/3109/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 7th, 2014, 6:19 p.m. CST, <b>Mark Michelson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I'm curious, has the reporter of ASTERISK-22946 confirmed that this patch fixes his issue? While this may be fixing the issue of ambiguous sip_pvts during quick re-registrations, I don't think that this is actually what is happening in the reporter's case. In the reporter's packet capture, the re-registration occurs 105 seconds after the initial REGISTER, so I would expect that the old sip_pvt should have been destroyed by the time the new REGISTER is being sent out.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">In their pcap, the repeated re-register (about every second after 105) was matching my own replicated test results, even though I had to reduce the expiry to 30 seconds to get there. This is definitely fixing that portion at least of the issue, but I will go ahead and send them a patch and ask them to check their results.
And they responded:
Stephan Eisvogel commented on ASTERISK-22946:
---------------------------------------------
Tested for 20 hours on latest 11-SVN, problem is gone. Thanks!
I suggest more testing by 2-3 others, though, before commit.</pre>
<br />
<p>- Scott</p>
<br />
<p>On January 7th, 2014, 1:04 p.m. CST, Scott Griepentrog wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers.</div>
<div>By Scott Griepentrog.</div>
<p style="color: grey;"><i>Updated Jan. 7, 2014, 1:04 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/ASTERISK-22946">ASTERISK-22946</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
Asterisk
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/branches/1.8/channels/chan_sip.c <span style="color: grey">(405032)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/3109/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>