Remove unused variable
[platform/core/api/camera.git] / src / camera.c
index ba855cf..e5c2403 100644 (file)
@@ -50,6 +50,20 @@ static int g_cam_dev_state_changed_cb_id;
 static GDBusConnection *g_cam_dev_state_changed_cb_conn;
 static guint g_cam_dev_state_changed_cb_subscribe_id;
 
+static void _camera_msg_send(int api, camera_cb_info_s *cb_info,
+       int *ret, int timeout);
+static void _camera_msg_send_param1(int api, camera_cb_info_s *cb_info,
+       int *ret, camera_msg_param *param, int timeout);
+static int _camera_import_tbm_key(tbm_bufmgr bufmgr, unsigned int tbm_key, tbm_bo *bo, tbm_bo_handle *bo_handle);
+static void _camera_release_imported_bo(tbm_bo *bo);
+static void _camera_preview_frame_create(camera_stream_data_s *stream, int num_buffer_key,
+       tbm_bo_handle *buffer_bo_handle, tbm_bo_handle *data_bo_handle, camera_preview_data_s *frame);
+static int _camera_media_packet_create(camera_cb_info_s *cb_info, camera_stream_data_s *stream,
+       camera_media_packet_data *mp_data, media_packet_h *packet);
+static int _camera_media_packet_data_create(int tbm_key, int num_buffer_key, tbm_bo bo,
+       tbm_bo *buffer_bo, tbm_bo data_bo, camera_media_packet_data **mp_data);
+static void _camera_media_packet_data_release(camera_media_packet_data *mp_data, camera_cb_info_s *cb_info);
+
 
 static void __global(void *data, struct wl_registry *registry,
        uint32_t name, const char *interface, uint32_t version)
@@ -205,8 +219,6 @@ int _camera_get_wl_info(Evas_Object *obj, camera_wl_info_s *wl_info)
                Ecore_Evas *ecore_evas = NULL;
                ret = CAMERA_ERROR_NONE;
 
-               wl_info->evas_obj = obj;
-
                evas_object_geometry_get(obj, &wl_info->window_x, &wl_info->window_y,
                        &wl_info->window_width, &wl_info->window_height);
 
@@ -226,7 +238,7 @@ int _camera_get_wl_info(Evas_Object *obj, camera_wl_info_s *wl_info)
                }
 
                LOGD("evas object : %p, rotation : %d, parent id : %u, window : %d,%d,%dx%d",
-                       wl_info->evas_obj, rotation, wl_info->parent_id,
+                       obj, rotation, wl_info->parent_id,
                        wl_info->window_x, wl_info->window_y,
                        wl_info->window_width, wl_info->window_height);
        } else {
@@ -333,6 +345,339 @@ _DONE:
 }
 
 
