Fix svace 2.2 issues

Change-Id: I2af7c92c499e2747ca10e1cc71f7f42a036ffc1b
Signed-off-by: DoHyun Pyun <dh79.pyun@samsung.com>

diff --git a/packaging/capi-network-bluetooth.spec b/packaging/capi-network-bluetooth.spec
index 502fb68a9dc566ce2f16ddf5b4d760a6d0f9b21c..038b8503427d7fbc296f25aac3706aa176296cb5 100644 (file)
--- a/packaging/capi-network-bluetooth.spec
+++ b/packaging/capi-network-bluetooth.spec
@@ -1,6 +1,6 @@
 Name:       capi-network-bluetooth
 Summary:    Network Bluetooth Framework
-Version:    0.3.2
+Version:    0.3.3
 Release:    1
 Group:      Connectivity/API
 License:    Apache-2.0

diff --git a/src/bluetooth-adapter.c b/src/bluetooth-adapter.c
index c7ddc0bfab7cf013eebe795bbb2ea2e02b028bd7..994d08a17570d904c140911eb3363e110ab9cabc 100644 (file)
--- a/src/bluetooth-adapter.c
+++ b/src/bluetooth-adapter.c
@@ -546,18 +546,11 @@ int bt_adapter_set_visibility(bt_adapter_visibility_mode_e visibility_mode,
 int bt_adapter_set_connectable_changed_cb(bt_adapter_connectable_changed_cb callback,
                                                        void *user_data)
 {
-       int ret = BT_ERROR_NONE;
-
        BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON);
        BT_CHECK_INIT_STATUS();
        BT_CHECK_INPUT_PARAMETER(callback);
        _bt_set_cb(BT_EVENT_CONNECTABLE_CHANGED_EVENT, callback, user_data);
-       if (ret != BT_ERROR_NONE) { /* LCOV_EXCL_LINE */
-               BT_ERR("%s(0x%08x)", _bt_convert_error_to_string(ret),
-                                               ret); /* LCOV_EXCL_LINE */
-       } /* LCOV_EXCL_LINE */
-
-       return ret;
+       return BT_ERROR_NONE;
 }
 
 int bt_adapter_unset_connectable_changed_cb(void)
@@ -805,18 +798,13 @@ int bt_adapter_set_state_changed_cb(bt_adapter_state_changed_cb callback,
                                                        void *user_data)
 {
        BT_DBG("");
-       int ret = BT_ERROR_NONE;
-
        BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON);
        BT_CHECK_INIT_STATUS();
        BT_CHECK_INPUT_PARAMETER(callback);
+
        _bt_set_cb(BT_EVENT_STATE_CHANGED, callback, user_data);
-       if (ret != BT_ERROR_NONE) { /* LCOV_EXCL_LINE */
-               BT_ERR("%s(0x%08x)", _bt_convert_error_to_string(ret),
-                                       ret); /* LCOV_EXCL_LINE */
-       } /* LCOV_EXCL_LINE */
 
-       return ret;
+       return BT_ERROR_NONE;
 }
 
 /* LCOV_EXCL_START */
@@ -836,53 +824,37 @@ int bt_adapter_le_set_state_changed_cb(bt_adapter_le_state_changed_cb callback,
 int bt_adapter_set_name_changed_cb(bt_adapter_name_changed_cb callback,
                                                        void *user_data)
 {
-       int ret = BT_ERROR_NONE;
-
        BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON);
        BT_CHECK_INIT_STATUS();
        BT_CHECK_INPUT_PARAMETER(callback);
+
        _bt_set_cb(BT_EVENT_NAME_CHANGED, callback, user_data);
-       if (ret != BT_ERROR_NONE) { /* LCOV_EXCL_LINE */
-               BT_ERR("%s(0x%08x)", _bt_convert_error_to_string(ret),
-                                               ret); /* LCOV_EXCL_LINE */
-       } /* LCOV_EXCL_LINE */
 
-       return ret;
+       return BT_ERROR_NONE;
 }
 
 int bt_adapter_set_visibility_mode_changed_cb(bt_adapter_visibility_mode_changed_cb callback,
                                                        void *user_data)
 {
-       int ret = BT_ERROR_NONE;
-
        BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON);
        BT_CHECK_INIT_STATUS();
        BT_CHECK_INPUT_PARAMETER(callback);
+
        _bt_set_cb(BT_EVENT_VISIBILITY_MODE_CHANGED, callback, user_data);
-       if (ret != BT_ERROR_NONE) { /* LCOV_EXCL_LINE */
-               BT_ERR("%s(0x%08x)", _bt_convert_error_to_string(ret),
-                                               ret); /* LCOV_EXCL_LINE */
-       } /* LCOV_EXCL_LINE */
 
-       return ret;
+       return BT_ERROR_NONE;
 }
 
 int bt_adapter_set_device_discovery_state_changed_cb(bt_adapter_device_discovery_state_changed_cb callback,
                                                        void *user_data)
 {
-       int ret = BT_ERROR_NONE;
-
        BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON);
        BT_CHECK_INIT_STATUS();
        BT_CHECK_INPUT_PARAMETER(callback);
+
        _bt_set_cb(BT_EVENT_DEVICE_DISCOVERY_STATE_CHANGED,
-                                       callback, user_data);
-       if (ret != BT_ERROR_NONE) { /* LCOV_EXCL_LINE */
-               BT_ERR("%s(0x%08x)", _bt_convert_error_to_string(ret),
-                                               ret); /* LCOV_EXCL_LINE */
-       } /* LCOV_EXCL_LINE */
 
