From: Jongkyu Koo <jk.koo@samsung.com>
Date: Fri, 16 Dec 2016 08:12:02 +0000 (+0900)
Subject: check return value of strdup()
X-Git-Tag: accepted/tizen/common/20161220.190908~5
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8add49bd39581fee67df731a46a418b11ea60b95;p=platform%2Fcore%2Fpim%2Fcontacts-service.git

check return value of strdup()

Change-Id: I7b9b8644c9641abb3d4ed4ed0b2565b11aae4d9f
Signed-off-by: Jongkyu Koo <jk.koo@samsung.com>
(cherry picked from commit 4892ea6d76d235ab4402f740a189621afb4eb99f)
---

diff --git a/common/ipc/ctsvc_ipc_marshal.c b/common/ipc/ctsvc_ipc_marshal.c
index 45c309a..fda5259 100644
--- a/common/ipc/ctsvc_ipc_marshal.c
+++ b/common/ipc/ctsvc_ipc_marshal.c
@@ -522,9 +522,13 @@ int ctsvc_ipc_unmarshal_string(const pims_ipc_data_h ipc_data, char **ppbufchar)
 	}
 
 	str = (char*)pims_ipc_data_get(ipc_data, &size);
-	if (str)
-		*ppbufchar = SAFE_STRDUP(str);
-
+	if (str) {
+		*ppbufchar = strdup(str);
+		if (NULL == *ppbufchar) {
+			ERR("strdup() Fail");
+			return CONTACTS_ERROR_SYSTEM;
+		}
+	}
 	return ret;
 }