From aca71f7aaec5ad13fef80bc9525b0f6f03ad2ff7 Mon Sep 17 00:00:00 2001 From: ardocrat Date: Mon, 25 May 2026 17:58:31 +0300 Subject: [PATCH] lmdb: return an error if removal of old DB file failed after migration --- store/src/lmdb.rs | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/store/src/lmdb.rs b/store/src/lmdb.rs index f213e0ac..79f1193c 100644 --- a/store/src/lmdb.rs +++ b/store/src/lmdb.rs @@ -252,9 +252,15 @@ impl Store { let migrate_from = Path::new(root_path).join(env_name); if migrate_from.exists() { match s.migrate_to_default_env(db_name, &migrate_from) { - Ok(_) => { - let _ = fs::remove_dir_all(&migrate_from); - } + Ok(_) => match fs::remove_dir_all(&migrate_from) { + Ok(_) => {} + Err(e) => { + return Err(Error::FileErr(format!( + "Can not remove old DB file: {:?}", + e + ))); + } + }, Err(e) => { error!("DB {} migration error: {:?}", env_name, e); match s.clear() {