<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/2187/">https://reviewboard.asterisk.org/r/2187/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<p>- Mark</p>
<br />
<p>On November 9th, 2012, 4:34 p.m., jrose 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 and Mark Michelson.</div>
<div>By jrose.</div>
<p style="color: grey;"><i>Updated Nov. 9, 2012, 4:34 p.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;">Certain conditions result in a hangup while in Meetme's confrun function's main loop result in the loop getting stuck because ast_waitfor_nandfds is unable to get a channel to execute normal break conditions. Russell removed a hangup check which accounted for these cases before since it seemed unnecessary, but it seems it is.
There are also a couple instances where multiple calls to ast_streamfile can be called in succession without error or hangup checking, and this can cause frame write warnings. Some of these are addressed by keeping menu switches from going to the default exit condition when dtmf = -1 because a caller hung up when DTMF was expected. In general though, this patch doesn't really aim to address those warnings and a few can still be caused with hangups during certain options that cause sound files to be played in succession such as rollcall announcement on entry.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">I tested this patch against a couple situations where I was aware I could cause the looping warning messages to occur reliably... notably hanging up while sound files were being played such as conf-onlyperson when first entering the conference. I also submitted the patch for reporters of these issues to test with. One has gotten back to me so far with confirmation that this patch addresses his problem.</pre>
</td>
</tr>
</table>
<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-20486">ASTERISK-20486</a>
</div>
<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/apps/app_meetme.c <span style="color: grey">(375992)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2187/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>