On success, we should exit the loop.

[m6w6/libmemcached] / tests / deprecated.cc

diff --git a/tests/deprecated.cc b/tests/deprecated.cc
index bf0e0c6dbe6ebe75379549c327285c59aa2dc47e..e98faf406678464af49e6a0a80e4b4ebee5d82e6 100644 (file)
--- a/tests/deprecated.cc
+++ b/tests/deprecated.cc
@@ -35,7 +35,8 @@
  *
  */
 
-#include <libtest/common.h>
+#include <config.h>
+#include <libtest/test.hpp>
 
 #include <cassert>
 
@@ -49,13 +50,16 @@ test_return_t server_list_null_test(memcached_st *ptr)
   (void)ptr;
 
   server_list= memcached_server_list_append_with_weight(NULL, NULL, 0, 0, NULL);
-  test_true(server_list == NULL);
+  test_true(server_list);
+  memcached_server_list_free(server_list);
 
   server_list= memcached_server_list_append_with_weight(NULL, "localhost", 0, 0, NULL);
-  test_true(server_list == NULL);
+  test_true(server_list);
+  memcached_server_list_free(server_list);
 
   server_list= memcached_server_list_append_with_weight(NULL, NULL, 0, 0, &rc);
-  test_true(server_list == NULL);
+  test_true(server_list);
+  memcached_server_list_free(server_list);
 
   return TEST_SUCCESS;
 }
@@ -63,7 +67,7 @@ test_return_t server_list_null_test(memcached_st *ptr)
 // Look for memory leak
 test_return_t regression_bug_728286(memcached_st *)
 {
-  memcached_server_st *servers = memcached_servers_parse("1.2.3.4:99");
+  memcached_server_st *servers= memcached_servers_parse("1.2.3.4:99");
   assert(servers);
   memcached_server_free(servers);