<p>Richard Mudgett <strong>uploaded patch set #5</strong> to the change originally created by Robert Mordec.</p><p><a href="https://gerrit.asterisk.org/9026">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">app_confbridge: Bridge and announcers not removed if conference ends quickly<br><br>If a conference is ended very quickly after it was created (i.e., the<br>first user immediately hangs up) then the conference bridge and announcer<br>channels are not removed.<br><br>When a conference is created, the push_announcer() function is added to<br>the playback queue task processor and the conference object reference is<br>bumped.  If a conference is ended while the push_announcer() function is<br>still going then the ao2_cleanup(conference) at the end of<br>push_announcer() will call the destructor function -<br>destroy_conference_bridge().<br><br>The destroy_conference_bridge() function will then add the<br>hangup_playback() task to the playback queue and will wait for it to end.<br>Since it is already a current task of the playback queue it will wait<br>forever.<br><br>This patch makes the conference thread call push_announcer() directly.<br>This way the conference object reference bump is not needed.  Since the<br>playback queue task processor is only used by the conference thread<br>itself, there is no danger of trying to play announcements before the<br>announcer is pushed to the bridge.<br><br>ASTERISK-27870 #close<br><br>Change-Id: I947a50fb121422d90fd1816d643a54d75185a477<br>---<br>M apps/app_confbridge.c<br>1 file changed, 3 insertions(+), 9 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/26/9026/5</pre><p>To view, visit <a href="https://gerrit.asterisk.org/9026">change 9026</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/9026"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I947a50fb121422d90fd1816d643a54d75185a477 </div>
<div style="display:none"> Gerrit-Change-Number: 9026 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Robert Mordec <r.mordec@slican.pl> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Robert Mordec <r.mordec@slican.pl> </div>