Fixed memory leak for vconf_get_str
[platform/core/api/connection.git] / src / connection.c
index 3d13899..3901e2e 100755 (executable)
@@ -23,6 +23,7 @@
 
 static __thread GSList *conn_handle_list = NULL;
 
+//LCOV_EXCL_START
 static int __connection_convert_net_state(int status)
 {
        switch (status) {
@@ -35,7 +36,7 @@ static int __connection_convert_net_state(int status)
        case VCONFKEY_NETWORK_BLUETOOTH:
                return CONNECTION_TYPE_BT;
        case VCONFKEY_NETWORK_DEFAULT_PROXY:
-               return CONNECTION_TYPE_PROXY;
+               return CONNECTION_TYPE_NET_PROXY;
        default:
                return CONNECTION_TYPE_DISCONNECTED;
        }
@@ -170,7 +171,7 @@ static int __connection_set_type_changed_callback(connection_h connection,
                        if (--refcount == 0) {
                                if (vconf_ignore_key_changed(VCONFKEY_NETWORK_STATUS,
                                                __connection_cb_type_change_cb) < 0) {
-                                       CONNECTION_LOG(CONNECTION_ERROR,
+                                       CONNECTION_LOG(CONNECTION_ERROR, //LCOV_EXCL_LINE
                                                        "Error to de-register vconf callback(%d)", refcount);
                                } else {
                                        CONNECTION_LOG(CONNECTION_INFO,
@@ -209,6 +210,9 @@ static gboolean __connection_cb_ip_changed_cb_idle(gpointer user_data)
                return FALSE;
 
        ip_addr = vconf_get_str(VCONFKEY_NETWORK_IP);
+       if (ip_addr == NULL)
+               CONNECTION_LOG(CONNECTION_ERROR, //LCOV_EXCL_LINE
+                       "vconf_get_str(VCONFKEY_NETWORK_IP) is Failed");
 
        callback = __connection_get_ip_changed_callback(local_handle);
        data = __connection_get_ip_changed_userdata(local_handle);
@@ -216,6 +220,8 @@ static gboolean __connection_cb_ip_changed_cb_idle(gpointer user_data)
        if (callback)
                callback(ip_addr, NULL, data);
 
+       g_free(ip_addr);
+
        return FALSE;
 }
 
@@ -257,7 +263,7 @@ static int __connection_set_ip_changed_callback(connection_h connection,
                        if (--refcount == 0) {
                                if (vconf_ignore_key_changed(VCONFKEY_NETWORK_IP,
                                                __connection_cb_ip_change_cb) < 0) {
-                                       CONNECTION_LOG(CONNECTION_ERROR,
+                                       CONNECTION_LOG(CONNECTION_ERROR, //LCOV_EXCL_LINE
                                                        "Error to de-register vconf callback(%d)", refcount);
                                } else {
                                        CONNECTION_LOG(CONNECTION_INFO,
@@ -296,6 +302,9 @@ static gboolean __connection_cb_proxy_changed_cb_idle(gpointer user_data)
                return FALSE;
 
        proxy = vconf_get_str(VCONFKEY_NETWORK_PROXY);
+       if (proxy == NULL)
+               CONNECTION_LOG(CONNECTION_ERROR, //LCOV_EXCL_LINE
+                       "vconf_get_str(VCONFKEY_NETWORK_PROXY) is Failed");
 
        callback = __connection_get_proxy_changed_callback(local_handle);
        data = __connection_get_proxy_changed_userdata(local_handle);
@@ -303,6 +312,8 @@ static gboolean __connection_cb_proxy_changed_cb_idle(gpointer user_data)
        if (callback)
                callback(proxy, NULL, data);
 
+       g_free(proxy);
+
        return FALSE;
 }
 
@@ -344,7 +355,7 @@ static int __connection_set_proxy_changed_callback(connection_h connection,
                        if (--refcount == 0) {
                                if (vconf_ignore_key_changed(VCONFKEY_NETWORK_PROXY,
                                                __connection_cb_proxy_change_cb) < 0) {
-                                       CONNECTION_LOG(CONNECTION_ERROR,
+                                       CONNECTION_LOG(CONNECTION_ERROR, //LCOV_EXCL_LINE
                                                        "Error to de-register vconf callback(%d)", refcount);
                                } else {
                                        CONNECTION_LOG(CONNECTION_INFO,
@@ -379,6 +390,7 @@ static int __connection_set_ethernet_cable_state_changed_cb(connection_h connect
        local_handle->ethernet_cable_state_changed_user_data = user_data;
        return CONNECTION_ERROR_NONE;
 }
+//LCOV_EXCL_STOP
 
 static int __connection_get_handle_count(void)
 {
@@ -397,19 +409,18 @@ EXPORT_API int connection_create(connection_h *connection)
 
        int rv = _connection_libnet_init();
        if (rv == NET_ERR_ACCESS_DENIED) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Access denied");
-               return CONNECTION_ERROR_PERMISSION_DENIED;
-       }
-       else if (rv != NET_ERR_NONE) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Failed to create connection[%d]", rv);
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Access denied"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_PERMISSION_DENIED; //LCOV_EXCL_LINE
+       } else if (rv != NET_ERR_NONE) {
+               CONNECTION_LOG(CONNECTION_ERROR, "Failed to create connection[%d]", rv); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
        }
 
        *connection = g_try_malloc0(sizeof(connection_handle_s));
        if (*connection != NULL)
                CONNECTION_LOG(CONNECTION_INFO, "New handle created[%p]", *connection);
        else
-               return CONNECTION_ERROR_OUT_OF_MEMORY;
+               return CONNECTION_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE
 
        conn_handle_list = g_slist_prepend(conn_handle_list, *connection);
 
@@ -459,8 +470,8 @@ EXPORT_API int connection_get_type(connection_h connection, connection_type_e* t
 
        rv = vconf_get_int(VCONFKEY_NETWORK_STATUS, &status);
        if (rv != VCONF_OK) {
-               CONNECTION_LOG(CONNECTION_ERROR, "vconf_get_int Failed = %d", status);
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "vconf_get_int Failed = %d", status); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
        }
 
        CONNECTION_LOG(CONNECTION_INFO, "Connected Network = %d", status);
@@ -491,8 +502,8 @@ EXPORT_API int connection_get_ip_address(connection_h connection,
        }
 
        if (*ip_address == NULL) {
-               CONNECTION_LOG(CONNECTION_ERROR, "vconf_get_str Failed");
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "vconf_get_str Failed"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED;//LCOV_EXCL_LINE
        }
 
        return CONNECTION_ERROR_NONE;
@@ -519,8 +530,8 @@ EXPORT_API int connection_get_proxy(connection_h connection,
        }
 
        if (*proxy == NULL) {
-               CONNECTION_LOG(CONNECTION_ERROR, "vconf_get_str Failed");
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "vconf_get_str Failed"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
        }
 
        return CONNECTION_ERROR_NONE;
@@ -533,10 +544,10 @@ EXPORT_API int connection_get_mac_address(connection_h connection, connection_ty
 
        CHECK_FEATURE_SUPPORTED(WIFI_FEATURE, ETHERNET_FEATURE);
 
-       if(type == CONNECTION_TYPE_WIFI)
+       if (type == CONNECTION_TYPE_WIFI)
                CHECK_FEATURE_SUPPORTED(WIFI_FEATURE);
-       else if(type == CONNECTION_TYPE_ETHERNET)
-               CHECK_FEATURE_SUPPORTED(ETHERNET_FEATURE);
+       else if (type == CONNECTION_TYPE_ETHERNET) //LCOV_EXCL_LINE
+               CHECK_FEATURE_SUPPORTED(ETHERNET_FEATURE); //LCOV_EXCL_LINE
 
        if (mac_addr == NULL || !(__connection_check_handle_validity(connection))) {
                CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter");
@@ -545,15 +556,16 @@ EXPORT_API int connection_get_mac_address(connection_h connection, connection_ty
 
        switch (type) {
        case CONNECTION_TYPE_WIFI:
+#if defined TIZEN_TV
                fp = fopen(WIFI_MAC_INFO_FILE, "r");
                if (fp == NULL) {
-                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to open file %s", WIFI_MAC_INFO_FILE);
-                       return CONNECTION_ERROR_OUT_OF_MEMORY;
+                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to open file %s", WIFI_MAC_INFO_FILE); //LCOV_EXCL_LINE
+                       return CONNECTION_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE
                }
 
                if (fgets(buf, sizeof(buf), fp) == NULL) {
-                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to get MAC info from %s", WIFI_MAC_INFO_FILE);
-                       fclose(fp);
+                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to get MAC info from %s", WIFI_MAC_INFO_FILE); //LCOV_EXCL_LINE
+                       fclose(fp); //LCOV_EXCL_LINE
                        return CONNECTION_ERROR_OPERATION_FAILED;
                }
 
@@ -561,13 +573,22 @@ EXPORT_API int connection_get_mac_address(connection_h connection, connection_ty
 
                *mac_addr = (char *)malloc(CONNECTION_MAC_INFO_LENGTH + 1);
                if (*mac_addr == NULL) {
-                       CONNECTION_LOG(CONNECTION_ERROR, "malloc() failed");
-                       fclose(fp);
-                       return CONNECTION_ERROR_OUT_OF_MEMORY;
+                       CONNECTION_LOG(CONNECTION_ERROR, "malloc() failed"); //LCOV_EXCL_LINE
+                       fclose(fp); //LCOV_EXCL_LINE
+                       return CONNECTION_ERROR_OUT_OF_MEMORY; //LCOV_EXCL_LINE
                }
                g_strlcpy(*mac_addr, buf, CONNECTION_MAC_INFO_LENGTH + 1);
                fclose(fp);
+#else
+               *mac_addr = vconf_get_str(VCONFKEY_WIFI_BSSID_ADDRESS);
+
+               if (*mac_addr == NULL) {
+                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to get vconf from %s", VCONFKEY_WIFI_BSSID_ADDRESS); //LCOV_EXCL_LINE
+                       return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
+               }
+#endif
                break;
+       //LCOV_EXCL_START
        case CONNECTION_TYPE_ETHERNET:
                fp = fopen(ETHERNET_MAC_INFO_FILE, "r");
                if (fp == NULL) {
@@ -590,20 +611,21 @@ EXPORT_API int connection_get_mac_address(connection_h connection, connection_ty
                        return CONNECTION_ERROR_OUT_OF_MEMORY;
                }
 
-               g_strlcpy(*mac_addr, buf,CONNECTION_MAC_INFO_LENGTH + 1);
+               g_strlcpy(*mac_addr, buf, CONNECTION_MAC_INFO_LENGTH + 1);
                fclose(fp);
 
                break;
-       default:
-               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter");
-               return CONNECTION_ERROR_INVALID_PARAMETER;
+       //LCOV_EXCL_STOP
+       default :
+               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE
        }
 
        /* Checking Invalid MAC Address */
-       if((strcmp(*mac_addr, "00:00:00:00:00:00") == 0) ||
+       if ((strcmp(*mac_addr, "00:00:00:00:00:00") == 0) ||
                        (strcmp(*mac_addr, "ff:ff:ff:ff:ff:ff") == 0)) {
-               CONNECTION_LOG(CONNECTION_ERROR, "MAC Address(%s) is invalid", *mac_addr);
-               return CONNECTION_ERROR_INVALID_OPERATION;
+               CONNECTION_LOG(CONNECTION_ERROR, "MAC Address(%s) is invalid", *mac_addr); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_INVALID_OPERATION; //LCOV_EXCL_LINE
        }
 
        CONNECTION_LOG(CONNECTION_INFO, "MAC Address %s", *mac_addr);
@@ -611,6 +633,7 @@ EXPORT_API int connection_get_mac_address(connection_h connection, connection_ty
        return CONNECTION_ERROR_NONE;
 }
 
+
 EXPORT_API int connection_get_cellular_state(connection_h connection, connection_cellular_state_e* state)
 {
        int rv = 0;
@@ -629,8 +652,8 @@ EXPORT_API int connection_get_cellular_state(connection_h connection, connection
 
        rv = vconf_get_int(VCONFKEY_NETWORK_CELLULAR_STATE, &status);
        if (rv != VCONF_OK) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Failed to get cellular state");
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Failed to get cellular state"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
        }
 
        CONNECTION_LOG(CONNECTION_INFO, "Cellular: %d", status);
@@ -662,8 +685,8 @@ EXPORT_API int connection_get_cellular_state(connection_h connection, connection
                }
 #endif
                if (rv != VCONF_OK) {
-                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to get cellular state");
-                       return CONNECTION_ERROR_OPERATION_FAILED;
+                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to get cellular state"); //LCOV_EXCL_LINE
+                       return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
                }
        }
 
@@ -688,8 +711,8 @@ EXPORT_API int connection_get_wifi_state(connection_h connection, connection_wif
 
        int rv = _connection_libnet_get_wifi_state(state);
        if (rv != CONNECTION_ERROR_NONE) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Fail to get Wi-Fi state[%d]", rv);
-               return rv;
+               CONNECTION_LOG(CONNECTION_ERROR, "Fail to get Wi-Fi state[%d]", rv); //LCOV_EXCL_LINE
+               return rv; //LCOV_EXCL_LINE
        }
 
        CONNECTION_LOG(CONNECTION_INFO, "Wi-Fi state: %d", *state);
@@ -697,7 +720,8 @@ EXPORT_API int connection_get_wifi_state(connection_h connection, connection_wif
        return CONNECTION_ERROR_NONE;
 }
 
-EXPORT_API int connection_get_ethernet_state(connection_h connection, connection_ethernet_state_e* state)
+//LCOV_EXCL_START
+EXPORT_API int connection_get_ethernet_state(connection_h connection, connection_ethernet_state_e *state)
 {
        CHECK_FEATURE_SUPPORTED(ETHERNET_FEATURE);
 
@@ -739,7 +763,7 @@ EXPORT_API int connection_unset_ethernet_cable_state_chaged_cb(connection_h conn
 {
        CHECK_FEATURE_SUPPORTED(ETHERNET_FEATURE);
 
-       if ( !(__connection_check_handle_validity(connection)) ) {
+       if (!(__connection_check_handle_validity(connection))) {
                CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter");
                return CONNECTION_ERROR_INVALID_PARAMETER;
        }
@@ -747,8 +771,9 @@ EXPORT_API int connection_unset_ethernet_cable_state_chaged_cb(connection_h conn
        return __connection_set_ethernet_cable_state_changed_cb(connection,
                                                        NULL, NULL);
 }
+//LCOV_EXCL_STOP
 
-EXPORT_API int connection_get_bt_state(connection_h connection, connection_bt_state_estate)
+EXPORT_API int connection_get_bt_state(connection_h connection, connection_bt_state_e *state)
 {
        CHECK_FEATURE_SUPPORTED(TETHERING_BLUETOOTH_FEATURE);
 
@@ -849,24 +874,24 @@ EXPORT_API int connection_add_profile(connection_h connection, connection_profil
        }
 
        if (profile_info->profile_type != NET_DEVICE_CELLULAR) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter");
-               return CONNECTION_ERROR_INVALID_PARAMETER;
+               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE
        }
 
        if (profile_info->ProfileInfo.Pdp.PSModemPath[0] != '/' ||
                        strlen(profile_info->ProfileInfo.Pdp.PSModemPath) < 2) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Modem object path is NULL");
-               return CONNECTION_ERROR_INVALID_PARAMETER;
+               CONNECTION_LOG(CONNECTION_ERROR, "Modem object path is NULL"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE
        }
 
        rv = net_add_profile(profile_info->ProfileInfo.Pdp.ServiceType,
                                                        (net_profile_info_t*)profile);
        if (rv == NET_ERR_ACCESS_DENIED) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Access denied");
-               return CONNECTION_ERROR_PERMISSION_DENIED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Access denied"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_PERMISSION_DENIED; //LCOV_EXCL_LINE
        } else if (rv != NET_ERR_NONE) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Failed to add profile[%d]", rv);
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Failed to add profile[%d]", rv); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
        }
 
        return CONNECTION_ERROR_NONE;
@@ -887,17 +912,17 @@ EXPORT_API int connection_remove_profile(connection_h connection, connection_pro
 
        if (profile_info->profile_type != NET_DEVICE_CELLULAR &&
            profile_info->profile_type != NET_DEVICE_WIFI) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter");
-               return CONNECTION_ERROR_INVALID_PARAMETER;
+               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE
        }
 
        rv = net_delete_profile(profile_info->ProfileName);
        if (rv == NET_ERR_ACCESS_DENIED) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Access denied");
-               return CONNECTION_ERROR_PERMISSION_DENIED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Access denied"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_PERMISSION_DENIED; //LCOV_EXCL_LINE
        } else if (rv != NET_ERR_NONE) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Failed to delete profile[%d]", rv);
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Failed to delete profile[%d]", rv); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
        }
 
        return CONNECTION_ERROR_NONE;
@@ -918,11 +943,11 @@ EXPORT_API int connection_update_profile(connection_h connection, connection_pro
 
        rv = net_modify_profile(profile_info->ProfileName, (net_profile_info_t*)profile);
        if (rv == NET_ERR_ACCESS_DENIED) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Access denied");
-               return CONNECTION_ERROR_PERMISSION_DENIED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Access denied"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_PERMISSION_DENIED; //LCOV_EXCL_LINE
        } else if (rv != NET_ERR_NONE) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Failed to modify profile[%d]", rv);
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Failed to modify profile[%d]", rv); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
        }
 
        return CONNECTION_ERROR_NONE;
@@ -1058,9 +1083,9 @@ EXPORT_API int connection_reset_profile(connection_h connection,
                return CONNECTION_ERROR_INVALID_PARAMETER;
        }
 
-       if(id < 0 || id > 1) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Wrong Parameter Passed");
-               return CONNECTION_ERROR_INVALID_PARAMETER;
+       if (id < 0 || id > 1) {
+               CONNECTION_LOG(CONNECTION_ERROR, "Wrong Parameter Passed"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_INVALID_PARAMETER; //LCOV_EXCL_LINE
        }
 
        return _connection_libnet_reset_profile(type, id, callback, user_data);
@@ -1127,7 +1152,7 @@ static int __get_cellular_statistic(connection_statistics_type_e statistics_type
 #endif
 
        if (llsize == NULL) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter");
+               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter"); //LCOV_EXCL_LINE
                return CONNECTION_ERROR_INVALID_PARAMETER;
        }
 
@@ -1196,12 +1221,12 @@ static int __get_cellular_statistic(connection_statistics_type_e statistics_type
 #endif
 
        if (rv != VCONF_OK || rv1 != VCONF_OK) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Failed to get cellular statistics");
-               return CONNECTION_ERROR_OPERATION_FAILED;
+               CONNECTION_LOG(CONNECTION_ERROR, "Failed to get cellular statistics"); //LCOV_EXCL_LINE
+               return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
        }
 
        *llsize = (long long)(last_size * 1000) + (long long)(size * 1000);
-       CONNECTION_LOG(CONNECTION_INFO,"%lld bytes", *llsize);
+       CONNECTION_LOG(CONNECTION_INFO, "%lld bytes", *llsize);
 
        return CONNECTION_ERROR_NONE;
 }
@@ -1213,7 +1238,7 @@ static int __get_statistic(connection_type_e connection_type,
        unsigned long long ull_size;
 
        if (llsize == NULL) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter");
+               CONNECTION_LOG(CONNECTION_ERROR, "Invalid parameter"); //LCOV_EXCL_LINE
                return CONNECTION_ERROR_INVALID_PARAMETER;
        }
 
@@ -1221,7 +1246,7 @@ static int __get_statistic(connection_type_e connection_type,
        if (rv == CONNECTION_ERROR_PERMISSION_DENIED)
                return rv;
        else if (rv != CONNECTION_ERROR_NONE) {
-               CONNECTION_LOG(CONNECTION_ERROR, "Failed to get statistics");
+               CONNECTION_LOG(CONNECTION_ERROR, "Failed to get statistics"); //LCOV_EXCL_LINE
                return CONNECTION_ERROR_OPERATION_FAILED;
        }
 
@@ -1249,12 +1274,12 @@ static int __get_statistic(connection_type_e connection_type,
                if (rv == CONNECTION_ERROR_PERMISSION_DENIED)
                        return rv;
                else if (rv != CONNECTION_ERROR_NONE) {
-                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to get Wi-Fi statistics");
-                       *llsize = 0;
-                       return CONNECTION_ERROR_OPERATION_FAILED;
+                       CONNECTION_LOG(CONNECTION_ERROR, "Failed to get Wi-Fi statistics"); //LCOV_EXCL_LINE
+                       *llsize = 0; //LCOV_EXCL_LINE
+                       return CONNECTION_ERROR_OPERATION_FAILED; //LCOV_EXCL_LINE
                }
 
-               CONNECTION_LOG(CONNECTION_INFO,"%lld bytes", ull_size);
+               CONNECTION_LOG(CONNECTION_INFO, "%lld bytes", ull_size);
                *llsize = (long long)ull_size;
        } else
                return CONNECTION_ERROR_INVALID_PARAMETER;
@@ -1297,7 +1322,7 @@ static int __reset_statistic(connection_type_e connection_type,
        if (rv != CONNECTION_ERROR_NONE)
                return rv;
 
-       CONNECTION_LOG(CONNECTION_INFO,"connection_reset_statistics success");
+       CONNECTION_LOG(CONNECTION_INFO, "connection_reset_statistics success");
 
        return CONNECTION_ERROR_NONE;
 }
@@ -1308,9 +1333,9 @@ EXPORT_API int connection_get_statistics(connection_h connection,
 {
        CHECK_FEATURE_SUPPORTED(TELEPHONY_FEATURE, WIFI_FEATURE);
 
-       if(connection_type == CONNECTION_TYPE_CELLULAR )
+       if (connection_type == CONNECTION_TYPE_CELLULAR)
                CHECK_FEATURE_SUPPORTED(TELEPHONY_FEATURE);
-       else if(connection_type == CONNECTION_TYPE_WIFI)
+       else if (connection_type == CONNECTION_TYPE_WIFI)
                CHECK_FEATURE_SUPPORTED(WIFI_FEATURE);
 
        if (!(__connection_check_handle_validity(connection)) || size == NULL) {
@@ -1327,9 +1352,9 @@ EXPORT_API int connection_reset_statistics(connection_h connection,
 {
        CHECK_FEATURE_SUPPORTED(TELEPHONY_FEATURE, WIFI_FEATURE);
 
-       if(connection_type == CONNECTION_TYPE_CELLULAR )
+       if (connection_type == CONNECTION_TYPE_CELLULAR)
                CHECK_FEATURE_SUPPORTED(TELEPHONY_FEATURE);
-       else if(connection_type == CONNECTION_TYPE_WIFI)
+       else if (connection_type == CONNECTION_TYPE_WIFI)
                CHECK_FEATURE_SUPPORTED(WIFI_FEATURE);
 
        if (!__connection_check_handle_validity(connection)) {