<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/6671">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Sean Bright: 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: Prevent "user=phone" being added multiple times to header<br><br>ast_sip_add_usereqphone adds "user=phone" to the header every time is is<br>called without checking whether the param already exists. Preventing<br>this by searching to string representation of header for "user=phone".<br><br>ASTERISK-26988 #close<br><br>Change-Id: Ib84383b07254de357dc6a98d91fc1d2c2c3719e6<br>---<br>M res/res_pjsip.c<br>1 file changed, 5 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/res/res_pjsip.c b/res/res_pjsip.c<br>index 510b6b4..e215946 100644<br>--- a/res/res_pjsip.c<br>+++ b/res/res_pjsip.c<br>@@ -3073,6 +3073,11 @@<br>             return;<br>       }<br> <br>+ if (pjsip_param_find(&sip_uri->other_param, &STR_USER)) {<br>+         /* Don't add it if it's already there */<br>+             return;<br>+      }<br>+<br>  param = PJ_POOL_ALLOC_T(pool, pjsip_param);<br>   param->name = STR_USER;<br>    param->value = STR_PHONE;<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6671">change 6671</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/6671"/><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: Ib84383b07254de357dc6a98d91fc1d2c2c3719e6 </div>
<div style="display:none"> Gerrit-Change-Number: 6671 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dtryba <daniel@tryba.nl> </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>
<div style="display:none"> Gerrit-Reviewer: dtryba <daniel@tryba.nl> </div>