Skip to content

Commit

Permalink
fix valgrind error in c_test caused by BlockBasedTableOptions
Browse files Browse the repository at this point in the history
Summary:
It was creating BlockBasedTableOptions object in a loop without calling
destroy()

Test Plan: valgrind ./c_test --leak-check=full --show-reachable=yes

Reviewers: sdong, igor

Reviewed By: igor

Subscribers: leveldb

Differential Revision: https://reviews.facebook.net/D22431
  • Loading branch information
Lei Jin committed Aug 26, 2014
1 parent ff6ec0e commit bda6f33
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion db/c_test.c
Original file line number Diff line number Diff line change
Expand Up @@ -542,7 +542,6 @@ int main(int argc, char** argv) {
policy = rocksdb_filterpolicy_create_bloom(10);
}

table_options = rocksdb_block_based_options_create();
rocksdb_block_based_options_set_filter_policy(table_options, policy);

// Create new database
Expand Down

0 comments on commit bda6f33

Please sign in to comment.