Fix String Attribute Writes

Used invalid pointer to string array before.
openPMD · Jun 18, 2018 · d5189ce · d5189ce
1 parent caba260
commit d5189ce
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/src/IO/ADIOS/ADIOS1IOHandler.cpp b/src/IO/ADIOS/ADIOS1IOHandler.cpp
@@ -718,7 +718,7 @@ ADIOS1IOHandlerImpl::writeAttribute(Writable* writable,
 
     Attribute att = Attribute(parameters.resource);
 
-    int nelems;
+    int nelems = 0;
     switch( parameters.dtype )
     {
         using DT = Datatype;
@@ -841,8 +841,10 @@ ADIOS1IOHandlerImpl::writeAttribute(Writable* writable,
         case DT::STRING:
         {
             using uptr = std::unique_ptr< void, std::function< void(void*) > >;
-            values = uptr(const_cast< char* >(att.get< std::string >().c_str()),
-                          [](void*){ });
+            auto const & v = att.get< std::string >();
+            values = uptr(new char[v.length() + 1u],
+                          [](void* ca){ delete [] (char*)ca; });
+            strcpy((char*)values.get(), v.c_str());
             break;
         }
         case DT::VEC_CHAR:
@@ -977,7 +979,7 @@ ADIOS1IOHandlerImpl::writeAttribute(Writable* writable,
                                             "",
                                             getBP1DataType(parameters.dtype),
                                             nelems,
-                                            values.get());
+                                            values.get()); // Invalid read of size 1 (could be in ADIOS itself?)
     VERIFY(status == err_no_error, "Internal error: Failed to define ADIOS attribute by value");
 
     if( parameters.dtype == Datatype::VEC_STRING )
@@ -1053,9 +1055,9 @@ ADIOS1IOHandlerImpl::readAttribute(Writable* writable,
         attrname += "/";
     attrname += parameters.name;
 
-    ADIOS_DATATYPES datatype;
-    int size;
-    void* data;
+    ADIOS_DATATYPES datatype = adios_unknown;
+    int size = 0;
+    void* data = nullptr;
 
     int status;
     status = adios_get_attr(f,