Add debug message temporarily

[platform/core/uifw/autofill-daemon.git] / src / autofill-daemon.c

diff --git a/src/autofill-daemon.c b/src/autofill-daemon.c
index fbda668..89e0095 100644 (file)
--- a/src/autofill-daemon.c
+++ b/src/autofill-daemon.c
@@ -234,24 +234,32 @@ bool __view_info_item_cb(rpc_port_autofill_item_h items, void *user_data)
 
     rpc_port_autofill_svc_view_info_h svi = (rpc_port_autofill_svc_view_info_h)user_data;
 
-    rpc_port_autofill_svc_item_h svc_item;
+    rpc_port_autofill_svc_item_h svc_item = NULL;
 
-    rpc_port_autofill_svc_item_create(&svc_item);
+    if (rpc_port_autofill_svc_item_create(&svc_item) != 0) {
+        LOGW("Failed to create autofill item");
+        return true;
+    }
+
+    LOGD("item : %p", svc_item);
 
     rpc_port_autofill_item_get_id(items, &id);
     rpc_port_autofill_svc_item_set_id(svc_item, id);
+    LOGD("id : %s", id);
     if (id) {
         free(id);
     }
 
     rpc_port_autofill_item_get_label(items, &label);
     rpc_port_autofill_svc_item_set_label(svc_item, label);
+    LOGD("label : %s", label);
     if (label) {
         free(label);
     }
 
     rpc_port_autofill_item_get_value(items, &value);
     rpc_port_autofill_svc_item_set_value(svc_item, value);
+    LOGD("value : %s", value);
     if (value) {
         free(value);
     }
@@ -262,7 +270,9 @@ bool __view_info_item_cb(rpc_port_autofill_item_h items, void *user_data)
     rpc_port_autofill_item_get_is_sensitive_data(items, &sensitive_data);
     rpc_port_autofill_svc_item_set_is_sensitive_data(svc_item, sensitive_data);
 
-    rpc_port_autofill_svc_view_info_add_items(svi, svc_item);
+    if (rpc_port_autofill_svc_view_info_add_items(svi, svc_item) != 0) {
+        LOGW("Failed to add item");
+    }
 
     rpc_port_autofill_svc_item_destroy(svc_item);
 
@@ -279,9 +289,10 @@ bool __save_item_cb(rpc_port_autofill_save_item_h items, void *user_data)
 
     rpc_port_autofill_svc_save_view_info_h svi = (rpc_port_autofill_svc_save_view_info_h)user_data;
 
-    rpc_port_autofill_svc_save_item_h svc_save_item;
+    rpc_port_autofill_svc_save_item_h svc_save_item = NULL;
 
-    rpc_port_autofill_svc_save_item_create(&svc_save_item);
+    if (rpc_port_autofill_svc_save_item_create(&svc_save_item) != 0)
+        return true;
 
     rpc_port_autofill_save_item_get_id(items, &id);
     rpc_port_autofill_svc_save_item_set_id(svc_save_item, id);
