From 48eeb93628ea045ef78b02579153eeb988851da7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=9B=D0=B5=D0=BE=D0=BD=D0=B8=D0=B4=20=D0=AE=D1=80=D1=8C?= =?UTF-8?q?=D0=B5=D0=B2=20=28Leonid=20Yuriev=29?= Date: Mon, 5 Dec 2022 19:14:08 +0300 Subject: [PATCH] =?UTF-8?q?mdbx:=20=D0=B8=D1=81=D0=BF=D1=80=D0=B0=D0=B2?= =?UTF-8?q?=D0=BB=D0=B5=D0=BD=D0=B8=D0=B5=20=D0=BF=D0=B0=D0=B4=D0=B5=D0=BD?= =?UTF-8?q?=D0=B8=D1=8F=20=D0=B2=20`env=5Fclose()`=20=D0=BF=D1=80=D0=B8=20?= =?UTF-8?q?=D0=B7=D0=B0=D0=BA=D1=80=D1=8B=D1=82=D0=B8=D0=B8=20=D1=81=D1=80?= =?UTF-8?q?=D0=B5=D0=B4=D1=8B=20=D0=BF=D0=B5=D1=80=D0=B5-=D0=BE=D1=82?= =?UTF-8?q?=D0=BA=D1=80=D1=8B=D1=82=D0=BE=D0=B9=20=D0=B2=20=D1=80=D0=B5?= =?UTF-8?q?=D0=B6=D0=B8=D0=BC=D0=B5=20=D1=82=D0=BE=D0=BB=D1=8C=D0=BA=D0=BE?= =?UTF-8?q?-=D0=B4=D0=BB=D1=8F-=D1=87=D1=82=D0=B5=D0=BD=D0=B8=D1=8F.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Ошибка не была замечена ранее из-за много-ходового сценария воспроизведения: 1. Создаём экземпляр MDBX_env посредством mdbx_env_create(); 2. Пытаемся открыть БД посредством mdbx_env_open() в режиме чтения-записи и эта попытка должны быть неудачной; 3. Не освобождая экземпляр MDBX_env повторно открываем его в режиме только-чтение; 4. Закрываем среду посредством mdbx_env_close(). Падение происходит на пункте 4, либо на пункте 3, если попытка повторного открытия будет не успешной. Причина в том, что внутренний экземпляр osal_ioring_t инициализировался только для режимов чтения-записи, а разрушался всегда. При этом после первого разрушения намеренно оставался в состоянии вызывающем падение при использовании без инициализации. [Simon Leier](https://t.me/leisim) сообщал об этой проблеме (теперь понятно что это было), но из-за сложности сценария проблему не удалось воспроизвести и идентифицировать. --- src/core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/core.c b/src/core.c index b463aa5b..76b3b399 100644 --- a/src/core.c +++ b/src/core.c @@ -14671,7 +14671,8 @@ __cold static int env_close(MDBX_env *env) { } munlock_all(env); - osal_ioring_destroy(&env->me_ioring); + if (!(env->me_flags & MDBX_RDONLY)) + osal_ioring_destroy(&env->me_ioring); lcklist_lock(); const int rc = lcklist_detach_locked(env);