Changeset 2293 for pjproject/trunk/pjmedia/src/pjmedia/transport_ice.c

Timestamp:

Sep 18, 2008 12:58:33 PM (15 years ago)

Author:

nanang

Message:

Ticket #631:

• Changed rem_rtp/rtcp_addr to src_rtp/rtcp_addr in media transport info
• Updated behaviour of media transport get info, when the transport hasn't receive any packet src_rtp/rtcp_addr will not be set.
• Fixed bug in pjsua_call_dump that rem_addr was unset.

File:

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

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

@@ -1425 +1425 @@
     }
 
-    /* Get remote address originating RTP & RTCP. */
-    info->rem_rtp_name  = tp_ice->rtp_src_addr;
-    info->rem_rtcp_name = tp_ice->rtcp_src_addr;
+    /* Set remote address originating RTP & RTCP if this transport has 
+     * received any packets.
+     */
+    if (tp_ice->rtp_src_cnt) {
+        info->src_rtp_name  = tp_ice->rtp_src_addr;
+        info->src_rtcp_name = tp_ice->rtcp_src_addr;
+    }
 
     return PJ_SUCCESS;
@@ -1455 +1459 @@
     tp_ice->addr_len = addr_len;
 
+    /* Init source RTP & RTCP addresses and counter */
+    tp_ice->rtp_src_addr = tp_ice->remote_rtp;
+    tp_ice->rtcp_src_addr = tp_ice->remote_rtcp;
+    tp_ice->rtp_src_cnt = 0;
+
     return PJ_SUCCESS;
 }
@@ -1551 +1560 @@
          * have been received.
          */
-        if (!tp_ice->use_ice &&
-            pj_sockaddr_cmp(&tp_ice->remote_rtp, src_addr) != 0 )
-        {
-            /* Check if the source address is recognized. */
-            if (pj_sockaddr_cmp(src_addr, &tp_ice->rtp_src_addr) != 0) {
-                /* Remember the new source address. */
-                pj_sockaddr_cp(&tp_ice->rtp_src_addr, src_addr);
-
-                /* Reset counter */
-                tp_ice->rtp_src_cnt = 0;
-            }
-
-            tp_ice->rtp_src_cnt++;
-
-            if ((tp_ice->options & PJMEDIA_ICE_NO_SRC_ADDR_CHECKING) == 0 &&
-                tp_ice->rtp_src_cnt >= PJMEDIA_RTP_NAT_PROBATION_CNT) 
-            {
-        
-                char addr_text[80];
-
-                /* Set remote RTP address to source address */
-                pj_sockaddr_cp(&tp_ice->remote_rtp, &tp_ice->rtp_src_addr);
-                tp_ice->addr_len = pj_sockaddr_get_len(&tp_ice->remote_rtp);
-
-                /* Reset counter */
-                tp_ice->rtp_src_cnt = 0;
-
-                PJ_LOG(4,(tp_ice->base.name,
-                          "Remote RTP address switched to %s",
-                          pj_sockaddr_print(&tp_ice->remote_rtp, addr_text,
-                                            sizeof(addr_text), 3)));
-
-                /* Also update remote RTCP address if actual RTCP source
-                 * address is not heard yet.
-                 */
-                if (!pj_sockaddr_has_addr(&tp_ice->rtcp_src_addr)) {
-                    pj_uint16_t port;
-
-                    pj_sockaddr_cp(&tp_ice->remote_rtcp, &tp_ice->remote_rtp);
-
-                    port = (pj_uint16_t)
-                           (pj_sockaddr_get_port(&tp_ice->remote_rtp)+1);
-                    pj_sockaddr_set_port(&tp_ice->remote_rtcp, port);
+        if (!tp_ice->use_ice) {
+
+            /* Increment counter and avoid zero */
+            if (++tp_ice->rtp_src_cnt == 0) 
+                tp_ice->rtp_src_cnt = 1;
+
+            if (pj_sockaddr_cmp(&tp_ice->remote_rtp, src_addr) != 0) {
+
+                /* Check if the source address is recognized. */
+                if (pj_sockaddr_cmp(src_addr, &tp_ice->rtp_src_addr) != 0) {
+                    /* Remember the new source address. */
+                    pj_sockaddr_cp(&tp_ice->rtp_src_addr, src_addr);
+                    /* Reset counter */
+                    tp_ice->rtp_src_cnt = 0;
+                }
+
+                if ((tp_ice->options & PJMEDIA_ICE_NO_SRC_ADDR_CHECKING)==0 &&
+                    tp_ice->rtp_src_cnt >= PJMEDIA_RTP_NAT_PROBATION_CNT) 
+                {
+                    char addr_text[80];
+
+                    /* Set remote RTP address to source address */
+                    pj_sockaddr_cp(&tp_ice->remote_rtp, &tp_ice->rtp_src_addr);
+                    tp_ice->addr_len = pj_sockaddr_get_len(&tp_ice->remote_rtp);
+
+                    /* Reset counter */
+                    tp_ice->rtp_src_cnt = 0;
 
                     PJ_LOG(4,(tp_ice->base.name,
-                              "Remote RTCP address switched to %s",
-                              pj_sockaddr_print(&tp_ice->remote_rtcp, addr_text,
+                              "Remote RTP address switched to %s",
+                              pj_sockaddr_print(&tp_ice->remote_rtp, addr_text,
                                                 sizeof(addr_text), 3)));
 
+                    /* Also update remote RTCP address if actual RTCP source
+                     * address is not heard yet.
+                     */
+                    if (!pj_sockaddr_has_addr(&tp_ice->rtcp_src_addr)) {
+                        pj_uint16_t port;
+
+                        pj_sockaddr_cp(&tp_ice->remote_rtcp, 
+                                       &tp_ice->remote_rtp);
+
+                        port = (pj_uint16_t)
+                               (pj_sockaddr_get_port(&tp_ice->remote_rtp)+1);
+                        pj_sockaddr_set_port(&tp_ice->remote_rtcp, port);
+
+                        PJ_LOG(4,(tp_ice->base.name,
+                                  "Remote RTCP address switched to %s",
+                                  pj_sockaddr_print(&tp_ice->remote_rtcp, 
+                                                    addr_text,
+                                                    sizeof(addr_text), 3)));
+                    }
                 }
             }