From: Jongkyu Koo <jk.koo@samsung.com>
Date: Thu, 31 Aug 2017 01:36:12 +0000 (+0900)
Subject: fix [ASan] Heap-buffer-overflow
X-Git-Tag: submit/tizen/20170929.023936~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5920e9816f13e81f7deda3c26e38e4c091fe80de;p=platform%2Fcore%2Fmessaging%2Fmsg-service.git

fix [ASan] Heap-buffer-overflow

Change-Id: Ia66384a915fdcbec9124a5b76a54f22d98fc219e
Signed-off-by: Jongkyu Koo <jk.koo@samsung.com>
---

diff --git a/vobject-engine/VMessage.c b/vobject-engine/VMessage.c
index f4a2ea4..4ee61c5 100755
--- a/vobject-engine/VMessage.c
+++ b/vobject-engine/VMessage.c
@@ -1222,11 +1222,16 @@ __VIsVmsgFile(char *pMsgRaw, int mode)
 	bool rtnValue = true;
 	char *pszVmsgBegin = "BEGIN:VMSG";
 
+	if (pMsgRaw == NULL)
+		return false;
+
 	switch (mode) {
 	case CHECK_START:
 		for (i = 0; i < 10; i++)
-			if (*pszVmsgBegin++ != *pMsgRaw++)
+			if (*pszVmsgBegin++ != *pMsgRaw++) {
 				rtnValue = false;
+				break;
+			}
 		break;
 
 	default: