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

Fix DBOptionsTest.GetLatestOptions

Browse Source 
Summary: RandomInitCFOptions will allocate a new compaction filter, which we have to delete afterward.

Test Plan: valgrind against the test

Reviewers: IslamAbdelRahman

Reviewed By: IslamAbdelRahman

Subscribers: andrewkr, dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D64011
This commit is contained in:
Yi Wu 2016-09-15 14:57:32 -07:00
parent 40cfa3e021
commit 8d9bf5c498
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
db/db_options_test.cc
View File

@ -47,7 +47,9 @@ class DBOptionsTest : public DBTestBase {
Options options; Options options;
test::RandomInitCFOptions(&options, rnd); test::RandomInitCFOptions(&options, rnd);
auto sanitized_options = SanitizeOptions(options, nullptr, options); auto sanitized_options = SanitizeOptions(options, nullptr, options);
return GetMutableCFOptionsMap(sanitized_options); auto opt_map = GetMutableCFOptionsMap(sanitized_options);
delete options.compaction_filter;
return opt_map;
} }
#endif // ROCKSDB_LITE #endif // ROCKSDB_LITE
}; };