Changeset 5680 for pjproject/trunk/pjlib/src/pj/ioqueue_select.c

Timestamp:

Nov 3, 2017 6:54:54 AM (7 years ago)

Author:

nanang

Message:

Fix #2055:

• Updated active socket to avoid double ioqueue key unregistration.
• Updated ioqueue to avoid double ioq key unregistration (select & epoll only, winnt & uwp does not seem to be affected).
• Added check for potential 'negative' ioqueue->count.

File:

• pjproject/trunk/pjlib/src/pj/ioqueue_select.c (modified) (1 diff)

pjproject/trunk/pjlib/src/pj/ioqueue_select.c

@@ -479 +479 @@
     pj_ioqueue_lock_key(key);
 
+    /* Best effort to avoid double key-unregistration */
+    if (IS_CLOSING(key)) {
+        pj_ioqueue_unlock_key(key);
+        return PJ_SUCCESS;
+    }
+
     /* Also lock ioqueue */
     pj_lock_acquire(ioqueue->lock);
 
-    pj_assert(ioqueue->count > 0);
-    --ioqueue->count;
+    /* Avoid "negative" ioqueue count */
+    if (ioqueue->count > 0) {
+        --ioqueue->count;
+    } else {
+        /* If this happens, very likely there is double unregistration
+         * of a key.
+         */
+        pj_assert(!"Bad ioqueue count in key unregistration!");
+        PJ_LOG(1,(THIS_FILE, "Bad ioqueue count in key unregistration!"));
+    }
+
 #if !PJ_IOQUEUE_HAS_SAFE_UNREG
     /* Ticket #520, key will be erased more than once */