From fe9f6911947a655b0f6dd579b40658f1beca17e8 Mon Sep 17 00:00:00 2001 From: sdong Date: Wed, 4 Feb 2015 15:32:06 -0800 Subject: [PATCH] Fix fault_injestion_test Summary: A bug in MockEnv causes fault_injestion_test to fail. I don't know why it doesn't fail every time but it doesn't seem to be right. Test Plan: Run fault_injestion_test Also run db_test with MEM_ENV=1 until the first failure. Reviewers: yhchiang, rven, igor Reviewed By: igor Subscribers: leveldb, dhruba Differential Revision: https://reviews.facebook.net/D32877 --- util/mock_env.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/util/mock_env.cc b/util/mock_env.cc index 41aeafaaf..bcfc611b5 100644 --- a/util/mock_env.cc +++ b/util/mock_env.cc @@ -630,10 +630,10 @@ Status MockEnv::LockFile(const std::string& fname, FileLock** flock) { return Status::IOError(fn, "Lock is already held."); } } else { - auto* file = new MemFile(fname, true); + auto* file = new MemFile(fn, true); file->Ref(); file->Lock(); - file_map_[fname] = file; + file_map_[fn] = file; } } *flock = new MockEnvFileLock(fn);