On Fri, Dec 18, 2020 at 11:27:14PM +0100, mwi...@suse.com wrote:
> From: Martin Wilck <mwi...@suse.com>
> 
> They break "make valgrind-test".
> 
Reviewed-by: Benjamin Marzinski <bmarz...@redhat.com>
> Signed-off-by: Martin Wilck <mwi...@suse.com>
> ---
>  tests/mpathvalid.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/tests/mpathvalid.c b/tests/mpathvalid.c
> index 5ffabb9..cfe4bae 100644
> --- a/tests/mpathvalid.c
> +++ b/tests/mpathvalid.c
> @@ -381,6 +381,7 @@ static void test_mpathvalid_is_path_good2(void **state)
>       assert_int_equal(mpathvalid_is_path(test_dev, MPATH_DEFAULT, &wwid,
>                                           wwids, 4), MPATH_IS_VALID);
>       assert_string_equal(wwid, TEST_WWID);
> +     free(wwid);
>  }
>  
>  static void test_mpathvalid_is_path_good3(void **state)
> @@ -395,6 +396,7 @@ static void test_mpathvalid_is_path_good3(void **state)
>       assert_int_equal(mpathvalid_is_path(test_dev, MPATH_SMART, &wwid,
>                                           wwids, 4), MPATH_IS_VALID);
>       assert_string_equal(wwid, TEST_WWID);
> +     free(wwid);
>  }
>  
>  /* mabybe valid with no matching paths */
> @@ -410,6 +412,7 @@ static void test_mpathvalid_is_path_good4(void **state)
>       assert_int_equal(mpathvalid_is_path(test_dev, MPATH_DEFAULT, &wwid,
>                                           wwids, 4), MPATH_IS_MAYBE_VALID);
>       assert_string_equal(wwid, TEST_WWID);
> +     free(wwid);
>  }
>  
>  /* maybe valid with matching paths */
> @@ -425,6 +428,7 @@ static void test_mpathvalid_is_path_good5(void **state)
>       assert_int_equal(mpathvalid_is_path(test_dev, MPATH_DEFAULT, &wwid,
>                                           wwids, 4), MPATH_IS_VALID);
>       assert_string_equal(wwid, TEST_WWID);
> +     free(wwid);
>  }
>  
>  #define setup_test(name) \
> -- 
> 2.29.0

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to