Fix dlog format issues

[platform/core/connectivity/bluetooth-agent.git] / map-agent / bluetooth_map_email.c

diff --git a/map-agent/bluetooth_map_email.c b/map-agent/bluetooth_map_email.c
index 03c0ea2..09edbfb 100644 (file)
--- a/map-agent/bluetooth_map_email.c
+++ b/map-agent/bluetooth_map_email.c
@@ -68,6 +68,7 @@
 
 extern guint64 current_push_map_id;
 
+/* LCOV_EXCL_START */
 static void __bt_map_parse_moved_mails(char *inbuf, int *from_box_id,
                                int *to_box_id, GList **mail_list)
 {
@@ -253,7 +254,7 @@ static void __bt_email_subscription_callback(GDBusConnection *connection,
 
                                old_folder = __bt_email_get_path(mailbox_from->mailbox_type);
                                handle = _bt_add_id(mailid, BT_MAP_ID_EMAIL);
-                               DBG("[MessageDeleted] Handle:%d", handle);
+                               DBG("[MessageDeleted] Handle:%llu", handle);
                                _bt_mns_client_event_notify("MessageShift", handle,
                                                "TELECOM/MSG/DELETED", old_folder, "EMAIL");
                                g_free(old_folder);
@@ -272,7 +273,7 @@ static void __bt_email_subscription_callback(GDBusConnection *connection,
                                        break;
 
                                handle = _bt_add_id(mailid, BT_MAP_ID_EMAIL);
-                               DBG("[SendingSuccess] Handle:%d", handle);
+                               DBG("[SendingSuccess] Handle:%llu", handle);
 
                                _bt_mns_client_event_notify("MessageShift", handle,
                                                "TELECOM/MSG/SENT", "TELECOM/MSG/OUTBOX", "EMAIL");
@@ -287,6 +288,7 @@ static void __bt_email_subscription_callback(GDBusConnection *connection,
                email_free_mailbox(&mailbox_from, 1);
        }
 }
+/* LCOV_EXCL_STOP */
 
 gboolean _bt_map_start_email_service(void)
 {
@@ -300,6 +302,7 @@ gboolean _bt_map_start_email_service(void)
                return FALSE;
        }
 
+       /* LCOV_EXCL_START */
        dbus_conn = g_bus_get_sync(G_BUS_TYPE_SYSTEM, NULL, &error);
        if (error) {
                ERR("g_bus_get_sync() failed (%s)", error->message);
@@ -320,6 +323,7 @@ gboolean _bt_map_start_email_service(void)
                return FALSE;
        }
 
+       /* LCOV_EXCL_STOP */
        return TRUE;
 }
 
@@ -329,8 +333,10 @@ gboolean _bt_map_stop_email_service(void)
 
        err = email_service_end();
        if (err != EMAIL_ERROR_NONE) {
+               /* LCOV_EXCL_START */
                ERR("email_service_end fail  error = %d\n", err);
                return FALSE;
+               /* LCOV_EXCL_STOP */
        }
 
        return TRUE;
@@ -350,6 +356,7 @@ gboolean _bt_map_email_get_supported_folders(gboolean folders[FOLDER_COUNT][MSG_
        if (err != EMAIL_ERROR_NONE)
                return FALSE;
 
+       /* LCOV_EXCL_START */
        err = email_get_mailbox_list(account_id, EMAIL_MAILBOX_ALL,
                                &mailbox_list, &mailbox_count);
        if (err != EMAIL_ERROR_NONE)
@@ -392,9 +399,11 @@ gboolean _bt_map_email_get_supported_folders(gboolean folders[FOLDER_COUNT][MSG_
        if (mailbox_list != NULL)
                email_free_mailbox(&mailbox_list, mailbox_count);
 
+       /* LCOV_EXCL_STOP */
        return TRUE;
 }
 
+/* LCOV_EXCL_START */
 static message_info_t *__bt_email_info_get(email_mail_list_item_t *email_struct,
                                                        guint8 subject_len)
 {
@@ -412,7 +421,7 @@ static message_info_t *__bt_email_info_get(email_mail_list_item_t *email_struct,
 #else
        snprintf(email_handle, sizeof(email_handle), "%llx", uid);
 #endif
-       DBG("******* MAP ID:%d, MailID:%d **********", uid, email_struct->mail_id);
+       DBG("******* MAP ID:%llu, MailID:%d **********", uid, email_struct->mail_id);
        email_info->handle = g_strdup(email_handle);
 
        dptime = email_struct->date_time;
@@ -622,6 +631,7 @@ gboolean _bt_map_get_email_list(char *folder, int max,
        DBG("EXIT");
        return TRUE;
 }
+/* LCOV_EXCL_STOP */
 
 gboolean _bt_map_update_mailbox(char *folder)
 {
@@ -646,10 +656,11 @@ gboolean _bt_map_set_email_read_status(int mail_id, int read_status)
 
        ret = email_get_mail_data(mail_id, &mail_data);
        if (ret != EMAIL_ERROR_NONE) {
-               ERR("email_get_mail_data failed\n");
+               ERR("email_get_mail_data failed\n"); /* LCOV_EXCL_LINE */
                return FALSE;
        }
 
+       /* LCOV_EXCL_START */
        ret = email_set_flags_field(mail_data->account_id, &mail_id, 1,
                                EMAIL_FLAGS_SEEN_FIELD, read_status, 0);
        if (ret != EMAIL_ERROR_NONE) {
@@ -658,6 +669,7 @@ gboolean _bt_map_set_email_read_status(int mail_id, int read_status)
        }
 
        email_free_mail_data(&mail_data, 1);
+       /* LCOV_EXCL_STOP */
        return TRUE;
 }
 
@@ -670,6 +682,7 @@ gboolean _bt_map_set_email_delete_status(int mail_id, int read_status)
        if (ret != EMAIL_ERROR_NONE)
                return FALSE;
 
+       /* LCOV_EXCL_START */
        ret = email_delete_mail(mail_data->mailbox_id, &mail_id, 1, 1);
        if (ret != EMAIL_ERROR_NONE) {
                email_free_mail_data(&mail_data, 1);
@@ -677,9 +690,11 @@ gboolean _bt_map_set_email_delete_status(int mail_id, int read_status)
        }
 
        email_free_mail_data(&mail_data, 1);
+       /* LCOV_EXCL_STOP */
        return TRUE;
 }
 
+/* LCOV_EXCL_START */
 static gchar *__bt_get_email_folder_name(int mailboxtype)
 {
        switch (mailboxtype) {
@@ -722,8 +737,8 @@ static char *__bt_prepare_email_bmseg(email_mail_data_t *mail_data)
        g_free(folder);
 
        /* List of recepient & sender */
-       DBG("Sender: %d", mail_data->email_address_sender);
-       DBG("Sender Alias: %d", mail_data->alias_sender);
+       DBG("Sender: %s", mail_data->email_address_sender);
+       DBG("Sender Alias: %s", mail_data->alias_sender);
        g_string_append_printf(msg, EMAIL_VCARD, mail_data->email_address_sender,
                        mail_data->email_address_sender);
 
@@ -744,15 +759,20 @@ static char *__bt_prepare_email_bmseg(email_mail_data_t *mail_data)
        if (body_file != NULL) {
                fseek(body_file, 0, SEEK_END);
                email_size = ftell(body_file);
+               if (email_size <= 0) {
+                       ERR("email_size is not a positive number");
+                       g_string_free(msg, TRUE);
+                       fclose(body_file);
+                       return NULL;
+               }
                rewind(body_file);
 
-               buf = (char *)g_malloc0(sizeof(char) * email_size);
+               buf = (char *)g_malloc0(sizeof(char) * (email_size + 1));
                read_size = fread(buf, 1, email_size, body_file);
                fclose(body_file);
                DBG("MESSAGE: [%s]", buf);
                if (read_size != email_size) {
                        ERR("Unequal Read size");
-                       email_free_mail_data(&mail_data, 1);
                        g_string_free(msg, TRUE);
                        g_free(buf);
                        return NULL;
@@ -777,6 +797,7 @@ static char *__bt_prepare_email_bmseg(email_mail_data_t *mail_data)
        FN_END;
        return g_string_free(msg, FALSE);
 }
+/* LCOV_EXCL_STOP */
 
 gboolean _bt_map_get_email_message(int mail_id, gboolean attach,
                gboolean transcode, gboolean first_request, gchar **bmseg)
@@ -790,6 +811,7 @@ gboolean _bt_map_get_email_message(int mail_id, gboolean attach,
        if (ret != EMAIL_ERROR_NONE)
                return FALSE;
 
+       /* LCOV_EXCL_START */
        ret = email_get_mail_data(mail_id, &mail_data);
        if (ret != EMAIL_ERROR_NONE)
                return FALSE;
@@ -798,9 +820,11 @@ gboolean _bt_map_get_email_message(int mail_id, gboolean attach,
 
        email_free_mail_data(&mail_data, 1);
        DBG("EXIT==>");
+       /* LCOV_EXCL_STOP */
        return TRUE;
 }
 
+/* LCOV_EXCL_START */
 static int __bt_map_save_email_to_outbox(char *subject, char *body,
                char *recepients)
 {
@@ -1097,3 +1121,5 @@ done:
 
        return TRUE;
 }
+/* LCOV_EXCL_STOP */
+