Fix for UTC error case

[platform/core/connectivity/wifi-mesh-manager.git] / src / mesh-service-interface.c

diff --git a/src/mesh-service-interface.c b/src/mesh-service-interface.c
index aa7e7a9..a37fca0 100644 (file)
--- a/src/mesh-service-interface.c
+++ b/src/mesh-service-interface.c
@@ -155,34 +155,27 @@ static gboolean _meshd_dbus_handle_enable(Manager *object,
 
        if (service->mesh_activated) {
                /* Already activated */
-               manager_complete_enable(object, invocation,
-                               MESHD_ERROR_OPERATION_FAILED);
-       } else {
-               /* Do API response first */
-               manager_complete_enable(object, invocation, ret);
-               service->mesh_activated = TRUE;
+               manager_complete_enable(object, invocation, MESHD_ERROR_NONE);
+               goto FINISH;
        }
 
+       /* Do API response first */
+       manager_complete_enable(object, invocation, ret);
+       service->mesh_activated = TRUE;
+
        meshd_check_null_ret_error("info", info, FALSE);
 
        /* Register event handler first */
        ret = mesh_request_register_event_handler();
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to register mesh event handler !! [%d]", ret);
-       }
 
        ret = mesh_interface_initialize(service->interface_info);
        if (MESHD_ERROR_NONE != ret) {
                MESH_LOGE("Failed to mesh_interface_initialize [%d]", ret);
                goto FINISH;
        }
-#if 0
-       ret = mesh_request_load_saved_mesh_network(&service->saved_mesh_network);
-       if (MESHD_ERROR_NONE != ret) {
-               MESH_LOGE("Failed to mesh_request_load_saved_mesh_network [%d]", ret);
-               goto FINISH;
-       }
-#endif
+
 FINISH:
        net_mesh_emit_mesh_enabled(meshd_dbus_get_object(), ret);
 
@@ -203,9 +196,8 @@ static gboolean _meshd_dbus_handle_disable(Manager *object,
        manager_complete_disable(object, invocation, ret);
 
        ret = mesh_request_unregister_event_handler();
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to unregister mesh event handler !! [%d]", ret);
-       }
 
        /* Terminate daemon */
        meshd_service_exit(service);
@@ -261,9 +253,8 @@ static gboolean _meshd_dbus_handle_cancel_scan(NetMesh *object,
        mesh_service *service = (mesh_service *)user_data;
 
        ret = mesh_request_cancel_scan(service);
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to mesh_request_cancel_scan");
-       }
 
        net_mesh_complete_cancel_scan(object, invocation, ret);
 
@@ -379,9 +370,8 @@ static gboolean _meshd_dbus_handle_enable_mesh(NetMesh *object,
                        and decide to make gate enabled */
        ret = mesh_request_set_mesh_gate(info->bridge_interface,
                        info->mesh_interface, info->external_interface);
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to mesh_request_set_mesh_gate [%d]", ret);
-       }
 #endif
 
        net_mesh_complete_enable_mesh(object, invocation, ret);
@@ -407,9 +397,8 @@ static gboolean _meshd_dbus_handle_disable_mesh(NetMesh *object,
        }
 
        ret = mesh_request_disable_network(service);
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to disable mesh network !");
-       }
 
        /* Make response */
        net_mesh_complete_disable_mesh(object, invocation, ret);
@@ -497,9 +486,8 @@ static gboolean _meshd_dbus_handle_set_gate(NetMesh *object,
        /* Set STP and gate_announce */
        ret = mesh_request_set_mesh_gate(info->bridge_interface,
                        info->mesh_interface, info->external_interface);
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to mesh_request_set_mesh_gate [%d]", ret);
-       }
 
        net_mesh_complete_set_gate(object, invocation, ret);
 
@@ -516,9 +504,9 @@ static gboolean _meshd_dbus_handle_unset_gate(NetMesh *object,
 
        ret = mesh_request_unset_mesh_gate(info->bridge_interface,
                        info->mesh_interface, info->external_interface);
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to mesh_request_unset_mesh_gate [%d]", ret);
-       }
+
        net_mesh_complete_unset_gate(object, invocation, ret);
 
        return TRUE;
@@ -545,9 +533,8 @@ static gboolean _meshd_dbus_handle_set_softap(NetMesh *object,
        ret = mesh_request_set_softap_config(info->softap_interface,
                ssid, mode, channel, visibility, max_sta,
                security, passphrase);
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to mesh_request_set_softap_config [%d]", ret);
-       }
 
        net_mesh_complete_set_softap(object, invocation, ret);
 
@@ -563,9 +550,8 @@ static gboolean _meshd_dbus_handle_enable_softap(NetMesh *object,
 
        /* Check softAP interface and execute it */
        ret = mesh_request_enable_softap(info->bridge_interface, info->softap_interface);
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to mesh_request_enable_softap [%d]", ret);
-       }
 
        net_mesh_complete_enable_softap(object, invocation, ret);
 
@@ -581,9 +567,8 @@ static gboolean _meshd_dbus_handle_disable_softap(NetMesh *object,
 
        /* Destroy softAP */
        ret = mesh_request_disable_softap(info->bridge_interface, info->softap_interface);
-       if (MESHD_ERROR_NONE != ret) {
+       if (MESHD_ERROR_NONE != ret)
                MESH_LOGE("Failed to mesh_request_disable_softap [%d]", ret);
-       }
 
        net_mesh_complete_disable_softap(object, invocation, ret);
 
@@ -719,7 +704,7 @@ static gboolean _meshd_dbus_handle_get_station_info(NetMesh *object,
         * tx retries:  0
         * tx failed:   0
         * beacon loss: 0
-        * signal:      -64 dBm
+        * signal:              -64 dBm
         * signal avg:  -63 dBm
         * tx bitrate:  54.0 MBit/s
         * rx bitrate:  5.5 MBit/s
@@ -856,7 +841,7 @@ static gboolean _meshd_dbus_handle_get_mpath_info(NetMesh *object,
         */
                /* Get mesh path information and make variant data */
                g_variant_builder_init(&builder, G_VARIANT_TYPE("aa{sv}"));
-       
+
                iter = service->mpath_list;
                while (iter != NULL) {
                        mesh_mpath_info_s *item = (mesh_mpath_info_s*)iter->data;