From c58bac701c49206419ace216c04ac163eb77517b Mon Sep 17 00:00:00 2001
From: Mayank Pundir <mpundir@dev4790.prn1.facebook.com>
Date: Wed, 23 Sep 2015 17:24:20 -0700
Subject: [PATCH] Fix valgrind failure due to memory leaks

Summary: Test cases for IsBottommostLevel function create FileMetaData objects which were not getting deleted in the destructor.

Test Plan: Valgrind check on compaction_picker_test

Reviewers: yhchiang, igor, sdong

Subscribers: rven, kradhakrishnan, IslamAbdelRahman, dhruba, anthony

Differential Revision: https://reviews.facebook.net/D47463
---
 db/compaction_picker_test.cc | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/db/compaction_picker_test.cc b/db/compaction_picker_test.cc
index bb4afb510..46b616fce 100644
--- a/db/compaction_picker_test.cc
+++ b/db/compaction_picker_test.cc
@@ -70,6 +70,12 @@ class CompactionPickerTest : public testing::Test {
   void DeleteVersionStorage() {
     vstorage_.reset();
     files_.clear();
+    for (uint32_t i = 0; i < input_files_.size(); ++i) {
+      for (uint32_t j = 0; j < input_files_[i].files.size(); ++j) {
+        delete input_files_[i].files[j];
+      }
+      input_files_[i].files.clear();
+    }
     input_files_.clear();
   }
 
-- 
GitLab