<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/6409">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_rtp_asterisk.c: Check RTP packet version earlier.<br><br>Change-Id: Ic6493a7d79683f3e5845dff1cee49445fd5a0adf<br>---<br>M res/res_rtp_asterisk.c<br>1 file changed, 5 insertions(+), 5 deletions(-)<br><br></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 b6cef97..77027aa 100644<br>--- a/res/res_rtp_asterisk.c<br>+++ b/res/res_rtp_asterisk.c<br>@@ -5070,6 +5070,11 @@<br>          return &ast_null_frame;<br>   }<br> <br>+ /* If the version is not what we expected by this point then just drop the packet */<br>+ if (version != 2) {<br>+          return &ast_null_frame;<br>+  }<br>+<br>  /* If strict RTP protection is enabled see if we need to learn the remote address or if we need to drop the packet */<br>         if (rtp->strict_rtp_state == STRICT_RTP_LEARN) {<br>           if (!ast_sockaddr_cmp(&rtp->strict_rtp_address, &addr)) {<br>@@ -5114,11 +5119,6 @@<br>                          ast_debug(0, "RTP NAT: Got audio from other end. Now sending to address %s\n",<br>                                        ast_sockaddr_stringify(&remote_address));<br>               }<br>-    }<br>-<br>- /* If the version is not what we expected by this point then just drop the packet */<br>- if (version != 2) {<br>-          return &ast_null_frame;<br>   }<br> <br>  /* Pull out the various other fields we will need */<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6409">change 6409</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/6409"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: Ic6493a7d79683f3e5845dff1cee49445fd5a0adf </div>
<div style="display:none"> Gerrit-Change-Number: 6409 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>