Changeset 2957

Timestamp:

Oct 20, 2009 2:44:00 PM (14 years ago)

Author:

bennylp

Message:

Fixed ticket #977: ICE does not change role to Controlling when talking to ice-lite agents (thanks Amaresh Marekar for the report)

File:

• pjproject/trunk/pjmedia/src/pjmedia/transport_ice.c (modified) (2 diffs)

pjproject/trunk/pjmedia/src/pjmedia/transport_ice.c

@@ -1148 +1148 @@
 
     /* Validate media transport */
-    /* For now, this transport only support RTP/AVP transport */
+    /* This transport only support RTP/AVP transport, unless if
+     * transport checking is disabled
+     */
     if ((tp_ice->media_option & PJMEDIA_TPMED_NO_TRANSPORT_CHECKING) == 0) {
         pjmedia_sdp_media *loc_m, *rem_m;
@@ -1411 +1413 @@
         }
 
+        /* Ticket #977: Update role if turns out we're supposed to be the 
+         * Controlling agent (e.g. when talking to ice-lite peer). 
+         */
+        if (tp_ice->rem_offer_state.local_role==PJ_ICE_SESS_ROLE_CONTROLLING &&
+            pj_ice_strans_has_sess(tp_ice->ice_st)) 
+        {
+            pj_ice_strans_change_role(tp_ice->ice_st, 
+                                      PJ_ICE_SESS_ROLE_CONTROLLING);
+        }
+
+
         /* start ICE */
     }