<p>Joshua Colp <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7349">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;">pjsip: 183 without To tag does not negotiate media<br><br>If a 183 with sdp response is receive without a To tag the sdp is not<br>negotiated. According to RFC 3261 section 12.1.2 while a To tag is required,<br>the client needs to still be able to handle the missing tag case for<br>backwards compatibility.<br><br>This patch, accepted by and applied to pjproject, makes it so if an incoming<br>180/183 with SDP comes in without a To tag it gets appropriately handled.<br><br>ASTERISK-27442 #close<br><br>Change-Id: Ic9d6b01e05e8f4874eebbd7adfe05d932025d203<br>---<br>A third-party/pjproject/patches/0040-183_without_to_tag.patch<br>1 file changed, 17 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/third-party/pjproject/patches/0040-183_without_to_tag.patch b/third-party/pjproject/patches/0040-183_without_to_tag.patch<br>new file mode 100644<br>index 0000000..e8692fe<br>--- /dev/null<br>+++ b/third-party/pjproject/patches/0040-183_without_to_tag.patch<br>@@ -0,0 +1,17 @@<br>+diff --git a/pjsip/src/pjsip-ua/sip_inv.c b/pjsip/src/pjsip-ua/sip_inv.c<br>+index c9686a0..fc52a63 100644<br>+--- a/pjsip/src/pjsip-ua/sip_inv.c<br>++++ b/pjsip/src/pjsip-ua/sip_inv.c<br>+@@ -4156,9 +4156,10 @@ static void inv_on_state_calling( pjsip_inv_session *inv, pjsip_event *e)<br>+              status = pjsip_inv_send_msg(inv, cancel);<br>+            }<br>+ <br>+-       if (dlg->remote.info->tag.slen) {<br>++     if (tsx->status_code != 100) {<br>+ <br>+-           inv_set_state(inv, PJSIP_INV_STATE_EARLY, e);<br>++               if (dlg->remote.info->tag.slen)<br>++                   inv_set_state(inv, PJSIP_INV_STATE_EARLY, e);<br>+ <br>+                inv_check_sdp_in_incoming_msg(inv, tsx, <br>+                                           e->body.tsx_state.src.rdata);<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7349">change 7349</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/7349"/><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: Ic9d6b01e05e8f4874eebbd7adfe05d932025d203 </div>
<div style="display:none"> Gerrit-Change-Number: 7349 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Kevin Harwell <kharwell@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>