From: Sage Weil <sage@redhat.com>
Date: Tue, 29 Dec 2015 18:10:17 +0000 (-0500)
Subject: os/bluestore/BlueFS: fix leak of iocs in writer
X-Git-Tag: v10.0.3~154^2~17
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=3f4bd46735a574a92ded4f9a40738d56a4202b0e;p=ceph.git

os/bluestore/BlueFS: fix leak of iocs in writer

Signed-off-by: Sage Weil <sage@redhat.com>
---

diff --git a/src/os/bluestore/BlueFS.h b/src/os/bluestore/BlueFS.h
index e682dec1121..cbdefff3c00 100644
--- a/src/os/bluestore/BlueFS.h
+++ b/src/os/bluestore/BlueFS.h
@@ -67,16 +67,12 @@ public:
     bufferlist tail_block;  ///< existing partial block at end of file, if any
 
     Mutex lock;
-    /*Cond cond;
-    bool num_aio_in_flight;
-    */
     vector<IOContext*> iocv;  ///< one for each bdev
 
     FileWriter(FileRef f, unsigned num_bdev)
       : file(f),
 	pos(0),
-	lock("BlueFS::FileWriter::lock") { //,
-	//num_aio_in_flight(0) {
+	lock("BlueFS::FileWriter::lock") {
       file->num_writers.inc();
       iocv.resize(num_bdev);
       for (unsigned i = 0; i < num_bdev; ++i) {
@@ -85,6 +81,9 @@ public:
     }
     ~FileWriter() {
       file->num_writers.dec();
+      for (auto p : iocv) {
+	delete p;
+      }
     }
 
     void append(const char *buf, size_t len) {