From: Piotr Kosko <p.kosko@samsung.com>
Date: Wed, 13 Jul 2016 08:34:05 +0000 (+0200)
Subject: [Messaging] Fixed SVACE issues
X-Git-Tag: submit/tizen/20160713.090136~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F63%2F79863%2F1;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git

[Messaging] Fixed SVACE issues

[Bug] Fixed issues 107586, 107587

[Verification] Code compiles without errors.

Change-Id: I9ef9bea7b189174726dfc9d993df5234c88122a8
Signed-off-by: Piotr Kosko <p.kosko@samsung.com>
---

diff --git a/src/messaging/email_manager.cc b/src/messaging/email_manager.cc
index cd275dbe..6fa77de9 100755
--- a/src/messaging/email_manager.cc
+++ b/src/messaging/email_manager.cc
@@ -1433,7 +1433,7 @@ std::string EmailManager::getMessageStatus(int id) {
   MessageStatus status = MessageStatus::STATUS_UNDEFINED;
 
   int ret = email_get_mail_data(id, &mail);
-  if (EMAIL_ERROR_NONE != ret ) {
+  if (EMAIL_ERROR_NONE != ret || !mail) {
     LoggerD("Failed to get data %d (%s)", ret, get_error_message(ret));
     return "";
   }
diff --git a/src/messaging/message.cc b/src/messaging/message.cc
index 196bf078..3f77486d 100755
--- a/src/messaging/message.cc
+++ b/src/messaging/message.cc
@@ -496,7 +496,10 @@ PlatformResult Message::convertPlatformEmail(std::shared_ptr<Message> message,
     }
 
     if(message->is_id_set()) {
-        email_get_mail_data(message->getId(), &mail_data);
+        int ret = email_get_mail_data(message->getId(), &mail_data);
+        if (EMAIL_ERROR_NONE != ret || !mail_data) {
+          return LogAndCreateResult(ErrorCode::UNKNOWN_ERR, "Failed to get mail data.");
+        }
     } else {
         mail_data = (email_mail_data_t*)malloc(sizeof(email_mail_data_t));
         if (!mail_data) {
@@ -505,7 +508,7 @@ PlatformResult Message::convertPlatformEmail(std::shared_ptr<Message> message,
         memset(mail_data, 0x00, sizeof(email_mail_data_t));
     }
 
-  std::unique_ptr<email_mail_data_t, void (*)(email_mail_data_t*)> mail_data_ptr(
+    std::unique_ptr<email_mail_data_t, void (*)(email_mail_data_t*)> mail_data_ptr(
       mail_data, [](email_mail_data_t* mail) {email_free_mail_data(&mail, 1);});
 
     if(!message->getFrom().empty()) {