From: Sebastien Ponce <sebastien.ponce@cern.ch>
Date: Wed, 7 Sep 2016 11:24:51 +0000 (+0200)
Subject: radosstriper : Fixed semaphore cleanup and completion lifetime in striper aio test
X-Git-Tag: v11.1.0~355^2~4
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ce944e826556fe9ec67cde3991106d89106df8ae;p=ceph.git

radosstriper : Fixed semaphore cleanup and completion lifetime in striper aio test

Signed-off-by: Sebastien Ponce <sebastien.ponce@cern.ch>
---

diff --git a/src/test/libradosstriper/aio.cc b/src/test/libradosstriper/aio.cc
index 8d92cbbdf10..6c3d612cdd2 100644
--- a/src/test/libradosstriper/aio.cc
+++ b/src/test/libradosstriper/aio.cc
@@ -22,6 +22,7 @@ public:
   }
 
   ~AioTestData() {
+    sem_unlink("test_libradosstriper_aio_sem");
     sem_close(m_sem);
   }
 
@@ -603,8 +604,8 @@ TEST_F(StriperTestPP, RemoveTestPP) {
   bufferlist bl;
   bl.append(buf, sizeof(buf));
   ASSERT_EQ(0, striper.write("RemoveTestPP", bl, sizeof(buf), 0));
-  boost::scoped_ptr<AioCompletion> my_completion(cluster.aio_create_completion(0, 0, 0));
-  ASSERT_EQ(0, striper.aio_remove("RemoveTestPP", my_completion.get()));
+  AioCompletion *my_completion = cluster.aio_create_completion(0, 0, 0);
+  ASSERT_EQ(0, striper.aio_remove("RemoveTestPP", my_completion));
   {
     TestAlarm alarm;
     ASSERT_EQ(0, my_completion->wait_for_complete());
@@ -612,4 +613,5 @@ TEST_F(StriperTestPP, RemoveTestPP) {
   ASSERT_EQ(0, my_completion->get_return_value());
   bufferlist bl2;
   ASSERT_EQ(-ENOENT, striper.read("RemoveTestPP", &bl2, sizeof(buf), 0));
+  my_completion->release();
 }