From: Chakradhar Date: Sat, 22 Sep 2018 09:06:22 +0000 (+0530) Subject: Resolved coverity isssues X-Git-Tag: submit/tizen/20181116.094538^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0af19bdc3441509806a32e92ccb77d9bccd5d4d0;p=platform%2Fcore%2Fmessaging%2Fmsg-service.git Resolved coverity isssues Change-Id: If6fa903b55dbd314849fddeb1503c7521314f357 Signed-off-by: Chakradhar --- diff --git a/framework/deliver-handler/MsgDeliverHandler.cpp b/framework/deliver-handler/MsgDeliverHandler.cpp index c87d802..0ffeca5 100755 --- a/framework/deliver-handler/MsgDeliverHandler.cpp +++ b/framework/deliver-handler/MsgDeliverHandler.cpp @@ -248,7 +248,8 @@ msg_error_t MsgHandleIncomingMsg(MSG_MESSAGE_INFO_S *pMsgInfo, bool *pSendNoti) } int activated_conv_id = -1; - MsgSettingGetInt(VCONFKEY_MESSAGE_ACTIVATED_CONVERSATION_ID, &activated_conv_id); + if (MsgSettingGetInt(VCONFKEY_MESSAGE_ACTIVATED_CONVERSATION_ID, &activated_conv_id) != MSG_SUCCESS) + MSG_ERR("MsgSettingGetInt() is failed"); if (activated_conv_id > 0 && (unsigned int)activated_conv_id == pMsgInfo->threadId) { MSG_WARN("Update message read status of current activated thread. [%d]", pMsgInfo->threadId); diff --git a/manager/src/msg-manager-notification.cpp b/manager/src/msg-manager-notification.cpp index 909a0d6..3a9ec57 100644 --- a/manager/src/msg-manager-notification.cpp +++ b/manager/src/msg-manager-notification.cpp @@ -1374,7 +1374,11 @@ int MsgMgrInsertDeliveryReportInstantNotification(msg_mgr_notification_type_t no MSG_MGR_BEGIN(); notification_h noti = notification_create(NOTIFICATION_TYPE_NOTI); - notification_set_pkgname(noti, _get_app_id()); + int noti_err = notification_set_pkgname(noti, _get_app_id()); + + if(noti_err != NOTIFICATION_ERROR_NONE){ + MSG_MGR_ERR("Fail to notification_set_pkgname : %d", noti_err); + } setTextDomain(noti); setNotiText(noti, NOTIFICATION_TEXT_TYPE_TITLE, "Delivery report", DELIVERY_MESSAGE);