Changeset 4084 for pjproject/trunk/pjmedia/src/test/vid_codec_test.c

Timestamp:

Apr 25, 2012 7:13:05 AM (12 years ago)

Author:

ming

Message:

Re #1446 (misc): Fix possible crash if video ports are destroyed in the wrong order (i.e. client port must be destroyed last).

File:

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

pjproject/trunk/pjmedia/src/test/vid_codec_test.c

@@ -418 +418 @@
         PJ_PERROR(3, (THIS_FILE, status, "  error"));
     }
-    if (capture) {
+    if (capture)
         pjmedia_vid_port_stop(capture);
+    if (renderer)
+        pjmedia_vid_port_stop(renderer);
+    if (capture)
         pjmedia_vid_port_destroy(capture);
-    }
-    if (renderer) {
-        pjmedia_vid_port_stop(renderer);
+    if (renderer)
         pjmedia_vid_port_destroy(renderer);
-    }
     if (codec) {
         pjmedia_event_unsubscribe(NULL, &codec_on_event, &codec_port_data,