Changeset 2813 for pjproject

Timestamp:

Jun 25, 2009 1:09:24 PM (15 years ago)

Author:

bennylp

Message:

Ticket #898: Race condition in sound auto-close feature may cause sound device to not be opened

• backported changes from #824

File:

• pjproject/branches/1.0/pjsip/src/pjsua-lib/pjsua_media.c (modified) (2 diffs)

pjproject/branches/1.0/pjsip/src/pjsua-lib/pjsua_media.c

@@ -469 +469 @@
     PJ_UNUSED_ARG(th);
 
-    PJ_LOG(4,(THIS_FILE,"Closing sound device after idle for %d seconds", 
-              pjsua_var.media_cfg.snd_auto_close_time));
-
-    entry->id = PJ_FALSE;
-
-    close_snd_dev();
+    PJSUA_LOCK();
+    if (entry->id) {
+        PJ_LOG(4,(THIS_FILE,"Closing sound device after idle for %d seconds", 
+                  pjsua_var.media_cfg.snd_auto_close_time));
+
+        entry->id = PJ_FALSE;
+
+        close_snd_dev();
+    }
+    PJSUA_UNLOCK();
 }
 
@@ -1606 +1610 @@
 {
     /* If sound device idle timer is active, cancel it first. */
+    PJSUA_LOCK();
     if (pjsua_var.snd_idle_timer.id) {
         pjsip_endpt_cancel_timer(pjsua_var.endpt, &pjsua_var.snd_idle_timer);
         pjsua_var.snd_idle_timer.id = PJ_FALSE;
     }
+    PJSUA_UNLOCK();
 
     /* Create sound port if none is instantiated */