Changeset 830 for pjproject/trunk

Timestamp:

Nov 25, 2006 8:48:54 AM (18 years ago)

Author:

bennylp

Message:

Minor fix in UDP media: transport should silently ignore detach command when it's not attached

File:

• pjproject/trunk/pjmedia/src/pjmedia/transport_udp.c (modified) (1 diff)

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

@@ -584 +584 @@
     pj_assert(tp);
 
-    /* User data is unreferenced on Release build */
-    PJ_UNUSED_ARG(user_data);
-
-    /* As additional checking, check if the same user data is specified */
-    pj_assert(user_data == udp->user_data);
-
-    /* First, mark transport as unattached */
-    udp->attached = PJ_FALSE;
-
-    /* Clear up application infos from transport */
-    udp->rtp_cb = NULL;
-    udp->rtcp_cb = NULL;
-    udp->user_data = NULL;
+    if (udp->attached) {
+        /* User data is unreferenced on Release build */
+        PJ_UNUSED_ARG(user_data);
+
+        /* As additional checking, check if the same user data is specified */
+        pj_assert(user_data == udp->user_data);
+
+        /* First, mark transport as unattached */
+        udp->attached = PJ_FALSE;
+
+        /* Clear up application infos from transport */
+        udp->rtp_cb = NULL;
+        udp->rtcp_cb = NULL;
+        udp->user_data = NULL;
+    }
 }