<p>Alexander Traud has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/6858">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">chan_sip: Crypto attribute not last but first on SDP media level.<br><br>This matches the behavior of the other SIP channel driver, chan_pjsip.<br><br>ASTERISK-27365<br><br>Change-Id: I8f23a51290a58b75816da2999ed1965441dfc5d6<br>---<br>M channels/chan_sip.c<br>1 file changed, 4 insertions(+), 4 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/58/6858/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/channels/chan_sip.c b/channels/chan_sip.c<br>index 6d85141..ef83666 100644<br>--- a/channels/chan_sip.c<br>+++ b/channels/chan_sip.c<br>@@ -13864,11 +13864,11 @@<br>                         case SDP_AUDIO:<br>                               if (needaudio) {<br>                                      add_content(resp, ast_str_buffer(m_audio));<br>-                                  add_content(resp, ast_str_buffer(a_audio));<br>-                                  add_content(resp, hold);<br>                                      if (a_crypto) {<br>                                               add_content(resp, a_crypto);<br>                                  }<br>+                                    add_content(resp, ast_str_buffer(a_audio));<br>+                                  add_content(resp, hold);<br>                              } else {<br>                                      add_content(resp, offer->decline_m_line);<br>                          }<br>@@ -13914,11 +13914,11 @@<br>          /* generate new SDP from scratch, no offers */<br>                if (needaudio) {<br>                      add_content(resp, ast_str_buffer(m_audio));<br>-                  add_content(resp, ast_str_buffer(a_audio));<br>-                  add_content(resp, hold);<br>                      if (a_crypto) {<br>                               add_content(resp, a_crypto);<br>                  }<br>+                    add_content(resp, ast_str_buffer(a_audio));<br>+                  add_content(resp, hold);<br>              }<br>             if (needvideo) { /* only if video response is appropriate */<br>                  add_content(resp, ast_str_buffer(m_video));<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6858">change 6858</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/6858"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I8f23a51290a58b75816da2999ed1965441dfc5d6 </div>
<div style="display:none"> Gerrit-Change-Number: 6858 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>