Changeset 2469 for pjproject/branches/projects/aps-direct/pjmedia/src/pjmedia/sound_port.c

Timestamp:

Feb 22, 2009 12:00:12 PM (14 years ago)

Author:

bennylp

Message:

• New convention about default audio device ID (now there is different ID for default capture/playback device. It should be backward compatible)
• Fixed crash if sound port is unable to open sound device
• Testing with the switchboard

File:

• pjproject/branches/projects/aps-direct/pjmedia/src/pjmedia/sound_port.c (modified) (2 diffs)

pjproject/branches/projects/aps-direct/pjmedia/src/pjmedia/sound_port.c

@@ -414 +414 @@
 {
     pjmedia_snd_port *snd_port;
+    pj_status_t status;
 
     PJ_ASSERT_RETURN(pool && prm && p_port, PJ_EINVAL);
@@ -430 +431 @@
     pj_memcpy(&snd_port->aud_param, prm, sizeof(*prm));
     
-    *p_port = snd_port;
-
-
     /* Start sound device immediately.
      * If there's no port connected, the sound callback will return
      * empty signal.
      */
-    return start_sound_device( pool, snd_port );
-
+    status = start_sound_device( pool, snd_port );
+    if (status != PJ_SUCCESS) {
+        pjmedia_snd_port_destroy(snd_port);
+        return status;
+    }
+
+    *p_port = snd_port;
+    return PJ_SUCCESS;
 }