<div>
<style type="text/css">
.successful a, .successful a:visited, .successful a:link, .successful a:hover,.successful a:active {color:#393}
.failed a, .failed a:visited, .failed a:link, .failed a:hover,.failed a:active {color:#d62829}
td a, td a:link, td a:visited, td a:hover, td a:active {background:transparent;font-family: Arial, sans-serif;text-decoration:underline;}
td a:link {color:#369;}
td a:visited {color:#444;}
td a:hover, td a:active {color:#036;}
td a:hover {text-decoration:none;}
</style>
<font size="2" color="black" face="Arial, Helvetica, sans-serif" style="font-family: Arial, sans-serif;font-size: 13px;color:#000">
<table align="center" border="0" cellpadding="5" cellspacing="0" width="98%">
<tr>
<td style="vertical-align:top">
<table width="100%" border="0" cellpadding="0" cellspacing="0" style="background-color:#e4f5e3;border-top:1px solid #b4e2b4;border-bottom:1px solid #b4e2b4;color:#393;">
<tr>
<td width="20" style="vertical-align:top;padding:5px 0 5px 10px">
<img src="http://bamboo.asterisk.org/images/jt/icn_plan_passed.gif" width="15" height="15">
</td>
<td width="100%" style="font-family: Arial, sans-serif; font-size: 13px; color:#393;padding:5px 10px">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/" style="font-family: Arial, sans-serif; font-size: 15px; font-weight:bold; color:#393">AST-162-160</a>
<span class="successful" style="font-family: Arial, sans-serif; font-size: 14px;"> was successful.</span>
<span class="successful" style="font-family: Arial, sans-serif; font-size: 13px;">
<br/> This build occured because it is a dependant of <a href="http://bamboo.asterisk.org/browse/AST-TESTSUITE-68">AST-TESTSUITE-68</a>.
<br/><strong>3</strong> tests in total.
</span>
</td>
</tr>
</table>
<br>
<table width="100%" border="0" cellpadding="0" cellspacing="0" style="background-color:#ecf1f7;border-top:1px solid #bbd0e5;border-bottom:1px solid #bbd0e5;color:#036;">
<tr>
<td width="60%" style="font-family: Arial, sans-serif;text-align:left;font-size:16px;font-weight:bold;color:#036;padding:5px 10px">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/commit" style="text-decoration: none; font-family: Arial, sans-serif;text-align:left;font-size:16px;font-weight:bold;color:#036" >Code Changes</a>
</td>
<td width="40%" style="font-family: Arial, sans-serif;text-align:right;font-size:13px;color:#036;padding:5px 10px">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/commit" style="font-family: Arial, sans-serif; font-size: 13px; color:#036">See full change details</a><img src="http://bamboo.asterisk.org/images/icons/more.gif" width="15" height="15" style="vertical-align:middle;padding:2px">
</td>
</tr>
</table>
<table width="100%" border="0" cellpadding="0" cellspacing="0">
<tr><td width="20" style="vertical-align:top;padding:10px 0 0px 10px">
<img src="http://bamboo.asterisk.org/images/icons/businessman.gif" width="15" height="15">
</td>
<td width="100%" style="font-family: Arial, sans-serif; font-size: 13px; color:#000;vertical-align:top;padding:10px 10px 0px 10px">
<a href="http://bamboo.asterisk.org/browse/author/mmichelson" style="font-family: Arial, sans-serif; font-size: 13px; font-weight:bold; color:#000">
mmichelson</a><br>
Recorded merge of revisions 254454 via svnmerge from <br/>
<a href="https://origsvn.digium.com/svn/asterisk/trunk">https://origsvn.digium.com/svn/asterisk/trunk</a><br/>
<br/>
................<br/>
r254454 | mmichelson | 2010-03-25 11:04:48 -0500 (Thu, 25 Mar 2010) | 50 lines<br/>
<br/>
Recorded merge of revisions 254452 via svnmerge from <br/>
<a href="https://origsvn.digium.com/svn/asterisk/branches/1.4">https://origsvn.digium.com/svn/asterisk/branches/1.4</a><br/>
<br/>
........<br/>
r254452 | mmichelson | 2010-03-25 10:59:56 -0500 (Thu, 25 Mar 2010) | 44 lines<br/>
<br/>
Several fixes regarding RFC2833 DTMF detection.<br/>
<br/>
Here is a copy and paste of the details from my request on<br/>
reviewboard that dealt with these changes:<br/>
<br/>
Fix 1. The first change in place is to fix Mantis issue 15811, which deals with a situation where Asterisk will incorrectly interpret out of order RFC2833 frames as duplicate DTMF digits. For instance, we would receive a sequence like:<br/>
<br/>
seqno 1: DTMF 1<br/>
seqno 2: DTMF 1<br/>
seqno 3: DTMF 1<br/>
seqno 4: DTMF 1<br/>
seqno 6: DTMF 1 (end)<br/>
seqno 5: DTMF 1<br/>
seqno 7: DTMF 1 (end)<br/>
seqno 8: DTMF 1 (end)<br/>
<br/>
Prior to this patch when we received the frame with seqno 5, we would interpret this as a new DTMF 1. With this patch, we will check the seqno of the incoming digit and not process the frame if the seqno is lower than the last recorded seqno. Note that we do not record the seqno of the dropped DTMF frame for future processing. While the above situation is what was designed to be fixed, the patch is written in such a way that the following would also be fixed too:<br/>
<br/>
seqno 9: DTMF 1<br/>
seqno 10: DTMF 1 (end)<br/>
seqno 11: DTMF 1 (end)<br/>
seqno 13: DTMF 2<br/>
seqno 12: DTMF 1 (end)<br/>
seqno 14: DTMF 2<br/>
seqno 15: DTMF 2 (end)<br/>
seqno 16: DTMF 2 (end)<br/>
seqno 17: DTMF 2 (end)<br/>
<br/>
In this second situation, the beginning of the DTMF 2 arrives before the final end frame of the DTMF 1. With the patch, seqno 12 is no processed and thus we properly interpret the DTMF.<br/>
<br/>
Fix 2. The second change in place is to fix an issue like the following:<br/>
<br/>
seqno 1: DTMF 1<br/>
seqno 2: DTMF 1<br/>
seqno 3: DTMF 1 (end) *packet lost*<br/>
seqno 4: DTMF 1 (end) *packet lost*<br/>
seqno 5: DTMF 1 (end) *packet lost*<br/>
seqno 6: DTMF 2<br/>
<br/>
When we receive seqno 6, we had code in place that was supposed to properly end the previously unended DTMF 1. The problem was that the code was essentially a no-op. The code would set up an end frame for the DTMF 1 but would immediately overwrite the frame with the begin for DTMF 2. I changed process_dtmf_rfc2833() so that instead of returning a single frame, it is given as an output parameter a list of frames. Each frame that needs to be returned is appended to this list.<br/>
<br/>
Fix 3. The final change is a minor one where an AST_CONTROL_SRCCHANGE frame could get lost. If we process a cisco DTMF or an RFC 3389 frame and no frame was returned, then we would return &ast_null_frame. The problem is that earlier in the function, we may have generated an AST_CONTROL_SRCCHANGE frame and put it in the list of frames we wish to return. This frame would be lost in such a case. The patch fixes this problem<br/>
........<br/>
................<br/>
</td>
<td width="60" style="font-family: Arial, sans-serif; font-size: 13px; ;color:#036;vertical-align:top;padding:10px 10px 0px 10px">
(254482)
</td></tr>
<tr><td width="20" style="vertical-align:top;padding:10px 0 0px 10px">
<img src="http://bamboo.asterisk.org/images/icons/businessman.gif" width="15" height="15">
</td>
<td width="100%" style="font-family: Arial, sans-serif; font-size: 13px; color:#000;vertical-align:top;padding:10px 10px 0px 10px">
<a href="http://bamboo.asterisk.org/browse/author/twilson" style="font-family: Arial, sans-serif; font-size: 13px; font-weight:bold; color:#000">
twilson</a><br>
Merged revisions 254453 via svnmerge from <br/>
<a href="https://origsvn.digium.com/svn/asterisk/trunk">https://origsvn.digium.com/svn/asterisk/trunk</a><br/>
<br/>
................<br/>
r254453 | twilson | 2010-03-25 11:03:51 -0500 (Thu, 25 Mar 2010) | 9 lines<br/>
<br/>
Merged revisions 254451 via svnmerge from <br/>
<a href="https://origsvn.digium.com/svn/asterisk/branches/1.4">https://origsvn.digium.com/svn/asterisk/branches/1.4</a><br/>
<br/>
........<br/>
r254451 | twilson | 2010-03-25 10:57:29 -0500 (Thu, 25 Mar 2010) | 2 lines<br/>
<br/>
Handle new SRCCHANGE control message here too<br/>
........<br/>
................<br/>
</td>
<td width="60" style="font-family: Arial, sans-serif; font-size: 13px; ;color:#036;vertical-align:top;padding:10px 10px 0px 10px">
(254499)
</td></tr>
</table><br>
<table width="100%" border="0" cellpadding="0" cellspacing="0" style="background-color:#ecf1f7;border-top:1px solid #bbd0e5;border-bottom:1px solid #bbd0e5;color:#036;">
<tr>
<td width="60%" style="font-family: Arial, sans-serif;text-align:left;font-size:16px;font-weight:bold;color:#036;vertical-align:top;padding:5px 10px">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/test" style="text-decoration: none; font-family: Arial, sans-serif;text-align:left;font-size:16px;font-weight:bold;color:#036" >Tests</a>
</td>
<td width="40%" style="font-family: Arial, sans-serif;text-align:right;font-size:13px;color:#036;padding:5px 10px">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/test" style="font-family: Arial, sans-serif; font-size: 13px;color:#036">See full test details</a><img src="http://bamboo.asterisk.org/images/icons/more.gif" width="15" height="15" style="vertical-align:middle;padding:2px">
</td>
</tr>
</table>
<table width="100%" border="0" cellpadding="0" cellspacing="0">
<tr>
<td colspan="2" width="100%" style="font-family: Arial, sans-serif; font-size: 13px; font-weight:bold; color:#000;vertical-align:top;padding:10px 0 0 10px">
Fixed Tests (1)
</td>
</tr>
<tr>
<td width="20" style="vertical-align:top;padding:5px 0 5px 10px">
<img src="http://bamboo.asterisk.org/images/jt/icn_plan_passed.gif" width="15" height="15">
</td>
<td width="100%" style="font-family: Arial, sans-serif; font-size: 13px; color:#000;vertical-align:top;padding:5px 10px">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/test/AsteriskTestSuite" style="font-family: Arial, sans-serif; font-size: 13px;color:#000">AsteriskTestSuite</a> :
<a href="http://bamboo.asterisk.org/browse/AST-162-160/test/AsteriskTestSuite:rfc2833_dtmf_detect" style="font-family: Arial, sans-serif; font-size: 13px;color:#000">Rfc2833 dtmf detect</a>
<span style="color:#999"></span>
</td>
</tr> </table><br>
<table width="100%" border="0" cellpadding="0" cellspacing="0" style="background-color:#ecf1f7;border-top:1px solid #bbd0e5;border-bottom:1px solid #bbd0e5;color:#036;">
<tr>
<td width="60%" style="font-family: Arial, sans-serif;text-align:left;font-size:16px;font-weight:bold;color:#036;padding:5px 10px">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/log" style="text-decoration: none; font-family: Arial, sans-serif;text-align:left;font-size:16px;font-weight:bold;color:#036" >Error Summary</a>
</td>
<td width="40%" style="font-family: Arial, sans-serif;text-align:right;font-size:13px;color:#036;padding:5px 10px">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/log" style="font-family: Arial, sans-serif; font-size: 13px;color:#036">See full build log</a>
<img src="http://bamboo.asterisk.org/images/icons/more.gif" width="15" height="15" style="vertical-align:middle;padding:2px">
</td>
</tr>
</table>
<table width="100%" border="0" cellpadding="0" cellspacing="0" style="background-color:#f2f2f2;border-top:1px solid #d9d9d9;border-bottom:1px solid #d9d9d9;color:#000;margin-top:5px;padding:10px">
<tr>
<td width="100%" style="font-family:'Courier New', Courier, monospace; font-size: 12px; color:#000;vertical-align:top">
src/lfs.c: In function 'lfs_g_setmode':<br/>
</td>
</tr>
<tr>
<td width="100%" style="font-family:'Courier New', Courier, monospace; font-size: 12px; color:#000;vertical-align:top">
src/lfs.c:235: warning: unused parameter 'f'<br/>
</td>
</tr>
<tr>
<td width="100%" style="font-family:'Courier New', Courier, monospace; font-size: 12px; color:#000;vertical-align:top">
src/lfs.c:235: warning: unused parameter 'arg'<br/>
</td>
</tr>
</table>
<br />
<table width="100%" border="0" cellpadding="0" cellspacing="0">
<tr><td colspan="2" align="center" style="font-family: Arial, sans-serif;text-align:center;font-size:11px;font-weight:bold;color:#999;vertical-align:top;padding:20px;padding-bottom:10px">
Email generated by <a href="http://bamboo.asterisk.org" style="font-family: Arial, sans-serif; font-size: 11px; color:#999">Atlassian Bamboo</a> - if you wish to stop receiving these emails edit your <a href="http://bamboo.asterisk.org/profile/userNotifications.action" style="font-family: Arial, sans-serif; font-size: 11px; color:#999">user profile</a> or <a href="http://bamboo.asterisk.org/viewAdministrators.action" style="font-family: Arial, sans-serif; font-size: 11px; color:#999">notify your administrator</a>
</td>
</tr>
<tr><td colspan="2" align="center" style="font-family: Arial, sans-serif;text-align:center;font-size:11px;font-weight:bold;color:#999;vertical-align:top">
<a href="http://www.atlassian.com/software/ideconnector/" style="font-family: Arial, sans-serif; font-size: 11px; color:#999">Want this information in your IDE?</a>
</td></tr>
</table>
</td>
<td width="150" style="vertical-align:top">
<table width="150" border="0" cellpadding="0" cellspacing="0" style="background-color:#ecf1f7;border-top:1px solid #bbd0e5;border-bottom:1px solid #bbd0e5;color:#036;">
<tr>
<td style="font-family: Arial, sans-serif;text-align:left;font-size:16px;font-weight:bold;color:#036;vertical-align:top;padding:5px 10px">
Actions
</td>
</tr>
</table>
<table width="150" border="0" cellpadding="0" cellspacing="0" style="background-color:#f5f9fc;border-bottom:1px solid #bbd0e5;">
<tr>
<td style="font-family: Ariel, sans-serif; font-size: 13px; color:#036;vertical-align:top;padding:5px 10px;line-height:1.7">
<a href="http://bamboo.asterisk.org/browse/AST-162-160/" style="font-family: Arial, sans-serif; font-size: 13px; color:#036">View Online</a>
<br>
<a href="http://bamboo.asterisk.org/browse/AST-162-160/comment" style="font-family: Arial, sans-serif; font-size: 13px; color:#036">Add Comments</a>
<br>
<a href="http://bamboo.asterisk.org/browse/AST-162-160/artifact" style="font-family: Arial, sans-serif; font-size: 13px; color:#036">View Artifacts</a>
<br>
<a href="http://bamboo.asterisk.org/download/AST-162/build_logs/AST-162-160.log" style="font-family: Arial, sans-serif; font-size: 13px; color:#036">Download Logs</a>
<br>
</td>
</tr>
</table>
</td>
<tr>
</table>
</font>
</div>