Changeset 773 for pjproject

Timestamp:

Oct 16, 2006 12:53:48 PM (18 years ago)

Author:

bennylp

Message:

Fixed bug in media UDP transport: when recv() returns error (e.g. ICMP packet is received on Windows), transport will be stopped! [thanks Tomer]

File:

• pjproject/trunk/pjmedia/src/pjmedia/transport_udp.c (modified) (4 diffs)

pjproject/trunk/pjmedia/src/pjmedia/transport_udp.c

@@ -411 +411 @@
          * have been received.
          */
-        if ((udp->options & PJMEDIA_UDP_NO_SRC_ADDR_CHECKING)==0) {
+        if (bytes_read>0 && 
+            (udp->options & PJMEDIA_UDP_NO_SRC_ADDR_CHECKING)==0) 
+        {
             if ((udp->rem_rtp_addr.sin_addr.s_addr != 
                  udp->rtp_src_addr.sin_addr.s_addr) ||
@@ -465 +467 @@
                                      &udp->rtp_addrlen);
 
-    } while (status == PJ_SUCCESS);
+    } while (status != PJ_EPENDING);
 }
 
@@ -495 +497 @@
          * different.
          */
-        if ((udp->options & PJMEDIA_UDP_NO_SRC_ADDR_CHECKING)==0 &&
+        if (bytes_read>0 &&
+            (udp->options & PJMEDIA_UDP_NO_SRC_ADDR_CHECKING)==0 &&
             ((udp->rem_rtcp_addr.sin_addr.s_addr != 
                udp->rtcp_src_addr.sin_addr.s_addr) ||
@@ -516 +519 @@
                                      &udp->rtcp_addr_len);
 
-    } while (status == PJ_SUCCESS);
+    } while (status != PJ_EPENDING);
 }