[Messaging] Adding checking return value of calloc 23/139523/3
authorSzymon Jastrzebski <s.jastrzebsk@partner.samsung.com>
Wed, 19 Jul 2017 08:26:56 +0000 (10:26 +0200)
committerSzymon Jastrzebski <s.jastrzebsk@partner.samsung.com>
Wed, 19 Jul 2017 09:06:39 +0000 (09:06 +0000)
SVACE detected problem with code DEREF_OF_NULL.RET.ALLOC.

[Verification] Code compiles.

Change-Id: Ibf604637e9f9d08e8c486a421afd94bbe6d6bddc
Signed-off-by: Szymon Jastrzebski <s.jastrzebsk@partner.samsung.com>
src/messaging/message.cc

index 57703b13aa2718a044524488bca9fc9dc15461a9..14f3763096b060fa24b909e9e56cea91f3f143a4 100755 (executable)
@@ -592,6 +592,10 @@ PlatformResult addSingleEmailAttachment(std::shared_ptr<Message> message,
     if (ret.IsError()) return ret;
 
     email_attachment_data_t* tmp = (email_attachment_data_t*)calloc(1, sizeof(email_attachment_data_t));
+    if (nullptr == tmp) {
+      return LogAndCreateResult(
+                        ErrorCode::UNKNOWN_ERR, "Unknown error while adding attachment.");
+    }
     tmp->attachment_name = strdup(att->getShortFileName().c_str());
     tmp->attachment_path  = strdup(std::string(dirPath + "/"
             + att->getShortFileName()).c_str());