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 c20128305da9b081c33247d4f7375261031acc33..8e944f4a7edfabac9c91eaf333c35e73ff5fafaa 100644 (file)
--- 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 a4d32a4a4eaf968556d0f34080231fdd2e9989a2..7ac4fba369dfbd486d8b0ea67cc3853bc089be68 100644 (file)
--- 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");