bin/memflush: fix exit code on connection failure
authorMichael Wallner <mike@php.net>
Mon, 14 Sep 2020 12:43:59 +0000 (14:43 +0200)
committerMichael Wallner <mike@php.net>
Mon, 14 Sep 2020 12:43:59 +0000 (14:43 +0200)
src/bin/memflush.cc

index 8cd933ff132da914db2755cbad02c88de895350b..49d691b8ed7320de626eb36b5c7f03252494280c 100644 (file)
@@ -59,9 +59,12 @@ int main(int argc, char *argv[])
   if (servers == NULL or memcached_server_list_count(servers) == 0)
   {
     std::cerr << "Invalid server list provided:" << opt_servers << std::endl;
+    free(opt_servers);
     return EXIT_FAILURE;
   }
 
+  free(opt_servers);
+
   memcached_st *memc= memcached_create(NULL);
   memcached_server_push(memc, servers);
   memcached_server_list_free(servers);
@@ -90,12 +93,11 @@ int main(int argc, char *argv[])
   if (rc != MEMCACHED_SUCCESS)
   {
     std::cerr << memcached_last_error_message(memc) << std::endl;
+    memcached_free(memc);
+    return EXIT_FAILURE;
   }
 
   memcached_free(memc);
-
-  free(opt_servers);
-
   return EXIT_SUCCESS;
 }