-       return ret;
+       return BT_ERROR_NONE;
 }
 
 #if !defined(TIZEN_PROFILE_WEARABLE) && !defined(TIZEN_PROFILE_IVI)
@@ -940,18 +912,13 @@ int bt_adapter_unset_visibility_mode_changed_cb(void)
 int bt_adapter_set_visibility_duration_changed_cb(bt_adapter_visibility_duration_changed_cb callback,
                                                        void *user_data)
 {
-       int ret = BT_ERROR_NONE;
-
        BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON);
        BT_CHECK_INIT_STATUS();
        BT_CHECK_INPUT_PARAMETER(callback);
+
        _bt_set_cb(BT_EVENT_VISIBILITY_DURATION_CHANGED, callback, user_data);
-       if (ret != BT_ERROR_NONE) { /* LCOV_EXCL_LINE */
-               BT_ERR("%s(0x%08x)", _bt_convert_error_to_string(ret),
-                                               ret); /* LCOV_EXCL_LINE */
-       } /* LCOV_EXCL_LINE */
 
-       return ret;
+       return BT_ERROR_NONE;
 }
 
 int bt_adapter_unset_visibility_duration_changed_cb(void)
@@ -1404,19 +1371,14 @@ int bt_adapter_set_manufacturer_data_changed_cb(
                bt_adapter_manufacturer_data_changed_cb callback,
                void *user_data)
 {
-       int ret = BT_ERROR_NONE;
-
        BT_CHECK_SUPPORTED_FEATURE(BT_FEATURE_COMMON);
        BT_CHECK_INIT_STATUS();
        BT_CHECK_INPUT_PARAMETER(callback);
+
        _bt_set_cb(BT_EVENT_MANUFACTURER_DATA_CHANGED,
                        callback, user_data);
-       if (ret != BT_ERROR_NONE) { /* LCOV_EXCL_LINE */
-               BT_ERR("%s(0x%08x)", _bt_convert_error_to_string(ret),
-                                               ret); /* LCOV_EXCL_LINE */
-       } /* LCOV_EXCL_LINE */
 
-       return ret;
+       return BT_ERROR_NONE;
 }
 
 int bt_adapter_unset_manufacturer_data_changed_cb(void)

diff --git a/src/bluetooth-common.c b/src/bluetooth-common.c
index e26223d93bcd3cb76b6ae86ec7d1b145fab0ccac..83457ccaae475944fb164f5ee9e5f1004ce20246 100644 (file)
--- a/src/bluetooth-common.c
+++ b/src/bluetooth-common.c
@@ -743,6 +743,9 @@ static bt_gatt_server_read_value_requested_cb __bt_gatt_attribute_get_read_cb(
                for (l2 = serv->services; l2 != NULL; l2 = l2->next) {
                        bt_gatt_service_s *svc = l2->data;
 
+                       if (!svc)
+                               return NULL;
+
                        if (g_strcmp0(svc->path, svc_path) == 0) {
                                for (l3 = svc->characteristics; l3 != NULL; l3 = l3->next) {
                                        bt_gatt_characteristic_s *chr = l3->data;
@@ -796,6 +799,9 @@ static bt_gatt_server_write_value_requested_cb __bt_gatt_attribute_get_value_cha
                for (l2 = serv->services; l2 != NULL; l2 = l2->next) {
                        bt_gatt_service_s *svc = l2->data;
 
+                       if (!svc)
+                               return NULL;
+
                        if (g_strcmp0(svc->path, svc_path) == 0) {
                                for (l3 = svc->characteristics; l3 != NULL; l3 = l3->next) {
                                        bt_gatt_characteristic_s *chr = l3->data;

diff --git a/test/bt_unit_test.c b/test/bt_unit_test.c
index b11dff90cdd08070608b71d277a87bdc4484c60e..64296aa20b733eb014b86e9b8da0d28227cde09f 100644 (file)
--- a/test/bt_unit_test.c
+++ b/test/bt_unit_test.c
@@ -2180,9 +2180,8 @@ void __bt_gatt_client_print_value(bt_gatt_h h)
        }
 
        TC_PRT("Value : %s", str);
-
-       g_free(str);
 fail:
+       g_free(str);
        g_free(value);
 }
 
@@ -5332,6 +5331,11 @@ int test_input_callback(void *data)
                                __bt_free_test_param(&g_test_param);
                        }
 
+                       if (slot_id < 0 || slot_id >= 3) {
+                               TC_PRT("Invalid slot id: %d", slot_id);
+                               break;
+                       }
+
                        advertiser = advertiser_list[slot_id];
 
                        TC_PRT("advertiser: %p", advertiser);
@@ -8375,7 +8379,7 @@ int test_input_callback(void *data)
                                        TC_PRT("IPSP : Current role is IPSP Sever !");
 
                                        /* Sockets Layer Call: send() */
-                                       send(ipsp_server_sock, data, strlen(data) + 1, 0);
+                                       n = send(ipsp_server_sock, data, strlen(data) + 1, 0);
                                        if (n < 0)
                                                TC_PRT("\nIPSP Error : While sending data !");
                                } else {
@@ -8384,7 +8388,7 @@ int test_input_callback(void *data)
                        } else if (role == 1) {
                                if (ipsp_client_sock) {
                                        /* Sockets Layer Call: send() */
-                                       send(ipsp_client_sock, data, strlen(data) + 1, 0);
+                                       n = send(ipsp_client_sock, data, strlen(data) + 1, 0);
                                        if (n < 0)
                                                TC_PRT("\nIPSP Error : While sending data !");
                                } else {