fixing null termination of the temporary buffer

Author: d-led

src/influxdb-cpp-rest/influxdb_line.h

@@ -88,7 +88,7 @@ namespace influxdb {
             }
 
             inline std::string get() const {
-                return res.data();
+                return std::string(res.data(),res.size());
             }
 
             inline bool empty() const {
@@ -174,7 +174,7 @@ namespace influxdb {
             }
         public:
             inline std::string get() const {
-                return res.data();
+                return std::string(res.data(),res.size());
             }
         };
 

src/influxdb-cpp-rest/influxdb_simple_async_api.cpp

@@ -68,7 +68,7 @@ struct influxdb::async_api::simple_db::impl {
                         .subscribe([this](std::shared_ptr<fmt::memory_buffer> const& w) {
                             if (w->size() > 0u) {
                                 try {
-                                    db.insert_async(w->data());
+                                    db.insert_async(std::string(w->data(),w->size()));
                                 } catch (const std::runtime_error& e) {
                                     throw std::runtime_error(std::string("async_api::insert failed: ") + e.what() + " -> Dropping " + std::to_string(w->size()) + " bytes");
                                 }

src/script/test.sh

@@ -2,8 +2,11 @@
 set -euo pipefail
 IFS=$'\n\t'
 
+# first parameter to the test script (directory with the test binaries) is mandatory
+echo "running tests from $1"
+pkill influxd || true
 ./influxdb-1.7.6-1/usr/bin/influxd &
-sleep 1
+sleep 3
 $1/test-influxdb-cpp-rest -d yes
 export LD_LIBRARY_PATH=$1
 $1/test-influxdb-c-rest -d yes