Changeset 5493 for pjproject/trunk/pjsip/src/pjsua-lib/pjsua_call.c

Timestamp:

Dec 6, 2016 11:23:39 AM (7 years ago)

Author:

ming

Message:

Fixed #1983: Add option to update call Via address

File:

• pjproject/trunk/pjsip/src/pjsua-lib/pjsua_call.c (modified) (6 diffs)

pjproject/trunk/pjsip/src/pjsua-lib/pjsua_call.c

@@ -570 +570 @@
 {
     opt->flag &= ~(PJSUA_CALL_UNHOLD | PJSUA_CALL_UPDATE_CONTACT |
-                   PJSUA_CALL_NO_SDP_OFFER | PJSUA_CALL_REINIT_MEDIA);
+                   PJSUA_CALL_NO_SDP_OFFER | PJSUA_CALL_REINIT_MEDIA |
+                   PJSUA_CALL_UPDATE_VIA);
 }
 
@@ -637 +638 @@
 
     return PJ_SUCCESS;
+}
+
+static void dlg_set_via(pjsip_dialog *dlg, pjsua_acc *acc)
+{
+    if (acc->cfg.allow_via_rewrite && acc->via_addr.host.slen > 0) {
+        pjsip_dlg_set_via_sent_by(dlg, &acc->via_addr, acc->via_tp);
+    } else if (!pjsua_sip_acc_is_using_stun(acc->index)) {
+        /* Choose local interface to use in Via if acc is not using
+         * STUN. See https://trac.pjsip.org/repos/ticket/1804
+         */
+        pjsip_host_port via_addr;
+        const void *via_tp;
+
+        if (pjsua_acc_get_uac_addr(acc->index, dlg->pool, &acc->cfg.id,
+                                   &via_addr, NULL, NULL,
+                                   &via_tp) == PJ_SUCCESS)
+        {
+            pjsip_dlg_set_via_sent_by(dlg, &via_addr,
+                                      (pjsip_transport*)via_tp);
+        }
+    }
 }
 
@@ -778 +800 @@
     pjsip_dlg_inc_lock(dlg);
 
-    if (acc->cfg.allow_via_rewrite && acc->via_addr.host.slen > 0) {
-        pjsip_dlg_set_via_sent_by(dlg, &acc->via_addr, acc->via_tp);
-    } else if (!pjsua_sip_acc_is_using_stun(acc_id)) {
-        /* Choose local interface to use in Via if acc is not using
-         * STUN. See https://trac.pjsip.org/repos/ticket/1804
-         */
-        pjsip_host_port via_addr;
-        const void *via_tp;
-
-        if (pjsua_acc_get_uac_addr(acc_id, dlg->pool, &acc->cfg.id,
-                                   &via_addr, NULL, NULL,
-                                   &via_tp) == PJ_SUCCESS)
-        {
-            pjsip_dlg_set_via_sent_by(dlg, &via_addr,
-                                      (pjsip_transport*)via_tp);
-        }
-    }
+    dlg_set_via(dlg, acc);
 
     /* Calculate call's secure level */
@@ -2500 +2506 @@
     }
 
+    if ((options & PJSUA_CALL_UPDATE_VIA) &&
+        pjsua_acc_is_valid(call->acc_id))
+    {
+        dlg_set_via(call->inv->dlg, &pjsua_var.acc[call->acc_id]);
+    }
+
     /* Create re-INVITE with new offer */
     status = pjsip_inv_reinvite( call->inv, new_contact, sdp, &tdata);
@@ -2621 +2633 @@
     }
 
+    if ((call->opt.flag & PJSUA_CALL_UPDATE_VIA) &&
+        pjsua_acc_is_valid(call->acc_id))
+    {
+        dlg_set_via(call->inv->dlg, &pjsua_var.acc[call->acc_id]);
+    }
+
     /* Create re-INVITE with new offer */
     status = pjsip_inv_reinvite( call->inv, new_contact, sdp, &tdata);
@@ -2728 +2746 @@
     {
         new_contact = &pjsua_var.acc[call->acc_id].contact;
+    }
+
+    if ((call->opt.flag & PJSUA_CALL_UPDATE_VIA) &&
+        pjsua_acc_is_valid(call->acc_id))
+    {
+        dlg_set_via(call->inv->dlg, &pjsua_var.acc[call->acc_id]);
     }