<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/1323/">https://reviewboard.asterisk.org/r/1323/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On July 20th, 2011, 11:23 a.m., <b>rmudgett</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 don&#39;t understand the scenario that causes the problem.
Has someone else answered the call from the queue?
Has the pickup attempt failed?</pre>
 </blockquote>




 <p>On July 20th, 2011, 11:35 a.m., <b>irroot</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;">app_queue places a call to perhaps more than 1 interface the caller hangs up in a race with a pickup of
the called interface from a 3 party [as best i can tell].

the result is a channel left orphaned that cannot be removed with no thread running on it this is happening frequently enough over various sites.

when the &quot;pickup&quot; is done the channel should be bridged or about to be bridged running autoservice on it till this happens is probably the correct thing to do WRT DTMF and the like and will allow the hangup frame to flush it.

 </pre>
 </blockquote>





 <p>On July 20th, 2011, 12:59 p.m., <b>rmudgett</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 think waiting for a bridge that may never happen is not the way to go here.  It sounds more like a problem in app_queue.c:wait_for_answer() not cleaning up correctly in a race between a queue member answering the call and the caller hanging up.</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;">i have only witnessed it in app_queue i suspect it may happen elsewhere ill look into app_queue further</pre>
<br />








<p>- irroot</p>


<br />
<p>On July 20th, 2011, 8:28 a.m., irroot wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Asterisk Developers.</div>
<div>By irroot.</div>


<p style="color: grey;"><i>Updated July 20, 2011, 8:28 a.m.</i></p>




<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;">seems like app_queue [and others likely] can leave a orphaned channel with no bridge and no thread running on it.

this is &quot;ungood&quot; here is a small patch that starts auto service waits for bridging and turns it off allowing the 
hangup frame to get through and shut down the channel ...  </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>/team/irroot/distrotech-customers-1.8/main/features.c <span style="color: grey">(328930)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/1323/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>