<p>Joshua Colp <strong>uploaded patch set #3</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/5800">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">bridge: Add a deferred queue.<br><br>This change adds a deferred queue to bridging. If a bridge<br>technology determines that a frame can not be written and<br>should be deferred it can indicate back to bridging to do so.<br>Bridging will then requeue any deferred frames upon a new<br>channel joining the bridge.<br><br>This change has been leveraged for T.38 request negotiate<br>control frames. Without the deferred queue there is a race<br>condition between the bridge receiving the T.38 request<br>negotiate and the second channel joining and being in the<br>bridge. If the channel is not yet in the bridge then the T.38<br>negotiation fails.<br><br>A unit test has also been added that confirms that a T.38<br>request negotiate control frame is deferred when no other<br>channel is in the bridge and that it is requeued when a new<br>channel joins the bridge.<br><br>ASTERISK-26923<br><br>Change-Id: Ie05b08523f399eae579130f4a5f562a344d2e415<br>---<br>M bridges/bridge_native_rtp.c<br>M bridges/bridge_simple.c<br>M include/asterisk/bridge_channel.h<br>M include/asterisk/bridge_channel_internal.h<br>M include/asterisk/bridge_technology.h<br>M main/bridge.c<br>M main/bridge_channel.c<br>A tests/test_bridging.c<br>8 files changed, 397 insertions(+), 10 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/00/5800/3</pre><p>To view, visit <a href="https://gerrit.asterisk.org/5800">change 5800</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/5800"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 14 </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Ie05b08523f399eae579130f4a5f562a344d2e415 </div>
<div style="display:none"> Gerrit-Change-Number: 5800 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Mark Michelson <mmichelson@digium.com> </div>