andreacavalli/rocksdb
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix a valgrind warning

Browse Source 
Summary:
A latest valgrind test found a recently added unit test has memory leak,
which is because DB is not closed at the end of the test.

Test Plan: re-run the valgrind locally and make sure there's no memory leakage any more.

Reviewers: emayanke

CC: leveldb

Differential Revision: https://reviews.facebook.net/D13725
This commit is contained in:
Kai Liu 2013-10-28 14:33:24 -07:00
parent 100fa8e013
commit 7e91b86f4d
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
db/deletefile_test.cc
View File

@ -218,6 +218,7 @@ TEST(DeleteFileTest, DeleteLogFiles) {
archived_log->PathName().c_str()); archived_log->PathName().c_str());
ASSERT_OK(db_->DeleteFile(archived_log->PathName())); ASSERT_OK(db_->DeleteFile(archived_log->PathName()));
ASSERT_TRUE(!env_->FileExists(dbname_ + "/" + archived_log->PathName())); ASSERT_TRUE(!env_->FileExists(dbname_ + "/" + archived_log->PathName()));
CloseDB();
} }
} //namespace rocksdb } //namespace rocksdb