dolysis/rocksdb
3
0
Fork 0
mirror of https://github.com/facebook/rocksdb.git synced 2024-11-27 02:44:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fixing some issues Valgrind found

Browse source 
Summary: Found some issues running Valgrind on `db_test` (there are still some outstanding ones) and fixed them.

Test Plan:
make check

ran `valgrind ./db_test` and saw that errors no longer occur

Reviewers: dhruba, vamsi, emayanke, sheki

Reviewed By: dhruba

CC: leveldb

Differential Revision: https://reviews.facebook.net/D7803
This commit is contained in:
Kosie van der Merwe 2013-01-08 12:16:40 -08:00
parent 4d339d7462
commit d8371ef1f6
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
db/db_impl.cc
View file

@ -262,7 +262,7 @@ DBImpl::~DBImpl() {
delete options_.block_cache;
}
if (options_.compression_per_level != NULL) {
delete options_.compression_per_level;
delete[] options_.compression_per_level;
}
delete logger_;
@ -1183,7 +1183,7 @@ void DBImpl::BGWork(void* db) {
}
void DBImpl::BackgroundCall() {
bool madeProgress;
bool madeProgress = false;
DeletionState deletion_state;
MutexLock l(&mutex_);
// Log(options_.info_log, "XXX BG Thread %llx process new work item", pthread_self());