Add check to confirm there're no pending operations before key return…

… to free list
pjsip · Nov 21, 2024 · cfe33e9 · cfe33e9
1 parent 1b2fe0b
commit cfe33e9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/pjlib/src/pj/ioqueue_winnt.c b/pjlib/src/pj/ioqueue_winnt.c
@@ -1042,6 +1042,9 @@ static void scan_closing_keys(pj_ioqueue_t *ioqueue)
             pj_assert(key->closing != 0);
 
             if (PJ_TIME_VAL_GTE(now, key->free_time)) {
+                /* Confirm that there're no any pending operations */
+                pj_assert(pj_list_empty(&key->pending_list));
+
                 pj_list_erase(key);
                 pj_list_push_back(&ioqueue->free_list, key);
             }

diff --git a/pjlib/src/pj/os_core_win32.c b/pjlib/src/pj/os_core_win32.c
@@ -54,7 +54,7 @@
 #else
 #   define LOG_MUTEX(expr)  PJ_LOG(6,expr)
 #endif
-#   define LOG_MUTEX_WARN(expr)  PJ_LOG(3,expr)
+#   define LOG_MUTEX_WARN(expr)  PJ_PERROR(3,expr)
 #define THIS_FILE       "os_core_win32.c"
 
 /*