Fixed SVACE issues

Change-Id: Ibc8e09939f74e23d46fb383cedf357b68d3ddccb
Signed-off-by: Manasij Sur Roy <manasij.r@samsung.com>

diff --git a/bt_roaming_agent/src/bt_server.c b/bt_roaming_agent/src/bt_server.c
index 7603002..ce0ed1a 100644 (file)
--- a/bt_roaming_agent/src/bt_server.c
+++ b/bt_roaming_agent/src/bt_server.c
@@ -143,6 +143,9 @@ __bt_send_data(int sock_fd, char *data)
        int ret = bt_socket_send_data(sock_fd, full_data, full_data_len);
        _INFO("bt_socket_send_data=[%d]", ret);
 
+       SAFE_DELETE(data);
+       SAFE_DELETE(full_data);
+
        return 0;
 }
 
@@ -162,9 +165,6 @@ __receive_finished(bt_server_t *server)
        server->expected_len = 0;
        server->rem_len = 0;
 
-       char *fullDataString = (char*)calloc(server->client_req_len + 1, sizeof(char));
-       memcpy(fullDataString, server->client_req, server->client_req_len);
-
        _INFO("bluetooth Total pack length = [%d]", server->client_req_len);
 
        /*Call ASM dbus and receive the response*/
@@ -331,6 +331,7 @@ __bt_sock_data_received_cb(bt_socket_received_data_s* data, void* user_data)
                sscanf(d_len_str, "%d", &d_len);
 
                _INFO("Data Len frame=[%d]", d_len);
+               SAFE_DELETE(d_len_str);
 
                server->expected_len = d_len;
 

diff --git a/roaming_agent/src/iotcon_handler.c b/roaming_agent/src/iotcon_handler.c
index 3dc8bad..1703d64 100644 (file)
--- a/roaming_agent/src/iotcon_handler.c
+++ b/roaming_agent/src/iotcon_handler.c
@@ -242,7 +242,7 @@ _create_ra_representation(ra_resource_s *ra, char *tlvRespB64)
        /*TODO:Call Mobile ASM with the incoming query (Base64 decoded TLV)*/
        //char tlv_b64_dec[5000] = {0,};
        //snprintf(tlv_b64_dec, 4999, "%s", ASM_GET_INFO_RESP);
-       iotcon_attributes_add_str(state, RA_QUERY_KEY_RESPONSE, tlvRespB64);
+       ret = iotcon_attributes_add_str(state, RA_QUERY_KEY_RESPONSE, tlvRespB64);
        if (IOTCON_ERROR_NONE != ret) {
                iotcon_attributes_destroy(state);
                iotcon_representation_destroy(repr);