+static void __camera_event_handler_preview(camera_cb_info_s *cb_info, char *recv_msg)
+{
+       int i = 0;
+       int ret = 0;
+       int num_buffer_key = 0;
+       int buffer_key[BUFFER_MAX_PLANE_NUM] = {0, };
+       int tbm_key = 0;
+       int data_key = 0;
+       unsigned char *buf_pos = NULL;
+       camera_msg_param param;
+
+       tbm_bo bo = NULL;
+       tbm_bo buffer_bo[BUFFER_MAX_PLANE_NUM] = {NULL, };
+       tbm_bo_handle bo_handle = {.ptr = NULL};
+       tbm_bo_handle buffer_bo_handle[BUFFER_MAX_PLANE_NUM] = {{.ptr = NULL}, };
+       tbm_bo data_bo = NULL;
+       tbm_bo_handle data_bo_handle = {.ptr = NULL};
+
+       camera_preview_data_s frame;
+       camera_stream_data_s *stream = NULL;
+       camera_media_packet_data *mp_data = NULL;
+       media_packet_h pkt = NULL;
+#ifdef TIZEN_FEATURE_EVAS_RENDERER
+       media_packet_h pkt_evas = NULL;
+#endif /* TIZEN_FEATURE_EVAS_RENDERER */
+
+       if (!cb_info || !recv_msg) {
+               LOGE("invalid param %p %p", cb_info, recv_msg);
+               return;
+       }
+
+       muse_camera_msg_get(tbm_key, recv_msg);
+       muse_camera_msg_get(num_buffer_key, recv_msg);
+       muse_camera_msg_get_array(buffer_key, recv_msg);
+       muse_camera_msg_get(data_key, recv_msg);
+
+       memset(&frame, 0x0, sizeof(camera_preview_data_s));
+
+       if (tbm_key <= 0) {
+               LOGE("invalid key %d", tbm_key);
+               return;
+       }
+
+       CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
+
+       if (num_buffer_key < 0 || num_buffer_key > BUFFER_MAX_PLANE_NUM) {
+               LOGE("invalid num buffer key %d", num_buffer_key);
+               goto _PREVIEW_CB_HANDLER_DONE;
+       }
+
+       if (data_key > 0) {
+               /* import tbm data_bo and get virtual address */
+               if (!_camera_import_tbm_key(cb_info->bufmgr, data_key, &data_bo, &data_bo_handle)) {
+                       LOGE("failed to import data key %d", data_key);
+                       goto _PREVIEW_CB_HANDLER_DONE;
+               }
+       }
+
+       /* import tbm bo and get virtual address */
+       if (!_camera_import_tbm_key(cb_info->bufmgr, tbm_key, &bo, &bo_handle)) {
+               LOGE("failed to import key %d", tbm_key);
+               goto _PREVIEW_CB_HANDLER_DONE;
+       }
+
+       buf_pos = (unsigned char *)bo_handle.ptr;
+
+       /* get stream info */
+       stream = (camera_stream_data_s *)buf_pos;
+
+       for (i = 0 ; i < num_buffer_key ; i++) {
+               /* import buffer bo and get virtual address */
+               if (!_camera_import_tbm_key(cb_info->bufmgr, buffer_key[i], &buffer_bo[i], &buffer_bo_handle[i])) {
+                       LOGE("failed to import buffer key %d", buffer_key[i]);
+                       goto _PREVIEW_CB_HANDLER_DONE;
+               }
+       }
+
+       /* call preview callback */
+       if (cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_PREVIEW]) {
+               _camera_preview_frame_create(stream, num_buffer_key, buffer_bo_handle, &data_bo_handle, &frame);
+
+               ((camera_preview_cb)cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_PREVIEW])(&frame,
+                       cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_PREVIEW]);
+       }
+
+       if (CHECK_PREVIEW_CB(cb_info, PREVIEW_CB_TYPE_EVAS)) {
+#ifdef TIZEN_FEATURE_EVAS_RENDERER
+               ret = _camera_media_packet_data_create(tbm_key, num_buffer_key, bo, buffer_bo, data_bo, &mp_data);
+
+               if (ret == CAMERA_ERROR_NONE) {
+                       ret = _camera_media_packet_create(cb_info, stream, mp_data, &pkt_evas);
+                       if (ret != CAMERA_ERROR_NONE) {
+                               LOGE("create pkt for evas failed");
+                               _camera_media_packet_data_release(mp_data, cb_info);
+                               mp_data = NULL;
+                       }
+               }
+#else /* TIZEN_FEATURE_EVAS_RENDERER */
+               LOGW("evas renderer is not supported");
+#endif /* TIZEN_FEATURE_EVAS_RENDERER */
+       }
+
+       /* call media packet callback */
+       if (cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW]) {
+               ret = _camera_media_packet_data_create(tbm_key, num_buffer_key, bo, buffer_bo, data_bo, &mp_data);
+
+               if (ret == CAMERA_ERROR_NONE) {
+                       ret = _camera_media_packet_create(cb_info, stream, mp_data, &pkt);
+
+                       if (ret == CAMERA_ERROR_NONE) {
+                               ((camera_media_packet_preview_cb)cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW])(pkt,
+                                       cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW]);
+                       } else {
+                               _camera_media_packet_data_release(mp_data, cb_info);
+                               mp_data = NULL;
+                       }
+               }
+       }
+
+#ifdef TIZEN_FEATURE_EVAS_RENDERER
+       /* call evas renderer */
+       if (pkt_evas) {
+               g_mutex_lock(&cb_info->evas_mutex);
+               if (cb_info->run_evas_render) {
+                       mm_evas_renderer_write(pkt_evas, cb_info->evas_info);
+               } else {
+                       LOGW("evas renderer is stopped, skip this buffer...");
+                       media_packet_destroy(pkt_evas);
+               }
+               pkt_evas = NULL;
+               g_mutex_unlock(&cb_info->evas_mutex);
+       }
+#endif /* TIZEN_FEATURE_EVAS_RENDERER */
+
+       /* send message for preview callback return */
+       if (!CHECK_PREVIEW_CB(cb_info, PREVIEW_CB_TYPE_EVAS))
+               _camera_msg_send(MUSE_CAMERA_API_PREVIEW_CB_RETURN, cb_info, NULL, 0);
+
+_PREVIEW_CB_HANDLER_DONE:
+       if (mp_data == NULL) {
+               /* release imported bo */
+               for (i = 0 ; i < num_buffer_key ; i++)
+                       _camera_release_imported_bo(&buffer_bo[i]);
+
+               /* unmap and unref tbm bo */
+               _camera_release_imported_bo(&data_bo);
+               _camera_release_imported_bo(&bo);
+
+               /* return buffer */
+               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
+
+               /*LOGD("return buffer Done");*/
+       }
+
+       return;
+}
+
+
+static void __camera_event_handler_capture(camera_cb_info_s *cb_info, char *recv_msg)
+{
+       int tbm_key = 0;
+       int tbm_key_main = 0;
+       int tbm_key_post = 0;
+       int tbm_key_thumb = 0;
+       unsigned char *buf_pos = NULL;
+
+       camera_msg_param param;
+       camera_image_data_s *rImage = NULL;
+       camera_image_data_s *rPostview = NULL;
+       camera_image_data_s *rThumbnail = NULL;
+
+       tbm_bo bo_main = NULL;
+       tbm_bo bo_post = NULL;
+       tbm_bo bo_thumb = NULL;
+       tbm_bo_handle bo_main_handle = {.ptr = NULL};
+       tbm_bo_handle bo_post_handle = {.ptr = NULL};
+       tbm_bo_handle bo_thumb_handle = {.ptr = NULL};
+
+       if (!cb_info || !recv_msg) {
+               LOGE("invalid param %p %p", cb_info, recv_msg);
+               return;
+       }
+
+       muse_camera_msg_get(tbm_key_main, recv_msg);
+       muse_camera_msg_get(tbm_key_post, recv_msg);
+       muse_camera_msg_get(tbm_key_thumb, recv_msg);
+
+       /*
+       LOGD("camera capture callback came in. tbm_key_main %d, tbm_key_post %d, tbm_key_thumb %d",
+               tbm_key_main, tbm_key_post, tbm_key_thumb);
+       */
+
+       if (tbm_key_main <= 0) {
+               LOGE("invalid key %d", tbm_key_main);
+               goto _CAPTURE_CB_HANDLER_DONE;
+       }
+
+       if (cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_CAPTURE] == NULL) {
+               LOGW("NULL callback");
+               goto _CAPTURE_CB_HANDLER_DONE;
+       }
+
+       /* import tbm bo and get virtual address */
+       if (!_camera_import_tbm_key(cb_info->bufmgr, tbm_key_main, &bo_main, &bo_main_handle)) {
+               LOGE("failed to import key [%d] for main", tbm_key_main);
+               goto _CAPTURE_CB_HANDLER_DONE;
+       }
+
+       buf_pos = (unsigned char *)bo_main_handle.ptr;
+       rImage = (camera_image_data_s *)buf_pos;
+       rImage->data = buf_pos + sizeof(camera_image_data_s);
+       if (rImage->exif && rImage->exif_size > 0) {
+               rImage->exif = rImage->data + rImage->size;
+       } else {
+               rImage->exif = NULL;
+               rImage->exif_size = 0;
+       }
+
+       LOGD("image info %dx%d, size %d, EXIF info %p, size %d",
+               rImage->width, rImage->height, rImage->size, rImage->exif, rImage->exif_size);
+
+       if (tbm_key_post > 0) {
+               /* import tbm bo and get virtual address */
+               if (_camera_import_tbm_key(cb_info->bufmgr, tbm_key_post, &bo_post, &bo_post_handle)) {
+                       buf_pos = (unsigned char *)bo_post_handle.ptr;
+                       rPostview = (camera_image_data_s *)buf_pos;
+                       LOGD("rPostview->size : %d", rPostview->size);
+                       rPostview->data = buf_pos + sizeof(camera_image_data_s);
+               } else {
+                       LOGE("failed to import key [%d] for postview", tbm_key_post);
+               }
+       }
+
+       if (tbm_key_thumb > 0) {
+               /* import tbm bo and get virtual address */
+               if (_camera_import_tbm_key(cb_info->bufmgr, tbm_key_thumb, &bo_thumb, &bo_thumb_handle)) {
+                       buf_pos = (unsigned char *)bo_thumb_handle.ptr;
+                       rThumbnail = (camera_image_data_s *)buf_pos;
+                       LOGD("rThumbnail->size : %d", rThumbnail->size);
+                       rThumbnail->data = buf_pos + sizeof(camera_image_data_s);
+               } else {
+                       LOGE("failed to import key [%d] for thumbnail", tbm_key_thumb);
+               }
+       }
+
+       ((camera_capturing_cb)cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_CAPTURE])(rImage,
+               rPostview, rThumbnail, cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_CAPTURE]);
+
+_CAPTURE_CB_HANDLER_DONE:
+       /* return buffer */
+       if (tbm_key_main > 0) {
+               _camera_release_imported_bo(&bo_main);
+
+               tbm_key = tbm_key_main;
+               CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
+
+               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
+       }
+
+       if (tbm_key_post > 0) {
+               _camera_release_imported_bo(&bo_post);
+
+               tbm_key = tbm_key_post;
+               CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
+
+               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
+       }
+
+       if (tbm_key_thumb > 0) {
+               _camera_release_imported_bo(&bo_thumb);
+
+               tbm_key = tbm_key_thumb;
+               CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
+
+               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
+       }
+
+       LOGD("return buffer done");
+
+       return;
+}
+
+
+static void __camera_event_handler_face_detection(camera_cb_info_s *cb_info, char *recv_msg)
+{
+       int count = 0;
+       int tbm_key = 0;
+       camera_detected_face_s *faces = NULL;
+       camera_msg_param param;
+
+       tbm_bo bo = NULL;
+       tbm_bo_handle bo_handle = {.ptr = NULL};
+
+       if (!cb_info || !recv_msg) {
+               LOGE("invalid param %p %p", cb_info, recv_msg);
+               return;
+       }
+
+       muse_camera_msg_get(count, recv_msg);
+       muse_camera_msg_get(tbm_key, recv_msg);
+
+       if (count >= 0 && cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION]) {
+               LOGD("FACE_DETECTION - count %d, tbm_key %d", count, tbm_key);
+
+               if (tbm_key > 0) {
+
+                       if (_camera_import_tbm_key(cb_info->bufmgr, tbm_key, &bo, &bo_handle)) {
+                               /* set face info */
+                               faces = bo_handle.ptr;
+                       }
+               }
+
+               ((camera_face_detected_cb)cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION])(faces,
+                       count, cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION]);
+
+               /* release bo */
+               _camera_release_imported_bo(&bo);
+       } else {
+               LOGW("skip face detection message [count %d, key %d, cb %p",
+                       count, tbm_key, cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION]);
+       }
+
+       /* return buffer */
+       if (tbm_key > 0) {
+               CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
+               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
+               /*LOGD("return buffer done");*/
+       }
+
+       return;
+}
+
+
 static int _camera_import_tbm_key(tbm_bufmgr bufmgr, unsigned int tbm_key, tbm_bo *bo, tbm_bo_handle *bo_handle)
 {
        tbm_bo tmp_bo = NULL;
@@ -671,7 +1016,7 @@ int _camera_get_media_packet_mimetype(int in_format, media_format_mimetype_e *mi
        return CAMERA_ERROR_NONE;
 }
 
-void _camera_preview_frame_create(camera_stream_data_s *stream, int num_buffer_key,
+static void _camera_preview_frame_create(camera_stream_data_s *stream, int num_buffer_key,
        tbm_bo_handle *buffer_bo_handle, tbm_bo_handle *data_bo_handle, camera_preview_data_s *frame)
 {
        int total_size = 0;
@@ -787,7 +1132,7 @@ void _camera_preview_frame_create(camera_stream_data_s *stream, int num_buffer_k
        return;
 }
 
-int _camera_media_packet_data_create(int tbm_key, int num_buffer_key, tbm_bo bo,
+static int _camera_media_packet_data_create(int tbm_key, int num_buffer_key, tbm_bo bo,
        tbm_bo *buffer_bo, tbm_bo data_bo, camera_media_packet_data **mp_data)
 {
        int i = 0;
@@ -819,7 +1164,7 @@ int _camera_media_packet_data_create(int tbm_key, int num_buffer_key, tbm_bo bo,
        return ret;
 }
 
-void _camera_media_packet_data_release(camera_media_packet_data *mp_data, camera_cb_info_s *cb_info)
+static void _camera_media_packet_data_release(camera_media_packet_data *mp_data, camera_cb_info_s *cb_info)
 {
        int i = 0;
        int tbm_key = 0;
@@ -858,7 +1203,7 @@ void _camera_media_packet_data_release(camera_media_packet_data *mp_data, camera
        return;
 }
 
-int _camera_media_packet_create(camera_cb_info_s *cb_info, camera_stream_data_s *stream,
+static int _camera_media_packet_create(camera_cb_info_s *cb_info, camera_stream_data_s *stream,
        camera_media_packet_data *mp_data, media_packet_h *packet)
 {
        media_packet_h pkt = NULL;
@@ -1082,10 +1427,6 @@ static void _camera_client_user_callback(camera_cb_info_s *cb_info, char *recv_m
 {
        int param1 = 0;
        int param2 = 0;
-       int tbm_key = 0;
-       tbm_bo bo = NULL;
-       tbm_bo_handle bo_handle = {NULL, };
-       camera_msg_param param;
 
        if (!recv_msg || event >= MUSE_CAMERA_EVENT_TYPE_NUM) {
                LOGE("invalid parameter - camera msg %p, event %d", recv_msg, event);
@@ -1094,25 +1435,24 @@ static void _camera_client_user_callback(camera_cb_info_s *cb_info, char *recv_m
 
        /*LOGD("get camera msg %s, event %d", recv_msg, event);*/
 
-       if (event == MUSE_CAMERA_EVENT_TYPE_PREVIEW) {
-               if (!(CHECK_PREVIEW_CB(cb_info, PREVIEW_CB_TYPE_EVAS)) &&
-                       cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_PREVIEW] == NULL &&
-                       cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW] == NULL) {
-                       /* return preview callback */
-                       _camera_msg_send(MUSE_CAMERA_API_PREVIEW_CB_RETURN, cb_info, NULL, 0);
-
-                       /* return buffer */
-                       muse_camera_msg_get(tbm_key, recv_msg);
-                       LOGW("all preview callback from user are NULL - return key %d", tbm_key);
-                       if (tbm_key > 0) {
-                               CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
-                               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-                       }
+       g_mutex_lock(&cb_info->user_cb_mutex[event]);
+
+       if (cb_info->user_cb[event] == NULL) {
+               if (event != MUSE_CAMERA_EVENT_TYPE_PREVIEW &&
+                       event != MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW &&
+                       event != MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION &&
+                       event != MUSE_CAMERA_EVENT_TYPE_CAPTURE) {
+                       g_mutex_unlock(&cb_info->user_cb_mutex[event]);
+                       LOGW("NULL callback for event %d, return here", event);
                        return;
                }
-       } else if (cb_info->user_cb[event] == NULL) {
-               LOGW("user callback for event %d is not set", event);
-               return;
+
+               /* return buffer message should be sent for some events.
+                - MUSE_CAMERA_EVENT_TYPE_PREVIEW
+                - MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW
+                - MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION
+                - MUSE_CAMERA_EVENT_TYPE_CAPTURE
+               */
        }
 
        switch (event) {
@@ -1150,168 +1490,7 @@ static void _camera_client_user_callback(camera_cb_info_s *cb_info, char *recv_m
                break;
        case MUSE_CAMERA_EVENT_TYPE_PREVIEW:
        case MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW:
-               {
-                       int i = 0;
-                       int ret = 0;
-                       int num_buffer_key = 0;
-                       int buffer_key[BUFFER_MAX_PLANE_NUM] = {0, };
-                       int data_key = 0;
-                       unsigned char *buf_pos = NULL;
-
-                       tbm_bo buffer_bo[BUFFER_MAX_PLANE_NUM] = {NULL, };
-                       tbm_bo_handle buffer_bo_handle[BUFFER_MAX_PLANE_NUM] = {{.ptr = NULL}, };
-                       tbm_bo data_bo = NULL;
-                       tbm_bo_handle data_bo_handle = {.ptr = NULL};
-
-                       camera_preview_data_s frame;
-                       camera_stream_data_s *stream = NULL;
-                       camera_media_packet_data *mp_data = NULL;
-                       media_packet_h pkt = NULL;
-#ifdef TIZEN_FEATURE_EVAS_RENDERER
-                       media_packet_h pkt_evas = NULL;
-#endif /* TIZEN_FEATURE_EVAS_RENDERER */
-
-                       muse_camera_msg_get(tbm_key, recv_msg);
-                       muse_camera_msg_get(num_buffer_key, recv_msg);
-                       muse_camera_msg_get_array(buffer_key, recv_msg);
-                       muse_camera_msg_get(data_key, recv_msg);
-
-                       memset(&frame, 0x0, sizeof(camera_preview_data_s));
-
-                       if (tbm_key <= 0) {
-                               LOGE("invalid key %d", tbm_key);
-                               break;
-                       }
-
-                       CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
-
-                       if (num_buffer_key < 0 || num_buffer_key > BUFFER_MAX_PLANE_NUM) {
-                               LOGE("invalid num buffer key %d", num_buffer_key);
-                               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-                               break;
-                       }
-
-                       if (data_key > 0) {
-                               /* import tbm data_bo and get virtual address */
-                               if (!_camera_import_tbm_key(cb_info->bufmgr, data_key, &data_bo, &data_bo_handle)) {
-                                       LOGE("failed to import data key %d", data_key);
-                                       _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-                                       break;
-                               }
-                       }
-
-                       /* import tbm bo and get virtual address */
-                       if (!_camera_import_tbm_key(cb_info->bufmgr, tbm_key, &bo, &bo_handle)) {
-                               LOGE("failed to import key %d", tbm_key);
-                               _camera_release_imported_bo(&data_bo);
-                               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-                               break;
-                       }
-
-                       buf_pos = (unsigned char *)bo_handle.ptr;
-
-                       /* get stream info */
-                       stream = (camera_stream_data_s *)buf_pos;
-
-                       for (i = 0 ; i < num_buffer_key ; i++) {
-                               /* import buffer bo and get virtual address */
-                               if (!_camera_import_tbm_key(cb_info->bufmgr, buffer_key[i], &buffer_bo[i], &buffer_bo_handle[i])) {
-                                       LOGE("failed to import buffer key %d", buffer_key[i]);
-
-                                       /* release imported bo */
-                                       _camera_release_imported_bo(&data_bo);
-                                       _camera_release_imported_bo(&bo);
-
-                                       for (i -= 1 ; i >= 0 ; i--)
-                                               _camera_release_imported_bo(&buffer_bo[i]);
-
-                                       /* send return buffer */
-                                       _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-
-                                       return;
-                               }
-                       }
-
-                       /* call preview callback */
-                       g_mutex_lock(&cb_info->preview_cb_mutex);
-
-                       if (cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_PREVIEW]) {
-                               _camera_preview_frame_create(stream, num_buffer_key, buffer_bo_handle, &data_bo_handle, &frame);
-
-                               ((camera_preview_cb)cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_PREVIEW])(&frame,
-                                       cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_PREVIEW]);
-                       }
-
-                       g_mutex_unlock(&cb_info->preview_cb_mutex);
-
-                       if (CHECK_PREVIEW_CB(cb_info, PREVIEW_CB_TYPE_EVAS)) {
-#ifdef TIZEN_FEATURE_EVAS_RENDERER
-                               ret = _camera_media_packet_data_create(tbm_key, num_buffer_key, bo, buffer_bo, data_bo, &mp_data);
-
-                               if (ret == CAMERA_ERROR_NONE) {
-                                       ret = _camera_media_packet_create(cb_info, stream, mp_data, &pkt_evas);
-                                       if (ret != CAMERA_ERROR_NONE) {
-                                               LOGE("create pkt for evas failed");
-                                               _camera_media_packet_data_release(mp_data, cb_info);
-                                               mp_data = NULL;
-                                       }
-                               }
-#else /* TIZEN_FEATURE_EVAS_RENDERER */
-                               LOGW("evas renderer is not supported");
-#endif /* TIZEN_FEATURE_EVAS_RENDERER */
-                       }
-
-                       /* call media packet callback */
-                       if (cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW]) {
-                               ret = _camera_media_packet_data_create(tbm_key, num_buffer_key, bo, buffer_bo, data_bo, &mp_data);
-
-                               if (ret == CAMERA_ERROR_NONE) {
-                                       ret = _camera_media_packet_create(cb_info, stream, mp_data, &pkt);
-
-                                       if (ret == CAMERA_ERROR_NONE) {
-                                               ((camera_media_packet_preview_cb)cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW])(pkt,
-                                                       cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW]);
-                                       } else {
-                                               _camera_media_packet_data_release(mp_data, cb_info);
-                                               mp_data = NULL;
-                                       }
-                               }
-                       }
-
-#ifdef TIZEN_FEATURE_EVAS_RENDERER
-                       /* call evas renderer */
-                       if (pkt_evas) {
-                               g_mutex_lock(&cb_info->evas_mutex);
-                               if (cb_info->run_evas_render) {
-                                       mm_evas_renderer_write(pkt_evas, cb_info->evas_info);
-                               } else {
-                                       LOGW("evas renderer is stopped, skip this buffer...");
-                                       media_packet_destroy(pkt_evas);
-                               }
-                               pkt_evas = NULL;
-                               g_mutex_unlock(&cb_info->evas_mutex);
-                       }
-#endif /* TIZEN_FEATURE_EVAS_RENDERER */
-
-                       /* send message for preview callback return */
-                       if (!CHECK_PREVIEW_CB(cb_info, PREVIEW_CB_TYPE_EVAS))
-                               _camera_msg_send(MUSE_CAMERA_API_PREVIEW_CB_RETURN, cb_info, NULL, 0);
-
-                       if (mp_data == NULL) {
-                               /* release imported bo */
-                               for (i = 0 ; i < num_buffer_key ; i++)
-                                       _camera_release_imported_bo(&buffer_bo[i]);
-
-                               /* unmap and unref tbm bo */
-                               _camera_release_imported_bo(&data_bo);
-                               _camera_release_imported_bo(&bo);
-
-                               /* return buffer */
-                               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-
-                               /*LOGD("return buffer Done");*/
-                       }
-               }
+               __camera_event_handler_preview(cb_info, recv_msg);
                break;
        case MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS:
                {
@@ -1319,7 +1498,7 @@ static void _camera_client_user_callback(camera_cb_info_s *cb_info, char *recv_m
 
                        muse_camera_msg_get(percent, recv_msg);
 
-                       LOGD("HDR progress - %d \%", percent);
+                       LOGD("HDR progress - %d %%", percent);
 
                        ((camera_attr_hdr_progress_cb)cb_info->user_cb[event])(percent, cb_info->user_data[event]);
                }
@@ -1370,39 +1549,7 @@ static void _camera_client_user_callback(camera_cb_info_s *cb_info, char *recv_m
                }
                break;
        case MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION:
-               {
-                       int count = 0;
-                       camera_detected_face_s *faces = NULL;
-
-                       muse_camera_msg_get(count, recv_msg);
-                       muse_camera_msg_get(tbm_key, recv_msg);
-
-                       if (count >= 0) {
-                               LOGD("FACE_DETECTION - count %d, tbm_key %d", count, tbm_key);
-
-                               if (tbm_key > 0) {
-
-                                       if (_camera_import_tbm_key(cb_info->bufmgr, tbm_key, &bo, &bo_handle)) {
-                                               /* set face info */
-                                               faces = bo_handle.ptr;
-                                       }
-                               }
-
-                               ((camera_face_detected_cb)cb_info->user_cb[event])(faces, count, cb_info->user_data[event]);
-
-                               /* release bo */
-                               _camera_release_imported_bo(&bo);
-
-                               /* return buffer */
-                               if (tbm_key > 0) {
-                                       CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
-                                       _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-                                       /*LOGD("return buffer done");*/
-                               }
-                       } else {
-                               LOGE("invalid message - count %d, key %d", count, tbm_key);
-                       }
-               }
+               __camera_event_handler_face_detection(cb_info, recv_msg);
                break;
        case MUSE_CAMERA_EVENT_TYPE_ERROR:
                {
@@ -1479,113 +1626,15 @@ static void _camera_client_user_callback(camera_cb_info_s *cb_info, char *recv_m
                }
                break;
        case MUSE_CAMERA_EVENT_TYPE_CAPTURE:
-               {
-                       camera_image_data_s *rImage = NULL;
-                       camera_image_data_s *rPostview = NULL;
-                       camera_image_data_s *rThumbnail = NULL;
-                       unsigned char *buf_pos = NULL;
-                       int tbm_key_main = 0;
-                       int tbm_key_post = 0;
-                       int tbm_key_thumb = 0;
-                       tbm_bo bo_main = NULL;
-                       tbm_bo bo_post = NULL;
-                       tbm_bo bo_thumb = NULL;
-                       tbm_bo_handle bo_main_handle = {NULL, };
-                       tbm_bo_handle bo_post_handle = {NULL, };
-                       tbm_bo_handle bo_thumb_handle = {NULL, };
-
-                       muse_camera_msg_get(tbm_key_main, recv_msg);
-                       muse_camera_msg_get(tbm_key_post, recv_msg);
-                       muse_camera_msg_get(tbm_key_thumb, recv_msg);
-
-                       /*
-                       LOGD("camera capture callback came in. tbm_key_main %d, tbm_key_post %d, tbm_key_thumb %d",
-                               tbm_key_main, tbm_key_post, tbm_key_thumb);
-                       */
-
-                       if (tbm_key_main <= 0) {
-                               LOGE("invalid key %d", tbm_key_main);
-                               break;
-                       }
-
-                       /* import tbm bo and get virtual address */
-                       if (!_camera_import_tbm_key(cb_info->bufmgr, tbm_key_main, &bo_main, &bo_main_handle))
-                               break;
-
-                       buf_pos = (unsigned char *)bo_main_handle.ptr;
-                       rImage = (camera_image_data_s *)buf_pos;
-                       rImage->data = buf_pos + sizeof(camera_image_data_s);
-                       if (rImage->exif && rImage->exif_size > 0) {
-                               rImage->exif = rImage->data + rImage->size;
-                       } else {
-                               rImage->exif = NULL;
-                               rImage->exif_size = 0;
-                       }
-
-                       LOGD("image info %dx%d, size %d, EXIF info %p, size %d",
-                               rImage->width, rImage->height, rImage->size, rImage->exif, rImage->exif_size);
-
-                       if (tbm_key_post > 0) {
-                               /* import tbm bo and get virtual address */
-                               if (!_camera_import_tbm_key(cb_info->bufmgr, tbm_key_post, &bo_post, &bo_post_handle))
-                                       break;
-
-                               buf_pos = (unsigned char *)bo_post_handle.ptr;
-                               rPostview = (camera_image_data_s *)buf_pos;
-                               LOGD("rPostview->size : %d", rPostview->size);
-                               rPostview->data = buf_pos + sizeof(camera_image_data_s);
-                       }
-
-                       if (tbm_key_thumb > 0) {
-                               /* import tbm bo and get virtual address */
-                               if (!_camera_import_tbm_key(cb_info->bufmgr, tbm_key_thumb, &bo_thumb, &bo_thumb_handle))
-                                       break;
-
-                               buf_pos = (unsigned char *)bo_thumb_handle.ptr;
-
-                               rThumbnail = (camera_image_data_s *)buf_pos;
-                               LOGD("rThumbnail->size : %d", rThumbnail->size);
-                               rThumbnail->data = buf_pos + sizeof(camera_image_data_s);
-                       }
-
-                       ((camera_capturing_cb)cb_info->user_cb[event])(rImage, rPostview, rThumbnail, cb_info->user_data[event]);
-
-                       /* unmap and unref tbm bo */
-                       _camera_release_imported_bo(&bo_main);
-
-                       /* return buffer */
-                       tbm_key = tbm_key_main;
-
-                       CAMERA_MSG_PARAM_SET(param, INT, tbm_key);
-
-                       _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-
-                       if (tbm_key_post > 0) {
-                               /* unmap and unref tbm bo */
-                               _camera_release_imported_bo(&bo_post);
-
-                               /* return buffer */
-                               param.value.value_INT = tbm_key_post;
-                               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-                       }
-
-                       if (tbm_key_thumb > 0) {
-                               /* unmap and unref tbm bo */
-                               _camera_release_imported_bo(&bo_thumb);
-
-                               /* return buffer */
-                               param.value.value_INT = tbm_key_thumb;
-                               _camera_msg_send_param1(MUSE_CAMERA_API_RETURN_BUFFER, cb_info, NULL, &param, 0);
-                       }
-
-                       LOGD("return buffer done");
-               }
+               __camera_event_handler_capture(cb_info, recv_msg);
                break;
        default:
                LOGW("unhandled event %d", event);
                break;
        }
 
+       g_mutex_unlock(&cb_info->user_cb_mutex[event]);
+
        return;
 }
 
@@ -2083,6 +2132,11 @@ static void *_camera_msg_recv_func(gpointer data)
 
                        LOGW("incompleted message [len %d]", remained_length);
 
+                       if (remained_msg) {
+                               free(remained_msg);
+                               remained_msg = NULL;
+                       }
+
                        remained_msg = (char *)malloc(remained_length + 1);
                        if (remained_msg) {
                                strncpy(remained_msg, recv_msg + prev_pos, remained_length);
@@ -2240,11 +2294,13 @@ static camera_cb_info_s *_camera_client_callback_new(gint sockfd)
        g_mutex_init(&cb_info->idle_event_mutex);
        g_cond_init(&cb_info->idle_event_cond);
        g_mutex_init(&cb_info->mp_data_mutex);
-       g_mutex_init(&cb_info->preview_cb_mutex);
 #ifdef TIZEN_FEATURE_EVAS_RENDERER
        g_mutex_init(&cb_info->evas_mutex);
 #endif /* TIZEN_FEATURE_EVAS_RENDERER */
 
+       for (i = 0 ; i < MUSE_CAMERA_EVENT_TYPE_NUM ; i++)
+               g_mutex_init(&cb_info->user_cb_mutex[i]);
+
        /* message handler thread */
        if (!__create_msg_handler_thread(&cb_info->msg_handler_info,
                CAMERA_MESSAGE_HANDLER_TYPE_GENERAL, "camera_msg_handler", cb_info)) {
@@ -2291,11 +2347,13 @@ ErrorExit:
                __destroy_msg_handler_thread(&cb_info->preview_cb_info);
                __destroy_msg_handler_thread(&cb_info->capture_cb_info);
 
+               for (i = 0 ; i < MUSE_CAMERA_EVENT_TYPE_NUM ; i++)
+                       g_mutex_clear(&cb_info->user_cb_mutex[i]);
+
                g_mutex_clear(&cb_info->fd_lock);
                g_mutex_clear(&cb_info->idle_event_mutex);
                g_cond_clear(&cb_info->idle_event_cond);
                g_mutex_clear(&cb_info->mp_data_mutex);
-               g_mutex_clear(&cb_info->preview_cb_mutex);
 #ifdef TIZEN_FEATURE_EVAS_RENDERER
                g_mutex_clear(&cb_info->evas_mutex);
 #endif /* TIZEN_FEATURE_EVAS_RENDERER */
@@ -2330,11 +2388,13 @@ static void _camera_client_callback_destroy(camera_cb_info_s *cb_info)
        __destroy_msg_handler_thread(&cb_info->preview_cb_info);
        __destroy_msg_handler_thread(&cb_info->capture_cb_info);
 
+       for (i = 0 ; i < MUSE_CAMERA_EVENT_TYPE_NUM ; i++)
+               g_mutex_clear(&cb_info->user_cb_mutex[i]);
+
        g_mutex_clear(&cb_info->fd_lock);
        g_mutex_clear(&cb_info->idle_event_mutex);
        g_cond_clear(&cb_info->idle_event_cond);
        g_mutex_clear(&cb_info->mp_data_mutex);
-       g_mutex_clear(&cb_info->preview_cb_mutex);
 #ifdef TIZEN_FEATURE_EVAS_RENDERER
        g_mutex_clear(&cb_info->evas_mutex);
 #endif /* TIZEN_FEATURE_EVAS_RENDERER */
@@ -2995,11 +3055,17 @@ int camera_start_face_detection(camera_h camera, camera_face_detected_cb callbac
 
        LOGD("Enter");
 
-       pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION] = callback;
-       pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION] = user_data;
-
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
+       if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION]);
+
+               pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION] = callback;
+               pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION] = user_data;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION]);
+       }
+
        LOGD("ret : 0x%x", ret);
 
        return ret;
@@ -3020,6 +3086,15 @@ int camera_stop_face_detection(camera_h camera)
 
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
+       if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION]);
+
+               pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION] = NULL;
+               pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION] = NULL;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_FACE_DETECTION]);
+       }
+
        LOGD("ret : 0x%x", ret);
 
        return ret;