@@ -317,7 +328,6 @@ bool __save_item_cb(rpc_port_autofill_save_item_h items, void *user_data)
 static int __auth_info_request_cb(rpc_port_stub_AutofillAppPort_context_h context, int context_id, rpc_port_autofill_view_info_h vi, void *user_data)
 {
     char *sender = NULL;
-    autofill_client_s *sender_client;
 
     if (!svc_rpc_h) {
         LOGW("Not initialized");
@@ -330,11 +340,12 @@ static int __auth_info_request_cb(rpc_port_stub_AutofillAppPort_context_h contex
     rpc_port_autofill_view_info_get_view_id(vi, &view_id);
     LOGD("app id : %s, view id : %s", sender, view_id);
 
-    rpc_port_stub_AutofillAppPort_context_get_tag(context, (void *)&sender_client);
-
     /* create view info */
-    rpc_port_autofill_svc_view_info_h svi;
-    rpc_port_autofill_svc_view_info_create(&svi);
+    rpc_port_autofill_svc_view_info_h svi = NULL;
+    if (rpc_port_autofill_svc_view_info_create(&svi) != 0) {
+        goto end;
+    }
+
     rpc_port_autofill_svc_view_info_set_app_id(svi, sender);
     rpc_port_autofill_svc_view_info_set_view_id(svi, view_id);
 
@@ -342,6 +353,9 @@ static int __auth_info_request_cb(rpc_port_stub_AutofillAppPort_context_h contex
 
     rpc_port_proxy_AutofillSvcPort_invoke_request_auth_info(svc_rpc_h, context_id, svi);
 
+    rpc_port_autofill_svc_view_info_destroy(svi);
+
+end:
     if (sender) {
         free(sender);
     }
@@ -368,8 +382,11 @@ static int __autofill_fill_request_cb(rpc_port_stub_AutofillAppPort_context_h co
 
     LOGD("app id : %s, view id : %s, context id : %d", sender, view_id, context_id);
 
-    rpc_port_autofill_svc_view_info_h svi;
-    rpc_port_autofill_svc_view_info_create(&svi);
+    rpc_port_autofill_svc_view_info_h svi = NULL;
+    if (rpc_port_autofill_svc_view_info_create(&svi) != 0) {
+        goto end;
+    }
+
     rpc_port_autofill_svc_view_info_set_app_id(svi, sender);
     rpc_port_autofill_svc_view_info_set_view_id(svi, view_id);
 
@@ -377,21 +394,22 @@ static int __autofill_fill_request_cb(rpc_port_stub_AutofillAppPort_context_h co
 
     rpc_port_proxy_AutofillSvcPort_invoke_send_fill_request(svc_rpc_h, context_id, svi);
 
+    rpc_port_autofill_svc_view_info_destroy(svi);
+
+end:
     if (sender)
         free(sender);
 
     if (view_id)
         free(view_id);
 
-    rpc_port_autofill_svc_view_info_destroy(svi);
-
     return 0;
 }
 
 static int __commit_cb(rpc_port_stub_AutofillAppPort_context_h context, int context_id, rpc_port_autofill_save_view_info_h vi, void *user_data)
 {
     char *sender = NULL;
-    autofill_client_s *sender_client;
+    char *view_id = NULL;
 
     if (!svc_rpc_h) {
         LOGW("Not initialized");
@@ -404,17 +422,17 @@ static int __commit_cb(rpc_port_stub_AutofillAppPort_context_h context, int cont
         free(sender);
     }
 
-    char *view_id = NULL;
+    rpc_port_autofill_svc_save_view_info_h svi = NULL;
+    if (rpc_port_autofill_svc_save_view_info_create(&svi) != 0) {
+        return 0;
+    }
+
     rpc_port_autofill_save_view_info_get_view_id(vi, &view_id);
     if (view_id) {
         LOGD("view id : %s", view_id);
     }
-
-    rpc_port_autofill_svc_save_view_info_h svi;
-    rpc_port_autofill_svc_save_view_info_create(&svi);
     rpc_port_autofill_svc_save_view_info_set_view_id(svi, view_id);
 
-    rpc_port_stub_AutofillAppPort_context_get_tag(context, (void *)&sender_client);
     rpc_port_autofill_save_view_info_foreach_items(vi, __save_item_cb, svi);
 
     rpc_port_proxy_AutofillSvcPort_invoke_commit(svc_rpc_h, context_id, svi);
@@ -435,9 +453,11 @@ bool fill_response_item_cb(rpc_port_autofill_svc_response_item_h response_items,
     char *presentation_text = NULL;
     char *value = NULL;
 
-    rpc_port_autofill_response_item_h res_item;
+    rpc_port_autofill_response_item_h res_item = NULL;
 
-    rpc_port_autofill_response_item_create(&res_item);
+    if (rpc_port_autofill_response_item_create(&res_item) != 0) {
+        return true;
+    }
 
     rpc_port_autofill_svc_response_item_get_id(response_items, &id);
     rpc_port_autofill_response_item_set_id(res_item, id);
@@ -468,8 +488,9 @@ bool fill_response_group_cb(rpc_port_autofill_svc_response_group_h response_grou
 {
     rpc_port_autofill_fill_response_h fr_h = (rpc_port_autofill_fill_response_h)user_data;
 
-    rpc_port_autofill_response_group_h res_group;
-    rpc_port_autofill_response_group_create(&res_group);
+    rpc_port_autofill_response_group_h res_group = NULL;
+    if (rpc_port_autofill_response_group_create(&res_group) != 0)
+        return true;
 
     rpc_port_autofill_svc_response_group_foreach_response_items(response_groups, fill_response_item_cb, res_group);
 
@@ -486,12 +507,14 @@ static void __fill_response_recv_cb(void *user_data, int context_id, rpc_port_au
     char *view_id = NULL;
     char *app_id = NULL;
 
+    /* create autofill response */
+    rpc_port_autofill_fill_response_h fill_response_h = NULL;
+    if (rpc_port_autofill_fill_response_create(&fill_response_h) != 0)
+        return;
+
     rpc_port_autofill_svc_fill_response_get_app_id(response_h, &app_id);
     rpc_port_autofill_svc_fill_response_get_view_id(response_h, &view_id);
 
-    /* create autofill response */
-    rpc_port_autofill_fill_response_h fill_response_h;
-    rpc_port_autofill_fill_response_create(&fill_response_h);
     rpc_port_autofill_fill_response_set_view_id(fill_response_h, view_id);
 
     rpc_port_autofill_svc_fill_response_foreach_response_groups(response_h, fill_response_group_cb, fill_response_h);
@@ -511,8 +534,8 @@ static void __fill_response_recv_cb(void *user_data, int context_id, rpc_port_au
 
 static void __auth_info_recv_cb(void *user_data, int context_id, rpc_port_autofill_svc_auth_info_h svc_auth_info_h)
 {
-    bool exist_autofill_data;
-    bool need_authentication;
+    bool exist_autofill_data = false;
+    bool need_authentication = false;
     char *service_name = NULL;
     char *service_logo_image_path = NULL;
     char *service_message = NULL;
@@ -530,7 +553,7 @@ static void __auth_info_recv_cb(void *user_data, int context_id, rpc_port_autofi
     LOGD("app id : %s, service name : %s, message : %s, logo path : %s", app_id, service_name, service_message, service_logo_image_path);
 
     /* transfer auth info */
-    rpc_port_autofill_auth_info_h auth_info_h;
+    rpc_port_autofill_auth_info_h auth_info_h = NULL;
     rpc_port_autofill_auth_info_create(&auth_info_h);
     rpc_port_autofill_auth_info_set_view_id(auth_info_h, view_id);
     rpc_port_autofill_auth_info_set_exist_autofill_data(auth_info_h, exist_autofill_data);
@@ -617,7 +640,7 @@ static char * __manager_get_autofill_service_cb(rpc_port_stub_AutofillManagerPor
         return false;
     }
 
-    char *app_id;
+    char *app_id = NULL;
     autofill_config_get_autofill_service_app_id(&app_id);
 
     LOGD("app id : %s", app_id);
@@ -667,7 +690,7 @@ static bool __manager_get_autofill_service_list_cb(rpc_port_stub_AutofillManager
         return false;
     }
 
-    rpc_port_list_string_h app_id_list_h;
+    rpc_port_list_string_h app_id_list_h = NULL;
     rpc_port_list_string_create(&app_id_list_h);
 
     ret = app_info_metadata_filter_foreach(handle, add_autofill_service_cb, app_id_list_h);
@@ -685,7 +708,7 @@ static bool __manager_get_autofill_service_list_cb(rpc_port_stub_AutofillManager
 static int connect_service()
 {
     int ret;
-    size_t service_id_len;
+    size_t service_id_len = 0;
 
     rpc_port_proxy_AutofillSvcPort_callback_s rpc_callback = {
         .connected = __on_connected,