fix ressource leak in move assignment operator

FlexibleSUSY · Aug 26, 2016 · 5fffa02 · 5fffa02
1 parent 1da8ee7
commit 5fffa02
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/gsl_vector.cpp b/src/gsl_vector.cpp
@@ -127,6 +127,9 @@ const GSL_vector& GSL_vector::operator=(const GSL_vector& rhs)
 
 GSL_vector& GSL_vector::operator=(GSL_vector&& rhs) noexcept
 {
+   if (vec)
+      gsl_vector_free(vec);
+
    if (this != &rhs)
       move_assign(std::move(rhs));
 

diff --git a/test/test_gsl_vector.cpp b/test/test_gsl_vector.cpp
@@ -141,7 +141,7 @@ BOOST_AUTO_TEST_CASE( test_move_assign )
    v[1] = 2.;
    v[2] = 3.;
 
-   GSL_vector v2;
+   GSL_vector v2(2);
    v2 = std::move(v);
 
    BOOST_CHECK(!v.raw());