Fix __warn_unused_result__, add more of it and fix the resulting warnings.

[meshlink] / test / blackbox / run_blackbox_tests / test_cases_random_port_bindings01.c

diff --git a/test/blackbox/run_blackbox_tests/test_cases_random_port_bindings01.c b/test/blackbox/run_blackbox_tests/test_cases_random_port_bindings01.c
index 24a7e4097e91ea0b27433166a4495051b78059ff..5cce98c0b4e805b296b8b70132b331efe64d9081 100644 (file)
--- a/test/blackbox/run_blackbox_tests/test_cases_random_port_bindings01.c
+++ b/test/blackbox/run_blackbox_tests/test_cases_random_port_bindings01.c
@@ -17,6 +17,10 @@
     51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
 */
 
+#ifdef NDEBUG
+#undef NDEBUG
+#endif
+
 /* Modify this to change the logging level of Meshlink */
 #define TEST_MESHLINK_LOG_LEVEL MESHLINK_DEBUG
 
@@ -138,9 +142,8 @@ void test_case_mesh_random_port_bindings_01(void **state) {
     of the instance should be unchanged.
 */
 bool test_steps_mesh_random_port_bindings_01(void) {
-       struct sockaddr_in servaddr;
        meshlink_handle_t *relay = NULL;
-       meshlink_destroy("relay_conf");
+       assert(meshlink_destroy("relay_conf"));
 
        meshlink_set_log_cb(NULL, MESHLINK_DEBUG, log_message);
 
@@ -168,7 +171,7 @@ bool test_steps_mesh_random_port_bindings_01(void) {
        ipv6_fd = -1;
 
        meshlink_close(relay);
-       meshlink_destroy("relay_conf");
+       assert(meshlink_destroy("relay_conf"));
 
        return true;
 }
@@ -189,9 +192,8 @@ void test_case_mesh_random_port_bindings_02(void **state) {
     The meshlink_set_port() API should fail.
 */
 bool test_steps_mesh_random_port_bindings_02(void) {
-       int port = -1;
        meshlink_handle_t *relay = NULL;
-       meshlink_destroy("relay_conf");
+       assert(meshlink_destroy("relay_conf"));
 
        meshlink_set_log_cb(NULL, MESHLINK_DEBUG, log_message);
 
@@ -205,7 +207,6 @@ bool test_steps_mesh_random_port_bindings_02(void) {
        assert_true(meshlink_start(relay));
 
        sleep(1);
-       port = meshlink_get_port(relay);
 
        devtool_trybind_probe = occupy_trybind_port;
        meshlink_stop(relay);
@@ -219,7 +220,7 @@ bool test_steps_mesh_random_port_bindings_02(void) {
        ipv6_fd = -1;
 
        meshlink_close(relay);
-       meshlink_destroy("relay_conf");
+       assert(meshlink_destroy("relay_conf"));
        return true;
 }
 
@@ -241,12 +242,9 @@ void test_case_mesh_random_port_bindings_03(void **state) {
     previous port number.
 */
 bool test_steps_mesh_random_port_bindings_03(void) {
-       int mode = 1;
        int port, new_port;
-       struct sockaddr_in servaddr;
-       struct sockaddr_in6 ipv6addr;
        meshlink_handle_t *relay = NULL;
-       meshlink_destroy("relay_conf");
+       assert(meshlink_destroy("relay_conf"));
 
        meshlink_set_log_cb(NULL, MESHLINK_DEBUG, log_message);
 
@@ -284,7 +282,7 @@ bool test_steps_mesh_random_port_bindings_03(void) {
        ipv6_fd = -1;
 
        meshlink_close(relay);
-       meshlink_destroy("relay_conf");
+       assert(meshlink_destroy("relay_conf"));
        return true;
 }