<p>Richard Mudgett has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/7436">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_rtp_asterisk.c: Increase strictrtp learning timeout time.<br><br>More complicated direct media reinvite negotiations can result in longer<br>delays before direct media flows.  The strictrtp learning timeout time<br>was too short.  One log showed that the first RTP packet came in just<br>after three seconds.<br><br>* Increase the strictrtp learning timeout time from 1.5 to 5 seconds.<br><br>ASTERISK-27453<br><br>Change-Id: Ic5e711164cbb91b4d1c1e40c83697755640f138c<br>---<br>M res/res_rtp_asterisk.c<br>1 file changed, 8 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/36/7436/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/res/res_rtp_asterisk.c b/res/res_rtp_asterisk.c<br>index 2730611..5aeb791 100644<br>--- a/res/res_rtp_asterisk.c<br>+++ b/res/res_rtp_asterisk.c<br>@@ -140,7 +140,14 @@<br>    STRICT_RTP_CLOSED,   /*! Drop all RTP packets not coming from source that was learned */<br> };<br> <br>-#define STRICT_RTP_LEARN_TIMEOUT     1500    /*!< milliseconds */<br>+/*!<br>+ * \brief Strict RTP learning timeout time in milliseconds<br>+ *<br>+ * \note Set to 5 seconds to allow reinvite chains for direct media<br>+ * to settle before media actually starts to arrive.  There may be a<br>+ * reinvite collision involved on the other leg.<br>+ */<br>+#define STRICT_RTP_LEARN_TIMEOUT        5000<br> <br> #define DEFAULT_STRICT_RTP -1 /*!< Enabled */<br> #define DEFAULT_ICESUPPORT 1<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7436">change 7436</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/7436"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ic5e711164cbb91b4d1c1e40c83697755640f138c </div>
<div style="display:none"> Gerrit-Change-Number: 7436 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>