<p>Torrey Searle has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/6251">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">res/res_pjsip_session: allow SDP answer to be regenerated<br><br>If an SDP answer hasn't been sent yet, it's legal to change it.<br>This is required for PJSIP_DTMF_MODE to work correctly, and can<br>also have use in the future for updating codecs too.<br><br>ASTERISK-27209 #close<br><br>Change-Id: Idbbfb7cb3f72fbd96c94d10d93540f69bd51e7a1<br>---<br>M channels/pjsip/dialplan_functions.c<br>M include/asterisk/res_pjsip_session.h<br>M res/res_pjsip_session.c<br>M res/res_pjsip_session.exports.in<br>4 files changed, 56 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/51/6251/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/channels/pjsip/dialplan_functions.c b/channels/pjsip/dialplan_functions.c<br>index a382c4e..2d3e2b1 100644<br>--- a/channels/pjsip/dialplan_functions.c<br>+++ b/channels/pjsip/dialplan_functions.c<br>@@ -1117,6 +1117,9 @@<br> <br>            ast_sip_session_refresh(data->session, NULL, NULL,<br>                         sip_session_response_cb, data->method, 1);<br>+        } else if (data->session->inv_session->state == PJSIP_INV_STATE_INCOMING) {<br>+         ast_debug(3, "Changing DTMF mode on channel %s during OFFER/ANSER exchange. Updating SDP answer\n", ast_channel_name(data->session->channel));<br>+               ast_sip_session_regenerate_answer(data->session, NULL);<br>    }<br> <br>  return 0;<br>diff --git a/include/asterisk/res_pjsip_session.h b/include/asterisk/res_pjsip_session.h<br>index ca919da..ff3f95b 100644<br>--- a/include/asterisk/res_pjsip_session.h<br>+++ b/include/asterisk/res_pjsip_session.h<br>@@ -624,6 +624,22 @@<br>              int generate_new_sdp);<br> <br> /*!<br>+ * \brief Regenerate SDP Answer<br>+ *<br>+ * This method is used when an sdp offer has been received but and SDP answer<br>+ * has not been sent yet.  As with any outgoing request in res_pjsip_session,<br>+ * this will call into registered supplements in case they wish to add anything.<br>+ *<br>+ * \param session The session on which the answer will be updated<br>+ * \param on_sdp_creation Callback called when SDP is created<br>+ * \param generate_new_sdp Boolean to indicate if a new SDP should be created<br>+ * \retval 0 Successfully updated the SDP answer<br>+ * \retval -1 Failure to updated the SDP answer<br>+ */<br>+int ast_sip_session_regenerate_answer(struct ast_sip_session *session,<br>+               ast_sip_session_sdp_creation_cb on_sdp_creation);<br>+<br>+/*!<br>  * \brief Send a SIP response<br>  *<br>  * This will send the SIP response specified in tdata and<br>diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c<br>index 8256d3c..01e5e75 100644<br>--- a/res/res_pjsip_session.c<br>+++ b/res/res_pjsip_session.c<br>@@ -968,6 +968,42 @@<br>        return 0;<br> }<br> <br>+int ast_sip_session_regenerate_answer(struct ast_sip_session *session,<br>+            ast_sip_session_sdp_creation_cb on_sdp_creation)<br>+{<br>+ pjsip_inv_session *inv_session = session->inv_session;<br>+    pjmedia_sdp_session *new_answer = NULL;<br>+      const pjmedia_sdp_session *previous_offer = NULL;<br>+    /* SDP can only be regenerated if an SDT answer is still pending to be sent */<br>+       if (inv_session->neg<br>+              && (pjmedia_sdp_neg_get_state(inv_session->neg) == PJMEDIA_SDP_NEG_STATE_REMOTE_OFFER<br>+                     || pjmedia_sdp_neg_get_state(inv_session->neg) ==PJMEDIA_SDP_NEG_STATE_WAIT_NEGO)) {<br>+              pjmedia_sdp_neg_get_neg_remote(inv_session->neg, &previous_offer);<br>+<br>+         if (pjmedia_sdp_neg_get_state(inv_session->neg) ==PJMEDIA_SDP_NEG_STATE_WAIT_NEGO) {<br>+                      /* Transition back to PJMEDIA_SDP_NEG_STATE_REMOTE_OFFER */<br>+                  pjmedia_sdp_neg_negotiate(inv_session->pool, inv_session->neg, 0);<br>+                     pjmedia_sdp_neg_set_remote_offer(inv_session->pool, inv_session->neg, previous_offer);<br>+         }<br>+<br>+         if ((new_answer = create_local_sdp(inv_session, session, previous_offer))) {<br>+                 if (on_sdp_creation) {<br>+                               if (on_sdp_creation(session, new_answer)) {<br>+                                  return -1;<br>+                           }<br>+                    }<br>+                    pjsip_inv_set_sdp_answer(inv_session, new_answer);<br>+                   return 0;<br>+            } else {<br>+                     ast_log(LOG_WARNING, "ast_sip_session_regenerate_answer: (channel: %s) unable to find previous offer!\n", ast_channel_name(session->channel));<br>+          }<br>+    } else {<br>+             ast_log(LOG_WARNING, "ast_sip_session_regenerate_answer: (channel: %s) invoked from illegal state %s!\n", ast_channel_name(session->channel), pjmedia_sdp_neg_state_str(pjmedia_sdp_neg_get_state(inv_session->neg)));<br>+       }<br>+<br>+ return -1;<br>+}<br>+<br> void ast_sip_session_send_response(struct ast_sip_session *session, pjsip_tx_data *tdata)<br> {<br>     handle_outgoing_response(session, tdata);<br>diff --git a/res/res_pjsip_session.exports.in b/res/res_pjsip_session.exports.in<br>index fdfc5fb..5bc0bf4 100644<br>--- a/res/res_pjsip_session.exports.in<br>+++ b/res/res_pjsip_session.exports.in<br>@@ -14,6 +14,7 @@<br>                 LINKER_SYMBOL_PREFIXast_sip_session_remove_datastore;<br>                 LINKER_SYMBOL_PREFIXast_sip_session_get_identity;<br>             LINKER_SYMBOL_PREFIXast_sip_session_refresh;<br>+         LINKER_SYMBOL_PREFIXast_sip_session_regenerate_answer;<br>                LINKER_SYMBOL_PREFIXast_sip_session_send_response;<br>            LINKER_SYMBOL_PREFIXast_sip_session_send_request;<br>             LINKER_SYMBOL_PREFIXast_sip_session_create_invite;<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/6251">change 6251</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/6251"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Idbbfb7cb3f72fbd96c94d10d93540f69bd51e7a1 </div>
<div style="display:none"> Gerrit-Change-Number: 6251 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Torrey Searle <tsearle@gmail.com> </div>