]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
radosstriper : Fixed semaphore cleanup and completion lifetime in striper aio test
authorSebastien Ponce <sebastien.ponce@cern.ch>
Wed, 7 Sep 2016 11:24:51 +0000 (13:24 +0200)
committerroot <root@lxbre43a05.cern.ch>
Tue, 1 Nov 2016 15:36:46 +0000 (16:36 +0100)
Signed-off-by: Sebastien Ponce <sebastien.ponce@cern.ch>
src/test/libradosstriper/aio.cc

index 8d92cbbdf10ebaef9729396c7f5d4e0057ab8c61..6c3d612cdd2ddb7df5e4e2c665be8e45cc0d2d0a 100644 (file)
@@ -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();
 }