From 8426c7411bcd1c3efec6a65db98266b0cad5f62a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marko=20M=C3=A4kel=C3=A4?= Date: Wed, 2 Jun 2021 08:48:09 +0300 Subject: [PATCH] MDEV-23399 fixup for MDEV-10814 buf_madvise_do_dump(): Fix a mutex release that was broken in commit 7cffb5f6e8a231a041152447be8980ce35d2c9b8. This function is not covered by any tests. Its only purpose is to be called from a debugger so that buffers that would normally be excluded from core dumps ever since commit b600f30786816e33c1706dd36cdabf21034dc781 can be included. --- storage/innobase/buf/buf0buf.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/storage/innobase/buf/buf0buf.cc b/storage/innobase/buf/buf0buf.cc index 4b0e1edd6c9..72f44ab9c32 100644 --- a/storage/innobase/buf/buf0buf.cc +++ b/storage/innobase/buf/buf0buf.cc @@ -1021,7 +1021,7 @@ buf_madvise_do_dump() ret+= madvise(chunk->mem, chunk->mem_size(), MADV_DODUMP); } - mysql_mutex_lock(&buf_pool.mutex); + mysql_mutex_unlock(&buf_pool.mutex); return ret; } #endif