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

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res_pjsip_nat.c: Remove unnecessary CMP_STOP.<br><br>Change-Id: I6279b0d723bc3b75b8d65e81e02da9ea9bc0c3da<br>---<br>M res/res_pjsip_nat.c<br>1 file changed, 2 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/res/res_pjsip_nat.c b/res/res_pjsip_nat.c<br>index 7dfd5ec..45b0d7c 100644<br>--- a/res/res_pjsip_nat.c<br>+++ b/res/res_pjsip_nat.c<br>@@ -35,6 +35,7 @@<br> static void rewrite_uri(pjsip_rx_data *rdata, pjsip_sip_uri *uri)<br> {<br>      pj_cstr(&uri->host, rdata->pkt_info.src_name);<br>+     uri->port = rdata->pkt_info.src_port;<br>   if (!strcasecmp("WSS", rdata->tp_info.transport->type_name)) {<br>                /* WSS is special, we don't want to overwrite the URI at all as it needs to be ws */<br>      } else if (strcasecmp("udp", rdata->tp_info.transport->type_name)) {<br>@@ -42,7 +43,6 @@<br>       } else {<br>              uri->transport_param.slen = 0;<br>     }<br>-    uri->port = rdata->pkt_info.src_port;<br> }<br> <br> static int rewrite_route_set(pjsip_rx_data *rdata, pjsip_dialog *dlg)<br>@@ -165,7 +165,7 @@<br>               ((details->type == transport_state->type) && (transport_state->factory) &&<br>                   !pj_strcmp(&transport_state->factory->addr_name.host, &details->local_address) &&<br>                    transport_state->factory->addr_name.port == details->local_port))) {<br>-                return CMP_MATCH | CMP_STOP;<br>+         return CMP_MATCH;<br>     }<br> <br>  return 0;<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6138">change 6138</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/6138"/><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: merged </div>
<div style="display:none"> Gerrit-Change-Id: I6279b0d723bc3b75b8d65e81e02da9ea9bc0c3da </div>
<div style="display:none"> Gerrit-Change-Number: 6138 </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>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>