serge1 / coffi Goto Github PK

diff --git a/coffi/coffi.hpp b/coffi/coffi.hpp
index 36cb6a4..d2a4156 100644
--- a/coffi/coffi.hpp
+++ b/coffi/coffi.hpp
@@ -646,24 +646,24 @@ class coffi : public coffi_strings,
     {
         std::streampos pos = stream.tellg();
         for (int i = 0; i < coff_header_->get_sections_count(); ++i) {
-            section* sec;
+            std::unique_ptr<section> sec;
             switch (architecture_) {
             case COFFI_ARCHITECTURE_PE:
             case COFFI_ARCHITECTURE_CEVA:
-                sec = new section_impl(this, this, this);
+                sec = std::make_unique<section_impl>(this, this, this);
                 break;
             case COFFI_ARCHITECTURE_TI:
-                sec = new section_impl_ti(this, this, this);
+                sec = std::make_unique<section_impl_ti>(this, this, this);
                 break;
             default:
-                sec = new section_impl(this, this, this);
+                sec = std::make_unique<section_impl>(this, this, this);
                 break;
             }
             if (!(sec->load(stream, i * sec->get_sizeof() + pos))) {
                 return false;
             }
             sec->set_index(i);
-            sections_.push_back(sec);
+            sections_.push_back(sec.release());
         }

         return true;
diff --git a/coffi/coffi_directory.hpp b/coffi/coffi_directory.hpp
index 3cde467..0999b05 100644
--- a/coffi/coffi_directory.hpp
+++ b/coffi/coffi_directory.hpp
@@ -196,11 +196,11 @@ class directories : public std::vector<directory*>
     {
         for (uint32_t i = 0;
              i < scn_->get_win_header()->get_number_of_rva_and_sizes(); ++i) {
-            directory* d = new directory(i);
+            std::unique_ptr<directory> d = std::make_unique< directory>(i);
             if (!d->load(stream)) {
                 return false;
             }
-            push_back(d);
+            push_back(d.release());
         }
         return true;
     }

diff --git a/coffi/coffi_relocation.hpp b/coffi/coffi_relocation.hpp
index 7cbec1f..d604f52 100644
--- a/coffi/coffi_relocation.hpp
+++ b/coffi/coffi_relocation.hpp
@@ -167,7 +167,7 @@ class relocation
     const symbol_provider*         sym_;
     const architecture_provider*   arch_;
     std::string                    symbol_name;
-    rel_entry_generic              header;
+    rel_entry_generic              header{};
 };

 } // namespace COFFI
diff --git a/coffi/coffi_section.hpp b/coffi/coffi_section.hpp
index 39de317..b92117d 100644
--- a/coffi/coffi_section.hpp
+++ b/coffi/coffi_section.hpp
@@ -354,7 +354,7 @@ template <class T> class section_impl_tmpl : public section

     //------------------------------------------------------------------------------
     T                        header;
-    uint32_t                 index;
+    uint32_t                 index{};
     std::string              name;
     char*                    data_;
     uint32_t                 data_reserved_;
diff --git a/coffi/coffi_directory.hpp b/coffi/coffi_directory.hpp
index 3cde467..6780b5f 100644
--- a/coffi/coffi_directory.hpp
+++ b/coffi/coffi_directory.hpp
@@ -163,7 +163,7 @@ class directory
     }

   private:
-    image_data_directory header;
+    image_data_directory header{};
     const char*          data_;
     uint32_t             index_;
 };