From: SeokHoon Lee <andy.shlee@samsung.com>
Date: Fri, 14 Apr 2017 01:55:34 +0000 (+0900)
Subject: fix sink create issue
X-Git-Tag: submit/tizen/20170516.041041~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b9de01093d0d9c919ad5bc55da65bc96b8bef332;p=platform%2Fcore%2Fapi%2Fscreen-mirroring.git

fix sink create issue

- fix pointer handle in scmirroring_sink_create.
- Fix ocurred problem, when sink create with NULL handle.

Signed-off-by: SeokHoon Lee <andy.shlee@samsung.com>
Change-Id: I6f026fa9f3a02e788045587a2542e6afc77e7c4d
---

diff --git a/packaging/capi-media-screen-mirroring.spec b/packaging/capi-media-screen-mirroring.spec
index c201283..8e944f4 100644
--- a/packaging/capi-media-screen-mirroring.spec
+++ b/packaging/capi-media-screen-mirroring.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-screen-mirroring
 Summary:    A screen mirroring library in Tizen C API
-Version:    0.1.72
+Version:    0.1.73
 Release:    0
 Group:      Multimedia/API
 License:    Apache-2.0
diff --git a/src/scmirroring_sink.c b/src/scmirroring_sink.c
index a4d32a4..7ac4fba 100644
--- a/src/scmirroring_sink.c
+++ b/src/scmirroring_sink.c
@@ -114,8 +114,11 @@ int scmirroring_sink_create(scmirroring_sink_h *scmirroring_sink)
 
 	scmirroring_retvm_if(scmirroring_sink == NULL, SCMIRRORING_ERROR_INVALID_PARAMETER, "scmirroring_sink is NULL");
 
-	scmirroring_sink_s *mhandle = (scmirroring_sink_s *)scmirroring_sink;
-	scmirroring_retvm_if(mhandle->magic_num == SCMIRRORING_MAGIC_NUMBER, SCMIRRORING_ERROR_INVALID_PARAMETER, "scmirroring_sink is created already");
+	scmirroring_sink_s *mhandle = (scmirroring_sink_s *)(*scmirroring_sink);
+	if (mhandle != NULL && mhandle->magic_num == SCMIRRORING_MAGIC_NUMBER) {
+		scmirroring_error("Already created handle");
+		return SCMIRRORING_ERROR_INVALID_PARAMETER;
+	}
 
 	scmirroring_sink_s *handle = (scmirroring_sink_s *)calloc(1, sizeof(scmirroring_sink_s));
 	scmirroring_retvm_if(handle == NULL, SCMIRRORING_ERROR_OUT_OF_MEMORY, "Fail to allocate memory for handle");