Commit 31790d7f authored by Sunny Bains's avatar Sunny Bains

In sync_close() fix a bug introduced by the fix for Bug #59683 where we iterate

over the mutex list and free each mutex. When UNIV_MEM_DEBUG is defined, we
need skip the hash mutex.

It is a minor bug affecting only UNIV_SYNC_DEBUG builds, found by Michael.
parent 12a54ac5
...@@ -1506,7 +1506,7 @@ sync_close(void) ...@@ -1506,7 +1506,7 @@ sync_close(void)
for (mutex = UT_LIST_GET_FIRST(mutex_list); for (mutex = UT_LIST_GET_FIRST(mutex_list);
mutex != NULL; mutex != NULL;
mutex = UT_LIST_GET_FIRST(mutex_list)) { /* No op */) {
#ifdef UNIV_MEM_DEBUG #ifdef UNIV_MEM_DEBUG
if (mutex == &mem_hash_mutex) { if (mutex == &mem_hash_mutex) {
...@@ -1516,6 +1516,8 @@ sync_close(void) ...@@ -1516,6 +1516,8 @@ sync_close(void)
#endif /* UNIV_MEM_DEBUG */ #endif /* UNIV_MEM_DEBUG */
mutex_free(mutex); mutex_free(mutex);
mutex = UT_LIST_GET_FIRST(mutex_list);
} }
mutex_free(&mutex_list_mutex); mutex_free(&mutex_list_mutex);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment