X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=vobject-engine%2FVCard.c;h=d0abf9770b312ead843557caaa69511bb8abd5f7;hb=bca0e9ec6458f92795ca1f032ae94325cf285914;hp=72735e44c3c75870bc3bfbedc7ee56018266413c;hpb=046d15077b1d6fc153da75f2ef1eb88b0bc0cc36;p=platform%2Fcore%2Fmessaging%2Fmsg-service.git

diff --git a/vobject-engine/VCard.c b/vobject-engine/VCard.c
index 72735e4..d0abf97 100755
--- 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));