tests: Free after use 31/256631/3
authorCheoleun Moon <chleun.moon@samsung.com>
Thu, 8 Apr 2021 07:46:02 +0000 (16:46 +0900)
committerCheoleun Moon <chleun.moon@samsung.com>
Thu, 8 Apr 2021 07:54:26 +0000 (16:54 +0900)
Change-Id: I92b74905b6fc64567be9ae748f13bc1882b38074
Signed-off-by: Cheoleun Moon <chleun.moon@samsung.com>
tests/unittest/vine-unittest-security.cpp
tests/verifier/vine-verifier.cpp
tests/vine-test/vine-multi-thread-test.cpp
tests/vine-test/vine-test.cpp

index 771a3d9..f8b4461 100755 (executable)
@@ -298,4 +298,5 @@ TEST_F(VineSecurityTest, GetPskP)
 
        EXPECT_EQ(VINE_ERROR_NONE,
                        vine_security_get_psk(security, &psk));
+       free(psk);
 }
index fc10a37..bd33951 100644 (file)
@@ -244,6 +244,7 @@ static void __register_service(vine_session_h session, int port)
 
        vine_session_set_registered_cb(session, __registered_cb, NULL);
        PRINT_IF_ERROR(vine_session_register(session, service, NULL), "vine_session_register");
+       vine_service_destroy(service);
 }
 
 static void __server_opened_cb(vine_dp_h dp, vine_error_e result, void *user_data)
index 7f9a831..1189d46 100644 (file)
@@ -149,6 +149,9 @@ static void __discovered_cb(vine_session_h session, vine_service_h service,
        PRINT_IF_ERROR(ret, "vine_service_foreach_attribute");
        PRINT_LOG("\n");
        fflush(stdout);
+
+       free(service_type);
+       free(service_name);
 }
 
 #define VINE_LOGGER_TIME_MESSAGE_LEN 17
index 2994cb9..4102b23 100644 (file)
@@ -219,6 +219,9 @@ static void __ip_resolved_cb(vine_session_h session, vine_service_h service,
        PRINT_IF_ERROR(ret, "vine_service_foreach_attribute");
        printf("\n");
        fflush(stdout);
+
+       free(service_type);
+       free(service_name);
 }
 
 static void __discovered_cb(vine_session_h session, vine_service_h service,
@@ -554,8 +557,10 @@ static vine_security_h __create_security_handle(int type, bool is_server)
        if (vine_security_create(&s) != VINE_ERROR_NONE)
                return NULL;
 
-       if (vine_security_set_type(s, (vine_security_type_e)type) != VINE_ERROR_NONE)
+       if (vine_security_set_type(s, (vine_security_type_e)type) != VINE_ERROR_NONE) {
+               vine_security_destroy(s);
                return NULL;
+       }
 
        if (type >= VINE_SECURITY_TYPE_TLS) {
                vine_security_set_ca_path(s, ROOT_CA_PATH);
@@ -612,7 +617,9 @@ static void __open_server()
        vine_dp_set_terminated_cb(g_server_dp, __terminated_cb, NULL);
        vine_dp_set_address_family(g_server_dp, (vine_address_family_e)addr_family);
        vine_dp_set_port(g_server_dp, port);
-       vine_dp_set_security(g_server_dp, __create_security_handle(security_type, true));
+       vine_security_h security = __create_security_handle(security_type, true);
+       vine_dp_set_security(g_server_dp, security);
+       vine_security_destroy(security);
 
        PRINT_RESULT(vine_dp_open(g_server_dp, __opened_cb, NULL), "vine_dp_open");
 }
@@ -652,7 +659,9 @@ static void __connect_server()
        vine_dp_set_remote_ip(g_client_dp,
                        addr_type ? VINE_ADDRESS_FAMILY_IPV6 : VINE_ADDRESS_FAMILY_IPV4, ip);
        PRINT_RESULT(vine_dp_set_port(g_client_dp, port), "vine_dp_set_port");
-       vine_dp_set_security(g_client_dp, __create_security_handle(security_type, false));
+       vine_security_h security = __create_security_handle(security_type, true);
+       vine_dp_set_security(g_client_dp, security);
+       vine_security_destroy(security);
        PRINT_RESULT(vine_dp_open(g_client_dp, __opened_cb, NULL), "vine_dp_open");
 }