Coverity related changes 63/228063/1 accepted/tizen/unified/20200323.173046 submit/tizen/20200318.140541 submit/tizen/20200319.061939
authorkamaljeet chauhan <kamal.jc@samsung.com>
Wed, 18 Mar 2020 14:01:20 +0000 (19:31 +0530)
committerkamaljeet chauhan <kamal.jc@samsung.com>
Wed, 18 Mar 2020 14:01:20 +0000 (19:31 +0530)
Change-Id: I3788ad9133cb8b6b4047b30b799af1339c0357f0
Signed-off-by: kamaljeet chauhan <kamal.jc@samsung.com>
plugin/sms_plugin/3gpp2/Sms3gpp2Setting.cpp
plugin/sms_plugin/3gpp2/Sms3gpp2Transport.cpp
proxy/MsgHandleSetting.cpp

index 2fa34ed..1c582d5 100755 (executable)
@@ -98,7 +98,8 @@ void Sms3gpp2Setting::initializeSimInfo(TapiHandle *handle)
                MSG_DEBUG("tel_get_sim_imsi() Error![%d]", tapiRet);
        }
 
-       MsgSettingSetString(MSG_SIM_IMSI, imsi);
+       if (MsgSettingSetString(MSG_SIM_IMSI, imsi) != MSG_SUCCESS)
+               MSG_DEBUG("MsgSettingSetString failed");
 
        if (pthread_create(&thd, NULL, &initSimInfo, handle) < 0) {
                MSG_ERR("pthread_create() error");
index d0facae..5d09046 100755 (executable)
@@ -197,7 +197,8 @@ unsigned char Sms3gpp2Transport::getSubmitMsgId()
 {
        msgSubmitId = ((msgSubmitId + 1) % SMS_MAX_MESSAGE_ID);
 
-       MsgSettingSetInt(MSG_MESSAGE_ID_COUNTER, msgSubmitId);
+       if (MsgSettingSetInt(MSG_MESSAGE_ID_COUNTER, msgSubmitId) != MSG_SUCCESS)
+               MSG_DEBUG("MsgSettingSetInt failed");
 
        return msgSubmitId;
 }
index da80acb..e76f27f 100755 (executable)
@@ -532,7 +532,7 @@ msg_error_t MsgHandle::getMmsRecvOpt(msg_struct_t msg_struct)
        }
 
        if (pEvent->result == MSG_SUCCESS) {
-               memcpy(mms_recv_opt->data, pEvent->data, sizeof(MSG_MMS_RECVOPT_S));
+               memcpy(mms_recv_opt->data, pEvent->data, sizeof(pEvent->data));
        }
 
        return pEvent->result;