Coverity fix: check return type

[platform/core/messaging/msg-service.git] / vobject-engine / VCard.c

diff --git a/vobject-engine/VCard.c b/vobject-engine/VCard.c
index 72735e4..d0abf97 100755 (executable)
--- a/vobject-engine/VCard.c
+++ b/vobject-engine/VCard.c
@@ -240,7 +240,7 @@ __VCardGetValue(char* szString, const ValueObj list[], int size)
 
        for (i = 0; i < size-1; i++) {
                VDATA_TRACE(" i : %d", i);
-               VDATA_TRACE(" for loop %d < %d, list[%d] : %p, list[%d].szName : %p", i, size, i, list[i], i, list[i].szName);
+               VDATA_TRACE(" for loop %d < %d, list[%d] : %d, list[%d].szName : %p", i, size, i, list[i].flag, i, list[i].szName);
                VDATA_TRACE(" i : %d", i);
                if (list[i].szName != NULL) {
                        VDATA_TRACE(" list[%d].szName != NULL", i);
@@ -512,8 +512,8 @@ __VCardFreeVTreeMemory(VTree * pTree)
                                while (pCurParam != NULL) {
                                        pNextParam = pCurParam->pNext;
                                        VDATA_TRACE("pNEXT ==> %p", pCurParam->pNext);
-                                       VDATA_TRACE("pPARAM ==> %p", pCurParam->parameter);
-                                       VDATA_TRACE("pVALUE ==> %p", pCurParam->paramValue);
+                                       VDATA_TRACE("pPARAM ==> %d", pCurParam->parameter);
+                                       VDATA_TRACE("pVALUE ==> %d", pCurParam->paramValue);
                                        VDATA_TRACE("pCurParam : %p", pCurParam);
                                        VDATA_TRACE("pCurParam->parameter : %d", pCurParam->parameter);
                                        VDATA_TRACE("pCurParam->paramValue : %d", pCurParam->paramValue);
@@ -1072,6 +1072,7 @@ char* vcard_encode(VTree *pVCardRaw)
 {
        VDATA_TRACE_BEGINE
        char*           pVCardRes = NULL;
+       char*           pTempVCard = NULL;
        VObject *       pTmpObj =  NULL;
        char*           pTemp = NULL;
        int                     len;
@@ -1118,13 +1119,15 @@ char* vcard_encode(VTree *pVCardRaw)
                if ((pTemp = __VCardTypeEncode(pTmpObj, pszCardTypeList[pTmpObj->property])) != NULL) {
                        len = strlen(pTemp);
                        total += len + sizeof(char) * 10;
-                       if ((pVCardRes = (char*)realloc(pVCardRes, total)) == NULL) {
+                       if ((pTempVCard = (char*)realloc(pVCardRes, total)) == NULL) {
                                VDATA_TRACE("vcard_encode():realloc failed\n");
                                VFREE(pTemp);
                                pTemp = NULL;
+                               VFREE(pVCardRes);
                                VDATA_TRACE_END
                                return NULL;
                        }
+                       pVCardRes = pTempVCard;
 
                        if (strncmp(pTemp, "VERSION", strlen("VERSION")) != 0)
                                g_strlcat(pVCardRes, pTemp, total - strlen(pVCardRes));
@@ -1142,11 +1145,13 @@ char* vcard_encode(VTree *pVCardRaw)
        }
 
        total += sizeof(char) * 12;
-       if ((pVCardRes = (char *)realloc(pVCardRes, total)) == NULL) {
+       if ((pTempVCard = (char *)realloc(pVCardRes, total)) == NULL) {
                VDATA_TRACE("vcard_encode:realloc failed\n");
+               VFREE(pVCardRes);
                VDATA_TRACE_END
                return NULL;
        }
+       pVCardRes = pTempVCard;
        g_strlcat(pVCardRes, "END:VCARD\r\n", total - strlen(pVCardRes));
        VDATA_TRACE_END
        return pVCardRes;
@@ -1195,6 +1200,7 @@ __VCardTypeEncode(VObject *pTypeObj, char *pType)
        int                     len;
        char*           pTemp = NULL;
        char*           szTypeValue = NULL;
+       char*           szTemp = NULL;
        int                     i;
        int                     enc = 0;
        char*           pEncode = NULL;
@@ -1227,21 +1233,27 @@ __VCardTypeEncode(VObject *pTypeObj, char *pType)
        pTemp = __VCardParamEncode(pTypeObj, &enc);
        if (pTemp != NULL) {
                len = strlen(pTemp);
-               if ((szTypeValue = (char *)realloc(szTypeValue, (total += len))) == NULL) {
+               if ((szTemp = (char *)realloc(szTypeValue, (total += len))) == NULL) {
                        VDATA_TRACE("__VCardTypeEncode():realloc failed\n");
                        VFREE(pTemp);
-                       pTemp = NULL
-                       VDATA_TRACE_END;
+                       pTemp = NULL;
+                       VFREE(szTypeValue);
+                       VDATA_TRACE_END
                        return NULL;
+               } else {
+                       szTypeValue = szTemp;
                }
                g_strlcat(szTypeValue, pTemp, total - strlen(szTypeValue));
                VFREE(pTemp);
                pTemp = NULL;
        }
 
-       if ((szTypeValue = (char *)realloc(szTypeValue, (total += 2))) == NULL) {
+       if ((szTemp = (char *)realloc(szTypeValue, (total += 2))) == NULL) {
+               VFREE(szTypeValue);
                VDATA_TRACE_END
                return NULL;
+       } else {
+               szTypeValue = szTemp;
        }
 
        g_strlcat(szTypeValue, ":", total - strlen(szTypeValue));
@@ -1328,21 +1340,27 @@ __VCardTypeEncode(VObject *pTypeObj, char *pType)
                }
        }
 
-       if ((pRes = (char *)realloc(pRes, strlen(pRes) + 3)) == NULL) {
+       if ((szTemp = (char *)realloc(pRes, strlen(pRes) + 3)) == NULL) {
                VFREE(pEncode);
                VFREE(szTypeValue);
+               VFREE(pRes);
                VDATA_TRACE_END
                return NULL;
+       } else {
+               pRes = szTemp;
        }
        g_strlcat(pRes, "\r\n", 2);
 
        len = strlen(pRes);
 
-       if ((szTypeValue = (char *)realloc(szTypeValue, (total += (len+3)))) == NULL) {
+       if ((szTemp = (char *)realloc(szTypeValue, (total += (len+3)))) == NULL) {
                VFREE(pEncode);
                VFREE(pRes);
+               VFREE(szTypeValue);
                VDATA_TRACE_END
                return NULL;
+       } else {
+               szTypeValue = szTemp;
        }
 
        g_strlcat(szTypeValue, pRes, total - strlen(szTypeValue));
@@ -1373,6 +1391,7 @@ __VCardParamEncode(VObject* pTypeObj, int* pEnc)
        int shift = 0;
        bool bSupported;
        char* szParam = NULL;
+       char* szTemp = NULL;
        VParam* pTemp = NULL;
        ValueObj*       pList = NULL;
 
@@ -1396,9 +1415,12 @@ __VCardParamEncode(VObject* pTypeObj, int* pEnc)
                bSupported = false;
 
                /** Expand szParam string. For appending.*/
-               if ((szParam = (char *)realloc(szParam, len += 15)) == NULL) {
+               if ((szTemp = (char *)realloc(szParam, len += 15)) == NULL) {
+                       VFREE(szParam);
                        VDATA_TRACE_END
                        return NULL;
+               } else {
+                       szParam = szTemp;
                }
 
                /** appending paramter name. */
@@ -1428,9 +1450,12 @@ __VCardParamEncode(VObject* pTypeObj, int* pEnc)
                        pList = pCharsetList; bSupported = true;
                        break;
                default:
-                       if ((szParam = (char*)realloc(szParam, 5)) == NULL) {
+                       if ((szTemp = (char*)realloc(szParam, 5)) == NULL) {
+                               VFREE(szParam);
                                VDATA_TRACE_END
                                return NULL;
+                       } else {
+                               szParam = szTemp;
                        }
                        g_strlcat(szParam, "NONE", 5 - strlen(szParam));
                }
@@ -1441,9 +1466,12 @@ __VCardParamEncode(VObject* pTypeObj, int* pEnc)
                        for (i = 0, sNum = 0x00000001; i < shift; i++) {
 
                                if (pTemp->paramValue & sNum) {
-                                       if ((szParam = (char *)realloc(szParam, (len += (strlen(pList[i].szName) + 2)))) == NULL) {
+                                       if ((szTemp = (char *)realloc(szParam, (len += (strlen(pList[i].szName) + 2)))) == NULL) {
+                                               VFREE(szParam);
                                                VDATA_TRACE_END
                                                return NULL;
+                                       } else {
+                                               szParam = szTemp;
                                        }
 
                                        g_strlcat(szParam, pList[i].szName, len - strlen(szParam));