revise logging and coding style
[platform/core/connectivity/nfc-manager-neard.git] / daemon / net_nfc_server_snep.c
index d476801..663382e 100644 (file)
 #include "net_nfc_util_gdbus_internal.h"
 #include "net_nfc_server_context.h"
 
-/* declaration */
-static gboolean _handle_start_server(
-               NetNfcGDbusSnep *object,
-               GDBusMethodInvocation *invocation,
-               guint arg_handle,
-               guint arg_sap,
-               const gchar *arg_san,
-               guint arg_user_data,
-               GVariant *arg_privilege);
-
-static gboolean _handle_start_client(
-               NetNfcGDbusSnep *object,
-               GDBusMethodInvocation *invocation,
-               guint arg_handle,
-               guint arg_sap,
-               const gchar *arg_san,
-               guint arg_user_data,
-               GVariant *arg_privilege);
-
-static gboolean _handle_client_send_request(
-               NetNfcGDbusSnep *object,
-               GDBusMethodInvocation *invocation,
-               guint arg_snep_handle,
-               guint arg_type,
-               GVariant *arg_ndef_msg,
-               GVariant *arg_privilege);
-
-static gboolean _handle_stop_snep(
-               NetNfcGDbusSnep *object,
-               GDBusMethodInvocation *invocation,
-               guint arg_handle,
-               guint arg_snep_handle,
-               GVariant *arg_privilege);
-
-static void snep_server_start_thread_func(gpointer user_data);
-
-static void snep_client_start_thread_func(gpointer user_data);
-
-static void snep_client_send_request_thread_func(gpointer user_data);
-
-static void snep_stop_service_thread_func(gpointer user_data);
-
 /* definition */
 static NetNfcGDbusSnep *snep_skeleton =  NULL;
 
 static void _emit_snep_event_signal(GVariant *parameter,
-               net_nfc_snep_handle_h handle,
-               net_nfc_error_e result,
-               uint32_t type,
-               data_s *data)
+               net_nfc_snep_handle_h handle, net_nfc_error_e result, uint32_t type, data_s *data)
 {
-       GDBusConnection *connection;
+       gboolean ret;
        char *client_id;
        void *user_data;
        GVariant *arg_data;
        GError *error = NULL;
+       GDBusConnection *connection;
 
-       g_variant_get(parameter, "(usu)",
-                       (guint *)&connection,
-                       &client_id,
+       g_variant_get(parameter, "(usu)", (guint *)&connection, &client_id,
                        (guint *)&user_data);
 
        arg_data = net_nfc_util_gdbus_data_to_variant(data);
 
-       if (g_dbus_connection_emit_signal(
+       ret = g_dbus_connection_emit_signal(
                                connection,
                                client_id,
                                "/org/tizen/NetNfcService/Snep",
@@ -99,27 +53,23 @@ static void _emit_snep_event_signal(GVariant *parameter,
                                "SnepEvent",
                                g_variant_new("(uui@a(y)u)", GPOINTER_TO_UINT(handle), type, (gint)result,
                                        arg_data, GPOINTER_TO_UINT(user_data)),
-                               &error) == false)
+                               &error);
+       if (false == ret)
        {
-               if (error != NULL && error->message != NULL) {
+               if (error != NULL && error->message != NULL)
                        NFC_ERR("g_dbus_connection_emit_signal failed : %s", error->message);
-               } else {
+               else
                        NFC_ERR("g_dbus_connection_emit_signal failed");
-               }
        }
 
        g_free(client_id);
 }
 
 static net_nfc_error_e _snep_server_cb(net_nfc_snep_handle_h handle,
-               net_nfc_error_e result,
-               uint32_t type,
-               data_s *data,
-               void *user_param)
+               net_nfc_error_e result, uint32_t type, data_s *data, void *user_param)
 {
-       GVariant *parameter = (GVariant *)user_param;
-
        data_s *temp = data;
+       GVariant *parameter = (GVariant *)user_param;
 
        NFC_DBG("type [%d], result [%d], data [%p], user_param [%p]",
                        type, result, data, user_param);
@@ -130,8 +80,7 @@ static net_nfc_error_e _snep_server_cb(net_nfc_snep_handle_h handle,
                {
                        uint32_t max_len = 0;
 
-                       net_nfc_server_snep_parse_get_request(data, &max_len,
-                                       temp);
+                       net_nfc_server_snep_parse_get_request(data, &max_len, temp);
                }
                break;
 
@@ -143,39 +92,30 @@ static net_nfc_error_e _snep_server_cb(net_nfc_snep_handle_h handle,
                break;
        }
 
-       if (result < NET_NFC_OK) {
+       if (result < NET_NFC_OK)
                type = NET_NFC_LLCP_STOP;
-       }
 
-       _emit_snep_event_signal(parameter, handle,
-                       result, type, data);
+       _emit_snep_event_signal(parameter, handle, result, type, data);
 
-       if (type == NET_NFC_LLCP_STOP) {
+       if (NET_NFC_LLCP_STOP == type)
                g_object_unref(parameter);
-       }
 
        return result;
 }
 
 static void snep_server_start_thread_func(gpointer user_data)
 {
-       NetNfcGDbusSnep *object;
-       GDBusMethodInvocation *invocation;
-       net_nfc_target_handle_s *arg_handle;
        guint arg_sap;
        gchar *arg_san;
+       GVariant *parameter;
        void *arg_user_data;
        net_nfc_error_e result;
-
-       GVariant *parameter;
+       NetNfcGDbusSnep *object;
        GDBusConnection *connection;
+       GDBusMethodInvocation *invocation;
+       net_nfc_target_handle_s *arg_handle;
 
-       if (user_data == NULL)
-       {
-               NFC_ERR("cannot get SNEP client data");
-
-               return;
-       }
+       RET_IF(NULL == user_data);
 
        g_variant_get((GVariant *)user_data,
                        "(uuuusu)",
@@ -195,20 +135,21 @@ static void snep_server_start_thread_func(gpointer user_data)
                        GPOINTER_TO_UINT(g_object_ref(connection)),
                        g_dbus_method_invocation_get_sender(invocation),
                        GPOINTER_TO_UINT(arg_user_data));
-       if (parameter != NULL) {
-               result = net_nfc_server_snep_server(arg_handle,
-                               arg_san,
-                               arg_sap,
-                               _snep_server_cb,
-                               parameter);
-               if (result != NET_NFC_OK) {
-                       NFC_ERR("net_nfc_server_snep_server failed, [%d]",
-                                       result);
+
+       if (parameter != NULL)
+       {
+               result = net_nfc_server_snep_server(arg_handle, arg_san, arg_sap,
+                               _snep_server_cb, parameter);
+               if (result != NET_NFC_OK)
+               {
+                       NFC_ERR("net_nfc_server_snep_server failed, [%d]", result);
                        g_object_unref(connection);
 
                        g_variant_unref(parameter);
                }
-       } else {
+       }
+       else
+       {
                NFC_ERR("g_variant_new failed");
 
                g_object_unref(connection);
@@ -232,17 +173,17 @@ static gboolean _handle_start_server(
                guint arg_user_data,
                GVariant *arg_privilege)
 {
-       GVariant *parameter;
+       bool ret;
        gboolean result;
+       GVariant *parameter;
 
-       NFC_INFO(">>> REQUEST from [%s]",
-                       g_dbus_method_invocation_get_sender(invocation));
+       NFC_INFO(">>> REQUEST from [%s]", g_dbus_method_invocation_get_sender(invocation));
 
        /* check privilege and update client context */
-       if (net_nfc_server_gdbus_check_privilege(invocation,
-                               arg_privilege,
-                               "nfc-manager::p2p",
-                               "rw") == false) {
+       ret = net_nfc_server_gdbus_check_privilege(invocation, arg_privilege,
+                               "nfc-manager::p2p", "rw");
+       if (false == ret)
+       {
                NFC_ERR("permission denied, and finished request");
 
                return FALSE;
@@ -258,8 +199,9 @@ static gboolean _handle_start_server(
 
        if (parameter != NULL)
        {
-               if ((result = net_nfc_server_controller_async_queue_push(
-                                               snep_server_start_thread_func, parameter)) == FALSE)
+               result = net_nfc_server_controller_async_queue_push(
+                                               snep_server_start_thread_func, parameter);
+               if (FALSE == result)
                {
                        NFC_ERR("net_nfc_server_controller_async_queue_push failed");
 
@@ -278,8 +220,7 @@ static gboolean _handle_start_server(
                NFC_ERR("g_variant_new failed");
 
                g_dbus_method_invocation_return_dbus_error(invocation,
-                               "org.tizen.NetNfcService.Snep.MemoryError",
-                               "Out of memory");
+                               "org.tizen.NetNfcService.Snep.MemoryError", "Out of memory");
 
                result = FALSE;
        }
@@ -301,32 +242,25 @@ static net_nfc_error_e _snep_start_client_cb(
 
        _emit_snep_event_signal(parameter, handle, result, type, data);
 
-       if (type == NET_NFC_LLCP_STOP) {
+       if (NET_NFC_LLCP_STOP == type)
                g_variant_unref(parameter);
-       }
 
        return result;
 }
 
 static void snep_client_start_thread_func(gpointer user_data)
 {
-       NetNfcGDbusSnep *object;
-       GDBusMethodInvocation *invocation;
-       net_nfc_target_handle_s *arg_handle;
        guint arg_sap;
        gchar *arg_san;
+       GVariant *parameter;
        void *arg_user_data;
        net_nfc_error_e result;
-
-       GVariant *parameter;
+       NetNfcGDbusSnep *object;
        GDBusConnection *connection;
+       GDBusMethodInvocation *invocation;
+       net_nfc_target_handle_s *arg_handle;
 
-       if (user_data == NULL)
-       {
-               NFC_ERR("cannot get SNEP client data");
-
-               return;
-       }
+       RET_IF(NULL == user_data);
 
        g_variant_get((GVariant *)user_data,
                        "(uuuusu)",
@@ -346,20 +280,21 @@ static void snep_client_start_thread_func(gpointer user_data)
                        GPOINTER_TO_UINT(g_object_ref(connection)),
                        g_dbus_method_invocation_get_sender(invocation),
                        GPOINTER_TO_UINT(arg_user_data));
-       if (parameter != NULL) {
-               result = net_nfc_server_snep_client(arg_handle,
-                               arg_san,
-                               arg_sap,
-                               _snep_start_client_cb,
-                               parameter);
-               if (result != NET_NFC_OK) {
-                       NFC_ERR("net_nfc_server_snep_client failed, [%d]",
-                                       result);
+
+       if (parameter != NULL)
+       {
+               result = net_nfc_server_snep_client(arg_handle, arg_san, arg_sap,
+                               _snep_start_client_cb, parameter);
+               if (result != NET_NFC_OK)
+               {
+                       NFC_ERR("net_nfc_server_snep_client failed, [%d]", result);
                        g_object_unref(connection);
 
                        g_variant_unref(parameter);
                }
-       } else {
+       }
+       else
+       {
                NFC_ERR("g_variant_new failed");
 
                g_object_unref(connection);
@@ -383,17 +318,17 @@ static gboolean _handle_start_client(
                guint arg_user_data,
                GVariant *arg_privilege)
 {
-       GVariant *parameter;
+       bool ret;
        gboolean result;
+       GVariant *parameter;
 
-       NFC_INFO(">>> REQUEST from [%s]",
-                       g_dbus_method_invocation_get_sender(invocation));
+       NFC_INFO(">>> REQUEST from [%s]", g_dbus_method_invocation_get_sender(invocation));
 
        /* check privilege and update client context */
-       if (net_nfc_server_gdbus_check_privilege(invocation,
-                               arg_privilege,
-                               "nfc-manager::p2p",
-                               "rw") == false) {
+       ret = net_nfc_server_gdbus_check_privilege(invocation, arg_privilege,
+                               "nfc-manager::p2p", "rw");
+       if (false == ret)
+       {
                NFC_ERR("permission denied, and finished request");
 
                return FALSE;
@@ -409,8 +344,9 @@ static gboolean _handle_start_client(
 
        if (parameter != NULL)
        {
-               if ((result = net_nfc_server_controller_async_queue_push(
-                                               snep_client_start_thread_func, parameter)) == FALSE)
+               result = net_nfc_server_controller_async_queue_push(
+                                               snep_client_start_thread_func, parameter);
+               if (FALSE == result)
                {
                        NFC_ERR("net_nfc_server_controller_async_queue_push failed");
 
@@ -429,8 +365,7 @@ static gboolean _handle_start_client(
                NFC_ERR("g_variant_new failed");
 
                g_dbus_method_invocation_return_dbus_error(invocation,
-                               "org.tizen.NetNfcService.Snep.MemoryError",
-                               "Out of memory");
+                               "org.tizen.NetNfcService.Snep.MemoryError", "Out of memory");
 
                result = FALSE;
        }
@@ -450,13 +385,14 @@ static net_nfc_error_e _snep_client_request_cb(
        NFC_DBG("type [%d], result [%d], data [%p], user_param [%p]",
                        type, result, data, user_param);
 
-       if (parameter != NULL) {
+       if (parameter != NULL)
+       {
+               GVariant *arg_ndef_msg;
                NetNfcGDbusSnep *object;
+               GVariant *arg_data = NULL;
+               net_nfc_snep_type_t arg_type;
                GDBusMethodInvocation *invocation;
                net_nfc_snep_handle_h arg_snep_handle;
-               net_nfc_snep_type_t arg_type;
-               GVariant *arg_ndef_msg;
-               GVariant *arg_data = NULL;
 
                g_variant_get(parameter,
                                "(uuuu@a(y))",
@@ -466,17 +402,13 @@ static net_nfc_error_e _snep_client_request_cb(
                                (guint *)&arg_type,
                                &arg_ndef_msg);
 
-               if (data != NULL && data->buffer != NULL && data->length > 0) {
+               if (data != NULL && data->buffer != NULL && data->length > 0)
                        arg_data = net_nfc_util_gdbus_data_to_variant(data);
-               } else {
+               else
                        arg_data = net_nfc_util_gdbus_buffer_to_variant(NULL, 0);
-               }
 
-               net_nfc_gdbus_snep_complete_client_request(object,
-                               invocation,
-                               result,
-                               type,
-                               arg_data);
+               net_nfc_gdbus_snep_complete_client_request(object, invocation, result,
+                               type, arg_data);
 
                g_variant_unref(arg_ndef_msg);
 
@@ -486,7 +418,9 @@ static net_nfc_error_e _snep_client_request_cb(
                g_variant_unref(parameter);
 
                result = NET_NFC_OK;
-       } else {
+       }
+       else
+       {
                result = NET_NFC_NULL_PARAMETER;
        }
 
@@ -495,20 +429,15 @@ static net_nfc_error_e _snep_client_request_cb(
 
 static void snep_client_send_request_thread_func(gpointer user_data)
 {
-       NetNfcGDbusSnep *object;
-       GDBusMethodInvocation *invocation;
-       net_nfc_snep_handle_h arg_snep_handle;
-       net_nfc_snep_type_t arg_type;
+       net_nfc_error_e result;
        GVariant *arg_ndef_msg;
+       NetNfcGDbusSnep *object;
        data_s data = { NULL, };
-       net_nfc_error_e result;
-
-       if (user_data == NULL)
-       {
-               NFC_ERR("cannot get SNEP client data");
+       net_nfc_snep_type_t arg_type;
+       GDBusMethodInvocation *invocation;
+       net_nfc_server_snep_context_t *arg_snep_handle;
 
-               return;
-       }
+       RET_IF(NULL == user_data);
 
        g_variant_get((GVariant *)user_data,
                        "(uuuu@a(y))",
@@ -523,17 +452,13 @@ static void snep_client_send_request_thread_func(gpointer user_data)
 
        net_nfc_util_gdbus_variant_to_data_s(arg_ndef_msg, &data);
 
-       result = net_nfc_server_snep_client_request(arg_snep_handle,
-                       arg_type,
-                       &data,
-                       _snep_client_request_cb,
-                       user_data);
+       result = net_nfc_server_snep_client_request(arg_snep_handle, arg_type,
+                       &data, _snep_client_request_cb, user_data);
        if (result != NET_NFC_OK)
        {
                GVariant *resp;
 
-               NFC_ERR("net_nfc_server_snep_client_request  "
-                               "failed, [%d]",result);
+               NFC_ERR("net_nfc_server_snep_client_request failed, [%d]",result);
 
                resp = net_nfc_util_gdbus_buffer_to_variant(NULL, 0);
 
@@ -559,17 +484,17 @@ static gboolean _handle_client_send_request(
                GVariant *arg_ndef_msg,
                GVariant *arg_privilege)
 {
-       GVariant *parameter;
+       bool ret;
        gboolean result;
+       GVariant *parameter;
 
-       NFC_INFO(">>> REQUEST from [%s]",
-                       g_dbus_method_invocation_get_sender(invocation));
+       NFC_INFO(">>> REQUEST from [%s]", g_dbus_method_invocation_get_sender(invocation));
 
        /* check privilege and update client context */
-       if (net_nfc_server_gdbus_check_privilege(invocation,
-                               arg_privilege,
-                               "nfc-manager::p2p",
-                               "rw") == false) {
+       ret = net_nfc_server_gdbus_check_privilege(invocation, arg_privilege,
+                               "nfc-manager::p2p", "rw");
+       if (false == ret)
+       {
                NFC_ERR("permission denied, and finished request");
 
                return FALSE;
@@ -584,8 +509,9 @@ static gboolean _handle_client_send_request(
 
        if (parameter != NULL)
        {
-               if ((result = net_nfc_server_controller_async_queue_push(
-                                               snep_client_send_request_thread_func, parameter)) == FALSE)
+               result = net_nfc_server_controller_async_queue_push(
+                                               snep_client_send_request_thread_func, parameter);
+               if (FALSE == result)
                {
                        NFC_ERR("net_nfc_server_controller_async_queue_push failed");
 
@@ -604,8 +530,7 @@ static gboolean _handle_client_send_request(
                NFC_ERR("g_variant_new failed");
 
                g_dbus_method_invocation_return_dbus_error(invocation,
-                               "org.tizen.NetNfcService.Snep.MemoryError",
-                               "Out of memory");
+                               "org.tizen.NetNfcService.Snep.MemoryError", "Out of memory");
 
                result = FALSE;
        }
@@ -616,37 +541,24 @@ static gboolean _handle_client_send_request(
 static void snep_stop_service_thread_func(gpointer user_data)
 {
        NetNfcGDbusSnep *object;
-       GDBusMethodInvocation *invocation;
        net_nfc_target_handle_s *handle;
+       GDBusMethodInvocation *invocation;
        net_nfc_snep_handle_h snep_handle;
 
-       if (user_data == NULL)
-       {
-               NFC_ERR("cannot get SNEP client data");
+       RET_IF(NULL == user_data);
 
-               return;
-       }
-
-       g_variant_get((GVariant *)user_data,
-                       "(uuuu)",
-                       (guint *)&object,
-                       (guint *)&invocation,
-                       (guint *)&handle,
-                       (guint *)&snep_handle);
+       g_variant_get((GVariant *)user_data, "(uuuu)", (guint *)&object, (guint *)&invocation,
+                       (guint *)&handle, (guint *)&snep_handle);
 
        g_assert(object != NULL);
        g_assert(invocation != NULL);
 
        /* TODO :
-               g_dbus_method_invocation_return_dbus_error(
-               invocation,
-               "org.tizen.NetNfcService.Snep.DataError",
-               "Cannot stop SNEP service");
-               */
+          g_dbus_method_invocation_return_dbus_error(invocation,
+          "org.tizen.NetNfcService.Snep.DataError", "Cannot stop SNEP service");
+       */
 
-       net_nfc_gdbus_snep_complete_stop_snep(object,
-                       invocation,
-                       NET_NFC_OK);
+       net_nfc_gdbus_snep_complete_stop_snep(object, invocation, NET_NFC_OK);
 
        g_object_unref(invocation);
        g_object_unref(object);
@@ -661,32 +573,30 @@ static gboolean _handle_stop_snep(
                guint arg_snep_handle,
                GVariant *arg_privilege)
 {
-       GVariant *parameter;
+       bool ret;
        gboolean result;
+       GVariant *parameter;
 
-       NFC_INFO(">>> REQUEST from [%s]",
-                       g_dbus_method_invocation_get_sender(invocation));
+       NFC_INFO(">>> REQUEST from [%s]", g_dbus_method_invocation_get_sender(invocation));
 
        /* check privilege and update client context */
-       if (net_nfc_server_gdbus_check_privilege(invocation,
-                               arg_privilege,
-                               "nfc-manager::p2p",
-                               "rw") == false) {
+       ret = net_nfc_server_gdbus_check_privilege(invocation, arg_privilege,
+                               "nfc-manager::p2p", "rw");
+       if (false == ret)
+       {
                NFC_ERR("permission denied, and finished request");
 
                return FALSE;
        }
 
-       parameter = g_variant_new("(uuuu)",
-                       GPOINTER_TO_UINT(g_object_ref(object)),
-                       GPOINTER_TO_UINT(g_object_ref(invocation)),
-                       arg_handle,
-                       arg_snep_handle);
+       parameter = g_variant_new("(uuuu)", GPOINTER_TO_UINT(g_object_ref(object)),
+                       GPOINTER_TO_UINT(g_object_ref(invocation)), arg_handle, arg_snep_handle);
 
        if (parameter != NULL)
        {
-               if ((result = net_nfc_server_controller_async_queue_push(
-                                               snep_stop_service_thread_func, parameter)) == FALSE)
+               result = net_nfc_server_controller_async_queue_push(
+                                               snep_stop_service_thread_func, parameter);
+               if (FALSE == result)
                {
                        NFC_ERR("net_nfc_server_controller_async_queue_push failed");
 
@@ -705,8 +615,7 @@ static gboolean _handle_stop_snep(
                NFC_ERR("g_variant_new failed");
 
                g_dbus_method_invocation_return_dbus_error(invocation,
-                               "org.tizen.NetNfcService.Snep.MemoryError",
-                               "Out of memory");
+                               "org.tizen.NetNfcService.Snep.MemoryError", "Out of memory");
 
                result = FALSE;
        }
@@ -717,45 +626,43 @@ static gboolean _handle_stop_snep(
 static void _snep_activate_cb(int event, net_nfc_target_handle_s *handle,
                uint32_t sap, const char *san, void *user_param)
 {
-       GVariant *parameter = (GVariant *)user_param;
        net_nfc_error_e result = NET_NFC_OK;
+       GVariant *parameter = (GVariant *)user_param;
 
-       NFC_DBG("event [%d], handle [%p], sap [%d], san [%s]",
-                       event, handle, sap, san);
+       NFC_DBG("event [%d], handle [%p], sap [%d], san [%s]", event, handle, sap, san);
 
-       if (event == NET_NFC_LLCP_START) {
-               GDBusConnection *connection;
-               GVariant *param = NULL;
+       if (NET_NFC_LLCP_START == event)
+       {
                char *client_id;
                void *user_data;
+               GVariant *param = NULL;
+               GDBusConnection *connection;
 
                /* start server */
-               g_variant_get(parameter, "(usu)",
-                               (guint *)&connection,
-                               &client_id,
+               g_variant_get(parameter, "(usu)", (guint *)&connection, &client_id,
                                (guint *)&user_data);
 
-               param = g_variant_new("(usu)",
-                               GPOINTER_TO_UINT(g_object_ref(connection)),
-                               client_id,
-                               GPOINTER_TO_UINT(user_data));
+               param = g_variant_new("(usu)", GPOINTER_TO_UINT(g_object_ref(connection)),
+                               client_id, GPOINTER_TO_UINT(user_data));
 
                g_free(client_id);
 
                result = net_nfc_server_snep_server(handle, (char *)san, sap,
                                _snep_server_cb, param);
-               if (result == NET_NFC_OK) {
-                       _emit_snep_event_signal(parameter, handle,
-                                       result, event, NULL);
-               } else {
-                       NFC_ERR("net_nfc_server_snep_server failed, [%d]",
-                                       result);
+               if (NET_NFC_OK == result)
+               {
+                       _emit_snep_event_signal(parameter, handle, result, event, NULL);
+               }
+               else
+               {
+                       NFC_ERR("net_nfc_server_snep_server failed, [%d]", result);
 
                        g_variant_unref(param);
                }
-       } else {
-               _emit_snep_event_signal(parameter, handle,
-                               result, NET_NFC_LLCP_UNREGISTERED, NULL);
+       }
+       else
+       {
+               _emit_snep_event_signal(parameter, handle, result, NET_NFC_LLCP_UNREGISTERED, NULL);
 
                /* unregister server */
                g_variant_unref(parameter);
@@ -764,36 +671,28 @@ static void _snep_activate_cb(int event, net_nfc_target_handle_s *handle,
 
 static void snep_register_server_thread_func(gpointer user_data)
 {
-       NetNfcGDbusSnep *object;
-       GDBusMethodInvocation *invocation;
        guint arg_sap;
        gchar *arg_san;
        guint arg_user_data;
-
        net_nfc_error_e result;
+       NetNfcGDbusSnep *object;
+       g_assert(user_data != NULL);
        GVariant *parameter = NULL;
        GDBusConnection *connection;
+       GDBusMethodInvocation *invocation;
 
-       g_assert(user_data != NULL);
-
-       g_variant_get((GVariant *)user_data,
-                       "(uuusu)",
-                       (guint *)&object,
-                       (guint *)&invocation,
-                       &arg_sap,
-                       &arg_san,
-                       &arg_user_data);
+       g_variant_get((GVariant *)user_data, "(uuusu)", (guint *)&object, (guint *)&invocation,
+                       &arg_sap, &arg_san, &arg_user_data);
 
        g_assert(object != NULL);
        g_assert(invocation != NULL);
 
        connection = g_dbus_method_invocation_get_connection(invocation);
 
-       parameter = g_variant_new("(usu)",
-                       GPOINTER_TO_UINT(g_object_ref(connection)),
-                       g_dbus_method_invocation_get_sender(invocation),
-                       arg_user_data);
-       if (parameter != NULL) {
+       parameter = g_variant_new("(usu)", GPOINTER_TO_UINT(g_object_ref(connection)),
+                       g_dbus_method_invocation_get_sender(invocation), arg_user_data);
+       if (parameter != NULL)
+       {
                /* register default snep server */
                result = net_nfc_server_llcp_register_service(
                                g_dbus_method_invocation_get_sender(invocation),
@@ -801,19 +700,20 @@ static void snep_register_server_thread_func(gpointer user_data)
                                arg_san,
                                _snep_activate_cb,
                                parameter);
-               if (result != NET_NFC_OK) {
+               if (result != NET_NFC_OK)
+               {
                        NFC_ERR("net_nfc_service_llcp_register_service failed, [%d]", result);
                        g_object_unref(connection);
                        g_variant_unref(parameter);
                }
-       } else {
+       }
+       else
+       {
                result = NET_NFC_ALLOC_FAIL;
                g_object_unref(connection);
        }
 
-       net_nfc_gdbus_snep_complete_server_register(object,
-                       invocation,
-                       result);
+       net_nfc_gdbus_snep_complete_server_register(object, invocation, result);
 
        g_free(arg_san);
 
@@ -831,33 +731,30 @@ static gboolean _handle_register_server(
                guint arg_user_data,
                GVariant *arg_privilege)
 {
-       GVariant *parameter;
+       bool ret;
        gboolean result;
+       GVariant *parameter;
 
-       NFC_INFO(">>> REQUEST from [%s]",
-                       g_dbus_method_invocation_get_sender(invocation));
+       NFC_INFO(">>> REQUEST from [%s]", g_dbus_method_invocation_get_sender(invocation));
 
        /* check privilege and update client context */
-       if (net_nfc_server_gdbus_check_privilege(invocation,
-                               arg_privilege,
-                               "nfc-manager::p2p",
-                               "rw") == false) {
+       ret = net_nfc_server_gdbus_check_privilege(invocation, arg_privilege,
+                               "nfc-manager::p2p", "rw");
+       if (false == ret)
+       {
                NFC_ERR("permission denied, and finished request");
 
                return FALSE;
        }
 
-       parameter = g_variant_new("(uuusu)",
-                       GPOINTER_TO_UINT(g_object_ref(object)),
-                       GPOINTER_TO_UINT(g_object_ref(invocation)),
-                       arg_sap,
-                       arg_san,
-                       arg_user_data);
+       parameter = g_variant_new("(uuusu)", GPOINTER_TO_UINT(g_object_ref(object)),
+                       GPOINTER_TO_UINT(g_object_ref(invocation)), arg_sap, arg_san, arg_user_data);
 
        if (parameter != NULL)
        {
-               if ((result = net_nfc_server_controller_async_queue_push(
-                                               snep_register_server_thread_func, parameter)) == FALSE)
+               result = net_nfc_server_controller_async_queue_push(
+                                               snep_register_server_thread_func, parameter);
+               if (FALSE == result)
                {
                        NFC_ERR("net_nfc_server_controller_async_queue_push failed");
 
@@ -876,8 +773,7 @@ static gboolean _handle_register_server(
                NFC_ERR("g_variant_new failed");
 
                g_dbus_method_invocation_return_dbus_error(invocation,
-                               "org.tizen.NetNfcService.Snep.MemoryError",
-                               "Out of memory");
+                               "org.tizen.NetNfcService.Snep.MemoryError", "Out of memory");
 
                result = FALSE;
        }
@@ -887,33 +783,24 @@ static gboolean _handle_register_server(
 
 static void snep_unregister_server_thread_func(gpointer user_data)
 {
-       NetNfcGDbusSnep *object;
-       GDBusMethodInvocation *invocation;
        guint arg_sap;
        gchar *arg_san;
-
        net_nfc_error_e result;
+       NetNfcGDbusSnep *object;
+       GDBusMethodInvocation *invocation;
 
        g_assert(user_data != NULL);
 
-       g_variant_get((GVariant *)user_data,
-                       "(uuus)",
-                       (guint *)&object,
-                       (guint *)&invocation,
-                       &arg_sap,
-                       &arg_san);
+       g_variant_get((GVariant *)user_data, "(uuus)", (guint *)&object, (guint *)&invocation,
+                       &arg_sap, &arg_san);
 
        g_assert(object != NULL);
        g_assert(invocation != NULL);
 
        result = net_nfc_server_llcp_unregister_service(
-                       g_dbus_method_invocation_get_sender(invocation),
-                       arg_sap,
-                       arg_san);
+                       g_dbus_method_invocation_get_sender(invocation), arg_sap, arg_san);
 
-       net_nfc_gdbus_snep_complete_server_unregister(object,
-                       invocation,
-                       result);
+       net_nfc_gdbus_snep_complete_server_unregister(object, invocation, result);
 
        g_free(arg_san);
 
@@ -930,32 +817,31 @@ static gboolean _handle_unregister_server(
                const gchar *arg_san,
                GVariant *arg_privilege)
 {
-       GVariant *parameter;
+       bool ret;
        gboolean result;
+       GVariant *parameter;
 
-       NFC_INFO(">>> REQUEST from [%s]",
-                       g_dbus_method_invocation_get_sender(invocation));
+       NFC_INFO(">>> REQUEST from [%s]", g_dbus_method_invocation_get_sender(invocation));
 
        /* check privilege and update client context */
-       if (net_nfc_server_gdbus_check_privilege(invocation,
-                               arg_privilege,
-                               "nfc-manager::p2p",
-                               "rw") == false) {
+       ret = net_nfc_server_gdbus_check_privilege(invocation, arg_privilege,
+                               "nfc-manager::p2p", "rw");
+
+       if (false == ret)
+       {
                NFC_ERR("permission denied, and finished request");
 
                return FALSE;
        }
 
-       parameter = g_variant_new("(uuus)",
-                       GPOINTER_TO_UINT(g_object_ref(object)),
-                       GPOINTER_TO_UINT(g_object_ref(invocation)),
-                       arg_sap,
-                       arg_san);
+       parameter = g_variant_new("(uuus)", GPOINTER_TO_UINT(g_object_ref(object)),
+                       GPOINTER_TO_UINT(g_object_ref(invocation)), arg_sap, arg_san);
 
        if (parameter != NULL)
        {
-               if ((result = net_nfc_server_controller_async_queue_push(
-                                               snep_unregister_server_thread_func, parameter)) == FALSE)
+               result = net_nfc_server_controller_async_queue_push(
+                                               snep_unregister_server_thread_func, parameter);
+               if (FALSE == result)
                {
                        NFC_ERR("net_nfc_server_controller_async_queue_push failed");
 
@@ -974,8 +860,7 @@ static gboolean _handle_unregister_server(
                NFC_ERR("g_variant_new failed");
 
                g_dbus_method_invocation_return_dbus_error(invocation,
-                               "org.tizen.NetNfcService.Snep.MemoryError",
-                               "Out of memory");
+                               "org.tizen.NetNfcService.Snep.MemoryError", "Out of memory");
 
                result = FALSE;
        }
@@ -985,50 +870,35 @@ static gboolean _handle_unregister_server(
 
 gboolean net_nfc_server_snep_init(GDBusConnection *connection)
 {
-       GError *error = NULL;
        gboolean result;
+       GError *error = NULL;
 
        if (snep_skeleton)
                g_object_unref(snep_skeleton);
 
        snep_skeleton = net_nfc_gdbus_snep_skeleton_new();
 
-       g_signal_connect(snep_skeleton,
-                       "handle-server-register",
-                       G_CALLBACK(_handle_register_server),
-                       NULL);
-
-       g_signal_connect(snep_skeleton,
-                       "handle-server-unregister",
-                       G_CALLBACK(_handle_unregister_server),
-                       NULL);
-
-       g_signal_connect(snep_skeleton,
-                       "handle-server-start",
-                       G_CALLBACK(_handle_start_server),
-                       NULL);
-
-       g_signal_connect(snep_skeleton,
-                       "handle-client-start",
-                       G_CALLBACK(_handle_start_client),
-                       NULL);
-
-       g_signal_connect(snep_skeleton,
-                       "handle-client-request",
-                       G_CALLBACK(_handle_client_send_request),
-                       NULL);
-
-       g_signal_connect(snep_skeleton,
-                       "handle-stop-snep",
-                       G_CALLBACK(_handle_stop_snep),
-                       NULL);
-
-       result = g_dbus_interface_skeleton_export(
-                       G_DBUS_INTERFACE_SKELETON(snep_skeleton),
-                       connection,
-                       "/org/tizen/NetNfcService/Snep",
-                       &error);
-       if (result == FALSE)
+       g_signal_connect(snep_skeleton, "handle-server-register",
+                       G_CALLBACK(_handle_register_server), NULL);
+
+       g_signal_connect(snep_skeleton, "handle-server-unregister",
+                       G_CALLBACK(_handle_unregister_server), NULL);
+
+       g_signal_connect(snep_skeleton, "handle-server-start",
+                       G_CALLBACK(_handle_start_server), NULL);
+
+       g_signal_connect(snep_skeleton, "handle-client-start",
+                       G_CALLBACK(_handle_start_client), NULL);
+
+       g_signal_connect(snep_skeleton, "handle-client-request",
+                       G_CALLBACK(_handle_client_send_request), NULL);
+
+       g_signal_connect(snep_skeleton, "handle-stop-snep",
+                       G_CALLBACK(_handle_stop_snep), NULL);
+
+       result = g_dbus_interface_skeleton_export(G_DBUS_INTERFACE_SKELETON(snep_skeleton),
+                       connection, "/org/tizen/NetNfcService/Snep", &error);
+       if (FALSE == result)
        {
                g_error_free(error);