[2850] Destroy ZoneTableSegment object upon exception (see full log)

Ideally, this should use something like a SegmentObjectHolder, but a
SegmentObjectHolder takes unnecessary arguments. In this limited
usecase, ZoneTableSegment::destroy() just calls its destructor, so
std::auto_ptr should be ok here.

diff --git a/src/lib/datasrc/tests/memory/zone_table_segment_mapped_unittest.cc b/src/lib/datasrc/tests/memory/zone_table_segment_mapped_unittest.cc
index d7bffa1d017163ffe7dfd791adf319a0a97e757d..8a6fcaf03ae6117ebcfe9ae0232eb2315d3a9baa 100644 (file)
--- a/src/lib/datasrc/tests/memory/zone_table_segment_mapped_unittest.cc
+++ b/src/lib/datasrc/tests/memory/zone_table_segment_mapped_unittest.cc
@@ -22,6 +22,7 @@
 #include <boost/scoped_ptr.hpp>
 #include <boost/interprocess/file_mapping.hpp>
 
+#include <memory>
 #include <cerrno>
 
 #include <sys/stat.h>
@@ -51,17 +52,17 @@ protected:
             Element::fromJSON(
                 "{\"mapped-file\": \"" + std::string(mapped_file2) + "\"}"))
     {
-        EXPECT_NE(static_cast<void*>(NULL), ztable_segment_);
+        EXPECT_NE(static_cast<void*>(NULL), ztable_segment_.get());
         // Verify that a ZoneTableSegmentMapped is created.
         ZoneTableSegmentMapped* mapped_segment =
-            dynamic_cast<ZoneTableSegmentMapped*>(ztable_segment_);
+            dynamic_cast<ZoneTableSegmentMapped*>(ztable_segment_.get());
         EXPECT_NE(static_cast<void*>(NULL), mapped_segment);
 
         createTestData();
     }
 
     ~ZoneTableSegmentMappedTest() {
-        ZoneTableSegment::destroy(ztable_segment_);
+        ZoneTableSegment::destroy(ztable_segment_.release());
         boost::interprocess::file_mapping::remove(mapped_file);
         boost::interprocess::file_mapping::remove(mapped_file2);
     }
@@ -81,7 +82,9 @@ protected:
     void addData(MemorySegment& segment);
     bool verifyData(const MemorySegment& segment);
 
-    ZoneTableSegment* ztable_segment_;
+    // Ideally, this should be something similar to a
+    // SegmentObjectHolder, not an auto_ptr.
+    std::auto_ptr<ZoneTableSegment> ztable_segment_;
     const ConstElementPtr config_params_;
     const ConstElementPtr config_params2_;
     std::vector<TestDataElement> test_data_;