@@ -3796,12 +3871,12 @@ int camera_set_preview_cb(camera_h camera, camera_preview_cb callback, void *use
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
-               g_mutex_lock(&pc->cb_info->preview_cb_mutex);
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_PREVIEW]);
 
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_PREVIEW] = callback;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_PREVIEW] = user_data;
 
-               g_mutex_unlock(&pc->cb_info->preview_cb_mutex);
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_PREVIEW]);
 
                SET_PREVIEW_CB_TYPE(pc->cb_info, PREVIEW_CB_TYPE_USER);
        }
@@ -3828,12 +3903,12 @@ int camera_unset_preview_cb(camera_h camera)
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
-               g_mutex_lock(&pc->cb_info->preview_cb_mutex);
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_PREVIEW]);
 
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_PREVIEW] = NULL;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_PREVIEW] = NULL;
 
-               g_mutex_unlock(&pc->cb_info->preview_cb_mutex);
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_PREVIEW]);
 
                UNSET_PREVIEW_CB_TYPE(pc->cb_info, PREVIEW_CB_TYPE_USER);
        }
@@ -3870,8 +3945,12 @@ int camera_set_media_packet_preview_cb(camera_h camera, camera_media_packet_prev
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW] = callback;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW] = user_data;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -3896,8 +3975,12 @@ int camera_unset_media_packet_preview_cb(camera_h camera)
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW] = NULL;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW] = NULL;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_MEDIA_PACKET_PREVIEW]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -3922,8 +4005,12 @@ int camera_set_state_changed_cb(camera_h camera, camera_state_changed_cb callbac
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_STATE_CHANGE]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_STATE_CHANGE] = callback;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_STATE_CHANGE] = user_data;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_STATE_CHANGE]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -3948,8 +4035,12 @@ int camera_unset_state_changed_cb(camera_h camera)
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_STATE_CHANGE]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_STATE_CHANGE] = NULL;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_STATE_CHANGE] = NULL;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_STATE_CHANGE]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -3974,8 +4065,12 @@ int camera_set_interrupted_cb(camera_h camera, camera_interrupted_cb callback, v
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_INTERRUPTED]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_INTERRUPTED] = callback;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_INTERRUPTED] = user_data;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_INTERRUPTED]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -4000,8 +4095,12 @@ int camera_unset_interrupted_cb(camera_h camera)
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_INTERRUPTED]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_INTERRUPTED] = NULL;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_INTERRUPTED] = NULL;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_INTERRUPTED]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -4026,8 +4125,12 @@ int camera_set_interrupt_started_cb(camera_h camera, camera_interrupt_started_cb
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_INTERRUPT_STARTED]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_INTERRUPT_STARTED] = callback;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_INTERRUPT_STARTED] = user_data;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_INTERRUPT_STARTED]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -4052,8 +4155,12 @@ int camera_unset_interrupt_started_cb(camera_h camera)
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_INTERRUPT_STARTED]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_INTERRUPT_STARTED] = NULL;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_INTERRUPT_STARTED] = NULL;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_INTERRUPT_STARTED]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -4078,8 +4185,12 @@ int camera_set_focus_changed_cb(camera_h camera, camera_focus_changed_cb callbac
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_FOCUS_CHANGE]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_FOCUS_CHANGE] = callback;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_FOCUS_CHANGE] = user_data;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_FOCUS_CHANGE]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -4104,8 +4215,12 @@ int camera_unset_focus_changed_cb(camera_h camera)
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_FOCUS_CHANGE]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_FOCUS_CHANGE] = NULL;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_FOCUS_CHANGE] = NULL;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_FOCUS_CHANGE]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -4130,8 +4245,12 @@ int camera_set_error_cb(camera_h camera, camera_error_cb callback, void *user_da
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_ERROR]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_ERROR] = callback;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_ERROR] = user_data;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_ERROR]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -4156,8 +4275,12 @@ int camera_unset_error_cb(camera_h camera)
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_ERROR]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_ERROR] = NULL;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_ERROR] = NULL;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_ERROR]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -6050,8 +6173,12 @@ int camera_attr_set_hdr_capture_progress_cb(camera_h camera, camera_attr_hdr_pro
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS] = callback;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS] = user_data;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -6076,8 +6203,12 @@ int camera_attr_unset_hdr_capture_progress_cb(camera_h camera)
        _camera_msg_send(api, pc->cb_info, &ret, CAMERA_CB_TIMEOUT);
 
        if (ret == CAMERA_ERROR_NONE) {
+               g_mutex_lock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS]);
+
                pc->cb_info->user_cb[MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS] = NULL;
                pc->cb_info->user_data[MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS] = NULL;
+
+               g_mutex_unlock(&pc->cb_info->user_cb_mutex[MUSE_CAMERA_EVENT_TYPE_HDR_PROGRESS]);
        }
 
        LOGD("ret : 0x%x", ret);
@@ -6394,7 +6525,7 @@ int camera_attr_get_pan_range(camera_h camera, int *min, int *max)
        muse_camera_api_e api = MUSE_CAMERA_API_ATTR_GET_PAN_RANGE;
 
        if (!pc || !pc->cb_info || !min || !max) {
-               LOGE("NULL pointer %p %p", pc, min, max);
+               LOGE("NULL pointer %p %p %p", pc, min, max);
                return CAMERA_ERROR_INVALID_PARAMETER;
        }